s-maling-celonis commented on PR #58383: URL: https://github.com/apache/spark/pull/58383#issuecomment-5523110127
Friendly ping — this has been ready for review for a week with no comments. SPARK-59081: `HiveSessionStateBuilder` builds its own `Analyzer` and never set `hintResolutionRules`, so `injectHintResolutionRule` is a no-op under `enableHiveSupport()` (Hive Thrift / `jdbc:hive2`). `BaseSessionStateBuilder` already wires `hintResolutionRules = customHintResolutionRules`. Same class of miss as SPARK-56453. Small patch + `HiveSparkSessionExtensionSuite`. Local `hive/testOnly` of that suite passed. Would someone who owns this area take a look? cc @MaxGekk @cloud-fan @dtenedor -- 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]
