clintropolis opened a new pull request, #13698:
URL: https://github.com/apache/druid/pull/13698

   ### Description
   This PR picks up where #6443 left off and merges `druid-core`, 
`extendedset`, and `druid-hll` into `druid-processing`. 
   
   My hope is that this is minimally disruptive to all open PRs, since its 
largely a straight move, the only expected conflicts coming from files which 
were newly added to the removed modules (which should be a very simple fix to 
move to the corresponding `processing/src/` path of the same package) as well 
as poms which have added dependency references.
   
   I chose `druid-processing` instead of `druid-core` because it was smaller, 
but I guess I don't feel super strongly either way about the final name, but 
have a slight affinity for this since it does contain basically all you need to 
make and query segments. Regardless of destination, all of these packages 
needed combined for this to work, since extendedset and hll depended on core, 
and processing depended on all 3.
   
   I tested that using pre-built extensions from older versions with these 
libraries split up continue to work when used with newer builds with 
consolidated `druid-processing`.
   
   #### Release note
   `druid-core`, `extendedset`, and `druid-hll` modules have been consolidated 
into `druid-processing` to simplify dependencies. Any extensions referencing 
these should be updated to use `druid-processing` instead. Existing extension 
binaries _should_ continue to function normally when used with newer versions 
of Druid.
   
   <hr>
   
   
   This PR has:
   
   - [x] been self-reviewed.
   - [x] a release note entry in the PR description.
   - [x] been tested in a test Druid cluster.
   


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