flyrain commented on issue #2382: URL: https://github.com/apache/iceberg/issues/2382#issuecomment-814268563
Yeah, by looking at more code, reverting it is more practical. I like the fail-fast approach, but checking the antlr-runtime version is not feasible since we want a unified iceberg runtime version for all Spark 3.x, and Spark3.1.1 has changed antlr4 as @pan3793 mentioned. Yes, integration test or something similar is necessary. This issue should be considered as a blocker since every customized iceberg sql will fail, but obviously it didn't get much attention. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
