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