HeartSaVioR commented on issue #27872: [SPARK-31115][SQL] Detect known Janino bug janino-compiler/janino#113 and apply workaround automatically via avoid using switch statement in generated code URL: https://github.com/apache/spark/pull/27872#issuecomment-597911868 It would be appreciated if we can elaborate the possible performance regression (even it's like a "thinking out loud"); "may" and "some cases" are too ambiguous. If the query doesn't fall into the case via hitting Janino issue, it shouldn't bring perf. regression, as it tries to generate code with `switch` and compiles first. If there's perf. regression here, it says I have some unexpected mistake in fix which should be fixed. If the query fall into the case via hitting Janino issue, either the query has been failing or fail back to non-codegen. Taking alternative is definitely better than failing the query; the remaining one is whether using if ~ else if chain is worse than non-codegen.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
