-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[minor] update HiveExternalCatalogVersionsSuite to test 2.4.0 #22984
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| object PROCESS_TABLES extends QueryTest with SQLTestUtils { | ||
| // Tests the latest version of every release line. | ||
| val testingVersions = Seq("2.1.3", "2.2.2", "2.3.2") | ||
| val testingVersions = Seq("2.1.3", "2.2.2", "2.3.2", "2.4.0") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when will we drop 2.1 officially?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2.1 should be removed actually. At least according to our default 18-month support rule, it ended support in June 2018, and I see no desire for a 2.1.4 release. 2.2 will end in Jan 2019 according to the same guideline. It's not a hard rule, but probably the right guidance to follow.
Certainly, as this test takes a while and can be flaky I'd like to keep it to 3 versions.
|
Test build #98608 has finished for PR 22984 at commit
|
|
Test build #98604 has finished for PR 22984 at commit
|
|
Retest this please. |
|
Test build #98617 has finished for PR 22984 at commit
|
|
It's an irrelevant failure. |
|
Retest this please. |
|
Test build #98626 has finished for PR 22984 at commit
|
|
Thank you, @cloud-fan and @srowen . Merged to master/branch-2.4. |
## What changes were proposed in this pull request? Since Spark 2.4.0 is released, we should test it in HiveExternalCatalogVersionsSuite ## How was this patch tested? N/A Closes #22984 from cloud-fan/minor. Authored-by: Wenchen Fan <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]> (cherry picked from commit 973f7c0) Signed-off-by: Dongjoon Hyun <[email protected]>
## What changes were proposed in this pull request? Since Spark 2.4.0 is released, we should test it in HiveExternalCatalogVersionsSuite ## How was this patch tested? N/A Closes apache#22984 from cloud-fan/minor. Authored-by: Wenchen Fan <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
## What changes were proposed in this pull request? Since Spark 2.4.0 is released, we should test it in HiveExternalCatalogVersionsSuite ## How was this patch tested? N/A Closes apache#22984 from cloud-fan/minor. Authored-by: Wenchen Fan <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]> (cherry picked from commit 973f7c0) Signed-off-by: Dongjoon Hyun <[email protected]>
## What changes were proposed in this pull request? Since Spark 2.4.0 is released, we should test it in HiveExternalCatalogVersionsSuite ## How was this patch tested? N/A Closes apache#22984 from cloud-fan/minor. Authored-by: Wenchen Fan <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]> (cherry picked from commit 973f7c0) Signed-off-by: Dongjoon Hyun <[email protected]>
What changes were proposed in this pull request?
Since Spark 2.4.0 is released, we should test it in HiveExternalCatalogVersionsSuite
How was this patch tested?
N/A