[GitHub] [spark] MaxGekk commented on pull request #37744: [SPARK-40300][SQL] Migrate onto the `DATATYPE_MISMATCH` error class

2022-09-06 Thread GitBox


MaxGekk commented on PR #37744:
URL: https://github.com/apache/spark/pull/37744#issuecomment-1237855327

   Merging to master. Thank you, @srielau @cloud-fan for review.


-- 
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 specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] MaxGekk commented on pull request #37744: [SPARK-40300][SQL] Migrate onto the `DATATYPE_MISMATCH` error class

2022-09-01 Thread GitBox


MaxGekk commented on PR #37744:
URL: https://github.com/apache/spark/pull/37744#issuecomment-1234048716

   The test failure is not related to this PR, I believe:
   ```
   YarnClusterSuite.run Spark in yarn-client mode with different 
configurations, ensuring redaction
   ```
   @cloud-fan @HyukjinKwon @gengliangwang Could you review the PR, please. It 
migrates a bunch of AnalysisException in golden files to error classes. 


-- 
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 specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] [spark] MaxGekk commented on pull request #37744: [SPARK-40300][SQL] Migrate onto the `DATATYPE_MISMATCH` error class

2022-09-01 Thread GitBox


MaxGekk commented on PR #37744:
URL: https://github.com/apache/spark/pull/37744#issuecomment-1233890597

   also cc @srielau @anchovYu Could you take a look at the PR which introduces 
new error classes.


-- 
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 specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org