Thanks for your feedback! 1. According to the upgrade and downgrade doc, I think the minimum requirements are a release note. Describe the layout change and how users should move the folder. I'll elaborate on the issue and notify you here. 2. It seems Pulsar Helm Chart support 2.9.3 now. I'll investigate how it can be relevant in days. If someone who maintains the Chart can provide some input, it will help! 3. "There are 3 issues". It's a description about the background/motivation, while we handle the first issue in this PIP. Although, subtasks can be divided into packaging changes, possible doc changes, and possible Chart changes.
Best, tison. Qiang Huang <qiang.huang1...@gmail.com> 于2022年8月17日周三 19:25写道： > Looks good. I have two points: > 1. It is necessary to supplement the upgrade and downgrade documentation in > Pulsar. > 2. There are 3 issues mentioned in the PIP, should we split it into 3 small > issues? > > Enrico Olivelli <eolive...@gmail.com> 于2022年8月17日周三 17:30写道： > > > I generally agree with the PIP > > > > Can you please explain the interactions with the Pulsar Helm chart ? > > also we have to draw a migration path, because users that will upgrade > > Pulsar will have to move the configuration files in another location > > > > Enrico > > > > Il giorno mer 17 ago 2022 alle ore 11:15 tison <wander4...@gmail.com> > > ha scritto: > > > > > > Hello, > > > > > > This is a PIP to package the Pulsar Trino distro and config in a > > dedicated > > > folder. > > > > > > Link: https://github.com/apache/pulsar/issues/17137 > > > Prototype: https://github.com/apache/pulsar/pull/17062 > > > > > > Below you can find the proposal (I will amend the GH issue while we > > discuss > > > it). > > > > > > Best, > > > tison. > > > > > > Motivation > > > ======== > > > > > > After https://github.com/apache/pulsar/pull/16683 merged, we upgrade > > > PrestoSQL dependency in Pulsar SQL to the first several Trino version. > To > > > handle the name change cases and gradually refactor Pulsar SQL as a > > > self-contained module so that we can move it into a standalone > > repository, > > > I find that there're three major issues to resolve. > > > > > > 1. Configs of Pulsar SQL go under the `conf/` folder and mix with other > > > Pulsar configs. > > > 2. Pulsar Docker images (base and all) bundle Pulsar SQL. > > > 3. Integration tests of Pulsar SQL are tightly coupled with the main > repo > > > (test infra). > > > > > > This proposal is aimed at resolving the first issue to package Pulsar > > Trino > > > distro and config in a dedicated folder; that is, to make it > > self-contained. > > > > > > Goal > > > ==== > > > > > > I have already prepared a draft to perform the changes as > > > https://github.com/apache/pulsar/pull/17062. Generally, we move the > > config > > > files under `PRESTO_HOME` and correspondingly update scripts. > > > > > > In this way, all Trino distro artifacts are under the same home path, > so > > > that we can later move it out as a whole. > > > > > > This change should not affect those who use Pulsar with the entry point > > > script, but it changes the layout of the release artifact, so I'd > prefer > > to > > > perform a PIP process. > > > > > > Implementation > > > ============ > > > > > > It's straightforward to inline in the "Goal" section. > > > > > > However, the name of the folder (`presto` or `trino`) and the level of > > the > > > folder (`lib/presto/` or `trino/`) is open to discussion. I think both > > are > > > fine and will try `trino/` first. > > > > > > To minimize unnecessary changes, I tend to keep the modules name > > > `pulsar-presto-xxx` as is. > > > > > > Alternatives > > > ========= > > > > > > I don't make a completed proposal to resolve all three issues listed > > above. > > > Because I'm still unfamiliar with the latter two topics yet and I'd > > prefer > > > to implement these improvements one by one since they're naturally > > > independent. If I try to make a completed proposal at once, it's highly > > > possible I give up halfway. > > > > > > Anything else? > > > =========== > > > > > > Previous discussion: > > > > > > [DISCUSS] Move Pulsar SQL to a separated repository? > > > https://lists.apache.org/thread/mflm0pb5235jjk80vol0vs7v0hvowkq8 > > > > > -- > BR, > Qiang Huang >
