+1 By the way, why the restful consumer still not be completed?
Thanks, Jiaqi Shen Haiting Jiang <jianghait...@gmail.com> 于2022年12月7日周三 15:56写道: > > I think we can remove it first because it is not a complete feature. > Anyone interested in working on it can find the code from the git history > and migrate it to another repo. > > Looks like it's not a complete implementation of PIP-64, but the REST > producer part seems to be working. > And some users may be using this partial feature, so we need to > consider the compatibility issue. > > It makes sense to move this out of the main repo. But I don't see a > strong reason to remove it directly. > We should try to provide transparent upgrading, but letting users > "find the code from the git history and migrate it to another repo" > doesn't seem like a good option here. This bar is too high for most > users. > > Thanks, > Haiting > > On Wed, Dec 7, 2022 at 1:42 PM <mattisonc...@gmail.com> wrote: > > > > Hi Haiting > > > Is it better to finish it first, and then remove this from the main > repo? > > I think we can remove it first because it is not a complete feature. > Anyone interested in working on it can find the code from the git history > and migrate it to another repo. > > > > Ps: it looks like the current implementation has some limits, maybe > another contributor like to use a different way. > > > > Please let me know if you have any concerns or if I am missing > something. Thanks a lot! > > > > Best > > Mattison > > On Dec 7, 2022, 12:10 +0800, Haiting Jiang <jianghait...@gmail.com>, > wrote: > > > Hi Mattison > > > > > > What's the status of "moving this feature to another project" > > > Is it better to finish it first, and then remove this from the main > repo? > > > > > > Thanks, > > > Haiting > > > > > > On Tue, Dec 6, 2022 at 6:37 PM <mattisonc...@gmail.com> wrote: > > > > > > > > > > > > Hello, everyone. > > > > > > > > I'd like to start the discussion about `Remove restful producer > component`. The Github repository path is here[1]. > > > > > > > > As discussed before[2], moving this feature to another project is > better. Also, we didn't provide the consumer part in the pulsar repo. I > think it's a good chance to remove it after 2.11 is released. > > > > > > > > Best, > > > > Mattison > > > > > > > > [1] > https://github.com/apache/pulsar/tree/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/rest > > > > [2] https://lists.apache.org/thread/fl2rbb6sxlzwgkt7ybx4jxfkfnlb27z1 > > > > > > > > >