lyne7-sc commented on PR #2379: URL: https://github.com/apache/auron/pull/2379#issuecomment-4926500830
@SteNicholas Thanks for the review. I updated this by keeping the distinction between Spark execution failures and reflection compatibility failures. `prepare()` is still allowed to fail normally. For the optional private Spark runtime-filter methods (`waitForSubqueries` and `filteredPartitions`), reflection failures such as missing or inaccessible methods now return `None`, so Auron can fall back to the normal Spark path when the method is unavailable for a Spark version. If either method is actually invoked and its implementation throws, the reflective `InvocationTargetException` is unwrapped and propagated. This keeps Spark-side runtime-filter and partition-planning failures visible instead of silently swallowing them. -- 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]
