[GitHub] [spark] itholic commented on pull request #39501: [SPARK-41295][SQL] Rename the error classes

2023-01-22 Thread via GitHub
itholic commented on PR #39501: URL: https://github.com/apache/spark/pull/39501#issuecomment-1399591723 @NarekDW Can you add `[SPARK-41296]` to PR title instead explaining in PR description ? `[SPARK-41295][SPARK-41296][SQL] Rename the error classes` -- This is an automated message

[GitHub] [spark] itholic commented on pull request #39501: [SPARK-41295][SQL] Rename the error classes

2023-01-22 Thread via GitHub
itholic commented on PR #39501: URL: https://github.com/apache/spark/pull/39501#issuecomment-1399590022 It looks good to me. Also cc @cloud-fan -- 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 g

[GitHub] [spark] itholic commented on pull request #39501: [SPARK-41295][SQL] Rename the error classes

2023-01-11 Thread GitBox
itholic commented on PR #39501: URL: https://github.com/apache/spark/pull/39501#issuecomment-1378569552 cc @srielau @MaxGekk Could you review this error classes when you find some time ?? -- This is an automated message from the Apache Git Service. To respond to the message, please log on