[GLUTEN-6067] Open spark 35 ut#8555
Conversation
|
Run Gluten Clickhouse CI on x86 |
f8884d0 to
4d4c31e
Compare
|
Run Gluten Clickhouse CI on x86 |
1 similar comment
|
Run Gluten Clickhouse CI on x86 |
|
Run Gluten Clickhouse CI on x86 |
2 similar comments
|
Run Gluten Clickhouse CI on x86 |
|
Run Gluten Clickhouse CI on x86 |
575d52c to
c493dad
Compare
|
Run Gluten Clickhouse CI on x86 |
c493dad to
88c4afb
Compare
|
Run Gluten Clickhouse CI on x86 |
|
@CodiumAI-Agent /review |
PR Reviewer Guide 🔍(Review updated until commit 590892c)Here are some key observations to aid the review process:
|
88c4afb to
590892c
Compare
|
Run Gluten Clickhouse CI on x86 |
|
Persistent review updated to latest commit 590892c |
|
Run Gluten Clickhouse CI on x86 |
6961d99 to
020fa16
Compare
|
Run Gluten Clickhouse CI on x86 |
020fa16 to
3e03543
Compare
|
Run Gluten Clickhouse CI on x86 |
| def includeCH(testNames: String*): SuiteSettings = { | ||
| this | ||
| } |
There was a problem hiding this comment.
yes,it's used to indicate this case is supported by ch backend
There was a problem hiding this comment.
yes,it's used to indicate this case is supported by ch backend
There was a problem hiding this comment.
I just consider the method names are not general for backends, They are only used in CH test settings. How about this ?
def includeBackend(backendName: String, testNames: String*)
def excludeBackend(backendName: String, testNames: String*)
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #6067)
How was this patch tested?