[GitHub] [spark] MaxGekk closed pull request #40433: [SPARK-42706][SQL][DOCS][3.4] Document the Spark SQL error classes in user-facing documentation

2023-03-15 Thread via GitHub
MaxGekk closed pull request #40433: [SPARK-42706][SQL][DOCS][3.4] Document the Spark SQL error classes in user-facing documentation URL: https://github.com/apache/spark/pull/40433 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [spark] MaxGekk commented on pull request #40433: [SPARK-42706][SQL][DOCS][3.4] Document the Spark SQL error classes in user-facing documentation

2023-03-15 Thread via GitHub
MaxGekk commented on PR #40433: URL: https://github.com/apache/spark/pull/40433#issuecomment-1469439953 +1, LGTM. Merging to 3.4. Thank you, @itholic and @cloud-fan @dongjoon-hyun @HyukjinKwon for review. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] mskapilks commented on pull request #40266: [SPARK-42660][SQL] Infer filters for Join produced by IN and EXISTS clause (RewritePredicateSubquery rule)

2023-03-15 Thread via GitHub
mskapilks commented on PR #40266: URL: https://github.com/apache/spark/pull/40266#issuecomment-1469424781 @wangyum @peter-toth Thanks for pointing on previous attempts. It does seem moving `RewritePredicateSubquery` rule is right way so that in future we don't add anymore rule to

[GitHub] [spark] dongjoon-hyun closed pull request #40434: [SPARK-42801][CONNECT][TESTS] Ignore flaky `write jdbc` test of `ClientE2ETestSuite` on Java 8

2023-03-15 Thread via GitHub
dongjoon-hyun closed pull request #40434: [SPARK-42801][CONNECT][TESTS] Ignore flaky `write jdbc` test of `ClientE2ETestSuite` on Java 8 URL: https://github.com/apache/spark/pull/40434 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [spark] dongjoon-hyun commented on pull request #40434: [SPARK-42801][CONNECT][TESTS] Ignore flaky `write jdbc` test of `ClientE2ETestSuite` on Java 8

2023-03-15 Thread via GitHub
dongjoon-hyun commented on PR #40434: URL: https://github.com/apache/spark/pull/40434#issuecomment-1469417616 Thank you, @HyukjinKwon and @LuciferYang . The `connect` module UT passed. Let me merge this because this is a removal of test coverage. -- This is an automated message from the

[GitHub] [spark] Stove-hust commented on pull request #40393: [SPARK-40082] Schedule mergeFinalize when push merge shuffleMapStage retry but no running tasks

2023-03-15 Thread via GitHub
Stove-hust commented on PR #40393: URL: https://github.com/apache/spark/pull/40393#issuecomment-1469402096 > @Stove-hust Haven't had a chance to look at it yet. I'll take a look at it this week. tks -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] LuciferYang commented on pull request #40389: [SPARK-42767][CONNECT][TESTS] Add a precondition to start connect server fallback with `in-memory` and auto ignored some tests strongly d

2023-03-15 Thread via GitHub
LuciferYang commented on PR #40389: URL: https://github.com/apache/spark/pull/40389#issuecomment-1469396942 Should we push forward this one? @HyukjinKwon -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [spark] dongjoon-hyun commented on pull request #40434: [SPARK-42801][CONNECT][TESTS] Ignore flaky `write jdbc` test of `ClientE2ETestSuite` on Java 8

2023-03-15 Thread via GitHub
dongjoon-hyun commented on PR #40434: URL: https://github.com/apache/spark/pull/40434#issuecomment-1469394962 cc @HyukjinKwon , @hvanhovell -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

<    1   2   3