zabetak opened a new pull request, #6615:
URL: https://github.com/apache/hive/pull/6615

   ### What changes were proposed in this pull request?
   - Delete the druid-handler module (storage handler, SerDe, I/O formats, 
Kafka ingestion support, Kerberos security utilities)
   - Delete the itests/qtest-druid module (MiniDruidCluster, test drivers)
   - Remove all Druid-related query test files (.q and .q.out)
   - Remove hive.druid.* configuration variables from HiveConf
   - Remove Druid constants and calcite-druid dependency
   - Remove Calcite optimizer rules: HiveDruidRules, DruidSqlOperatorConverter, 
SortedDynPartitionTimeGranularityOptimizer
   - Remove DruidQuery handling from CalcitePlanner, HiveVolcanoPlanner, 
HiveFilterProjectTransposeRule, HiveRelFieldTrimmer, ASTBuilder, ASTConverter, 
and HiveMaterializedViewsRegistry
   - Remove DRUID/DRUID_LOCAL/DRUID_KAFKA cluster types from test infra
   - Relocate Kafka test utilities (SingleNodeKafkaCluster, Wikipedia) from 
itests/qtest-druid to itests/util under org.apache.hadoop.hive.kafka
   - Update itests/util dependencies to use kafka artifacts directly instead of 
pulling them transitively via hive-it-druid
   
   ### Why are the changes needed?
   The Druid integration is broken for the past 5 years and seemingly unused. 
   
   1. Remove CVE due to the use of old Druid dependencies
   2. Improve compilation performance by removing Druid specific code & rules
   3. Facilitate code maintenance by dropping unused code
   
   ### Does this PR introduce _any_ user-facing change?
   No really, in the sense that Druid integration is already broken in the past 
few releases.
   
   ### How was this patch tested?
   Existing tests


-- 
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]

Reply via email to