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

   ### Description
   
   Druid MSQ engine offers a very convenient method of performing SQL-based 
batch ingestion, export, etc.
   
   Given the importance and heavy adoption of MSQ, this patch converts MSQ into 
a core capability of Druid
   rather than an extension, as was always intended.
   
   ### Changes
   
   - Move `multi-stage-query` module out from `extensions-core`
   - Add MSQ modules to the `Cli*` classes of applicable services
   - Update dependencies as needed
   - Remove mentions of extension from example `common.runtime.properties` files
   - Update embedded tests to not load MSQ extension modules
   
   ### Pending
   
   - Once initial feedback is received, the docs will also be updated to remove 
mentions of the multi-stage-query extension.
   
   ### Release note
   
   - Multi-stage-query (MSQ engine) is now a core capability of Druid rather 
than an extension.
   - Remove `druid-multi-stage-query` from `druid.extensions.loadList` in 
`common.runtime.properties`.
   
   ### Upgrade notes
   
   While upgrading, the extension `druid-multi-stage-query` must be removed 
from `druid.extensions.loadList`.
   Otherwise, Druid services would fail to start as they would not be able to 
locate this extension.
   
   
   <hr>
   
   This PR has:
   
   - [ ] been self-reviewed.
      - [ ] using the [concurrency 
checklist](https://github.com/apache/druid/blob/master/dev/code-review/concurrency.md)
 (Remove this item if the PR doesn't have any relation to concurrency.)
   - [ ] added documentation for new or modified features or behaviors.
   - [ ] a release note entry in the PR description.
   - [ ] added Javadocs for most classes and all non-trivial methods. Linked 
related entities via Javadoc links.
   - [ ] added or updated version, license, or notice information in 
[licenses.yaml](https://github.com/apache/druid/blob/master/dev/license.md)
   - [ ] added comments explaining the "why" and the intent of the code 
wherever would not be obvious for an unfamiliar reader.
   - [ ] added unit tests or modified existing tests to cover new code paths, 
ensuring the threshold for [code 
coverage](https://github.com/apache/druid/blob/master/dev/code-review/code-coverage.md)
 is met.
   - [ ] added integration tests.
   - [ ] 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