Re: [PR] [SPARK-46991][SQL] Replace `IllegalArgumentException` by `SparkIllegalArgumentException` in `catalyst` [spark]

2024-03-03 Thread via GitHub
panbingkun commented on PR #45033: URL: https://github.com/apache/spark/pull/45033#issuecomment-1975489055 +1, LGTM. Thank you. -- 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 commen

Re: [PR] [SPARK-46991][SQL] Replace `IllegalArgumentException` by `SparkIllegalArgumentException` in `catalyst` [spark]

2024-02-11 Thread via GitHub
MaxGekk commented on PR #45033: URL: https://github.com/apache/spark/pull/45033#issuecomment-1937630476 @panbingkun @srielau @dongjoon-hyun @LuciferYang @beliefer @cloud-fan Could you review this PR, please. -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] [SPARK-46991][SQL] Replace `IllegalArgumentException` by `SparkIllegalArgumentException` in `catalyst` [spark]

2024-02-11 Thread via GitHub
dongjoon-hyun closed pull request #45033: [SPARK-46991][SQL] Replace `IllegalArgumentException` by `SparkIllegalArgumentException` in `catalyst` URL: https://github.com/apache/spark/pull/45033 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] [SPARK-46991][SQL] Replace `IllegalArgumentException` by `SparkIllegalArgumentException` in `catalyst` [spark]

2024-02-11 Thread via GitHub
dongjoon-hyun commented on PR #45033: URL: https://github.com/apache/spark/pull/45033#issuecomment-1937910390 Merged to master for Apache Spark 4.0.0. -- 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