You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "huaxingao (via GitHub)" <gi...@apache.org> on 2024/07/06 00:23:47 UTC

Re: [PR] [SPARK-46351][SQL] Require an error class in `AnalysisException` [spark]

huaxingao commented on PR #44277:
URL: https://github.com/apache/spark/pull/44277#issuecomment-2211534681

   @MaxGekk 
   I am working on Spark 4.0/Iceberg integration. I just found out that it is no longer allowed to create an `AnalysisException` with a String error message. In Iceberg extension, there are many instances where an `AnalysisException` is created with a String error message. For example: 
   ```
   throw new AnalysisException("Recursive cycle in view detected.")
   ```
    What error class should be used for this type of error?
   


-- 
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-unsubscribe@spark.apache.org

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


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