tisonkun opened a new pull request, #16683: URL: https://github.com/apache/pulsar/pull/16683
**I will only continue this work if the first step seems good. You may not review this patch at the moments. Although, comments are welcome.** ### Motivation See also previous discussion at https://lists.apache.org/thread/mflm0pb5235jjk80vol0vs7v0hvowkq8. Instead of firstly create a pulsar-sql repository and highly possibly leave it uncompleted, the original pain points of maintaining Pulsar SQL (a.k.a. Pulsar Presto Connector) are: 1. Pulsar (All) docker image is too large. 2.PrestoSQL version is too old and pollutes old dependencies. Thus, I'd suggest we continuously kaizen the situation by upgrading version and decouple modules. When moving out the codebase to another repository a trivial step, we consider to do it. This patch is a small step to upgrade PrestoSQL to the first Trino version, and it contains most of the renaming work. After this change, if users running sql worker with pulsar provided scripts, the experience should be as usual. But people directly depend on pulsar-presto-connector will find the dependency is switched to Trino. I'm unsure whether or not I should write a PIP of this kaizen way. I'll treat this change at the earliest in 2.12 release train and don't rush things. ### Modifications 1. Update dependencies in `pom.xml` from `io.prestosql` to `io.trino`. 2. Update `prestosql` in code to `trino`. Fortunately, the breaking changes mentioned in #16494 aren't included in this version bump. ### Verifying this change - [ ] Make sure that the change passes the CI checks. This change is already covered by existing tests. We should keep all existing test passed. ### Does this pull request potentially affect one of the following parts: *If `yes` was chosen, please highlight the changes* - Dependencies (does it add or upgrade a dependency): (yes) ### Documentation - [ ] `doc-required` (Your PR needs to update docs and you will update later) - [x] `doc-not-needed` It should be required. But leave it as is to verify CI first. -- 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]
