LuciferYang commented on PR #57218: URL: https://github.com/apache/spark/pull/57218#issuecomment-4959894297
Good point, done. I've switched it to `SparkException.internalError` and removed the condition from `error-conditions.json`. One thing worth flagging: `internalError` has no overload that takes the parser context, so the `ParseException` position info (offending fragment, line/column) is dropped. Since this branch is unreachable from normal parsing — the grammar allows only one of `STORED AS` / `STORED BY`, and a duplicated clause is already caught earlier as `DUPLICATE_CLAUSES` — losing the context has no user-facing impact here. Let me know if you'd prefer to keep the context anyway. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
