Re: [VOTE] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Yunze Xu
+1 (non-binding) Thanks, Yunze > On Sep 27, 2022, at 11:11, Zixuan Liu wrote: > > +1(non-binding) > > Thanks, > Zixuan > > Haiting Jiang 于2022年9月27日周二 09:56写道: > >> +1 (non) >> >> Haiting >> >> On Tue, Sep 27, 2022 at 9:30 AM Zike Yang wrote: >>> >>> +1 (non-binding) >>> >>> >>> Zi

Re: [VOTE] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Zixuan Liu
+1(non-binding) Thanks, Zixuan Haiting Jiang 于2022年9月27日周二 09:56写道: > +1 (non) > > Haiting > > On Tue, Sep 27, 2022 at 9:30 AM Zike Yang wrote: > > > > +1 (non-binding) > > > > > > Zike Yang > > > > On Tue, Sep 27, 2022 at 9:24 AM Kai Wang > wrote: > > > > > > +1 (non-binding) > > > > > > Tha

Re: [VOTE] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Haiting Jiang
+1 (non) Haiting On Tue, Sep 27, 2022 at 9:30 AM Zike Yang wrote: > > +1 (non-binding) > > > Zike Yang > > On Tue, Sep 27, 2022 at 9:24 AM Kai Wang wrote: > > > > +1 (non-binding) > > > > Thanks, > > Kai

Re: [VOTE] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Zike Yang
+1 (non-binding) Zike Yang On Tue, Sep 27, 2022 at 9:24 AM Kai Wang wrote: > > +1 (non-binding) > > Thanks, > Kai

Re: [VOTE] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Kai Wang
+1 (non-binding) Thanks, Kai

[GitHub] [pulsar] tisonkun edited a comment on the discussion: Archive Crowdin based translation initiative

2022-09-26 Thread GitBox
GitHub user tisonkun edited a comment on the discussion: Archive Crowdin based translation initiative @tuhaihe @Anonymitaet @urfreespace Thanks for your feedback. To prevent spreading discussion to the next solution of translation, I update the topic as "Archive Crowdin-based translation init

[GitHub] [pulsar] tisonkun edited a comment on the discussion: Archive Crowdin based translation initiative

2022-09-26 Thread GitBox
GitHub user tisonkun edited a comment on the discussion: Archive Crowdin based translation initiative @tuhaihe @Anonymitaet @urfreespace Thanks for your feedback. To prevent spreading discussion to the next solution of translation, I update the topic as "Archive Crowdin-based translation init

[GitHub] [pulsar] tisonkun added a comment to the discussion: Archive Crowdin based translation initiative

2022-09-26 Thread GitBox
GitHub user tisonkun added a comment to the discussion: Archive Crowdin based translation initiative @tuhaihe @Anonymitaet @urfreespace Thanks for your feedback. To prevent spreading discussion to the next solution of translation, I update the topic as "Archive Crowdin-based translation initi

[GitHub] [pulsar] tisonkun edited a discussion: Archive Crowdin based translation initiative

2022-09-26 Thread GitBox
GitHub user tisonkun edited a discussion: Archive Crowdin based translation initiative ## Motivation Three years ago we created the [pulsar-translation](https://github.com/apache/pulsar-translation) repository to try to handle documentation translation with Crowdin. However, after three yea

Re: [VOTE] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread tison
+1 (non-binding) Best, tison. Baodi Shi 于2022年9月27日周二 08:22写道: > +1(non-binding) > > Baodi Shi > Thanks > > 2022年9月27日上午2:56,Matteo Merli 写道: > > > https://github.com/apache/pulsar/issues/17724 > > > > ## Motivation > > Pulsar C++ code base is in the same main repository for the Pulsar projec

[GitHub] [pulsar-test-infra] tisonkun commented on pull request #71: Improve comment messages

2022-09-26 Thread GitBox
tisonkun commented on PR #71: URL: https://github.com/apache/pulsar-test-infra/pull/71#issuecomment-1258845388 @Anonymitaet @nodece can we merge this patch now? I don't see further blocker here. -- This is an automated message from the Apache Git Service. To respond to the message, please

回复:[VOTE] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Baodi Shi
+1(non-binding)Baodi ShiThanks2022年9月27日上午2:56,Matteo Merli 写道:https://github.com/apache/pulsar/issues/17724## MotivationPulsar C++ code base is in the same main repository for the Pulsar project.While the decision was the right one at the time, there is aconsiderable overheadin keeping the C+

[VOTE] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Matteo Merli
https://github.com/apache/pulsar/issues/17724 ## Motivation Pulsar C++ code base is in the same main repository for the Pulsar project. While the decision was the right one at the time, there is a considerable overhead in keeping the C++ client in its current position. ### Issues with the cur

[GitHub] [pulsar-helm-chart] arnarg opened a new pull request, #291: Alphabetically sort list of super users

2022-09-26 Thread GitBox
arnarg opened a new pull request, #291: URL: https://github.com/apache/pulsar-helm-chart/pull/291 Fixes #288 ### Motivation When specifying multiple roles in `.Values.auth.superUsers` the values are converted to a comma-separated list by piping the dict through `values` and `

Re: [DISCUSS] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Zixuan Liu
+1 Thanks, Zixuan Anon Hxy 于2022年9月26日周一 17:52写道: > +1 LGTM > > Thanks, > Xiaoyu Hou > > Enrico Olivelli 于2022年9月26日周一 17:39写道: > > > +100 to this ! > > > > Thanks for this proposal > > > > Enrico > > > > Il giorno lun 26 set 2022 alle ore 10:04 Zike Yang ha > > scritto: > > > > > > +1. Looks

[DISCUSS] Use AUTO_CONSUME schema with Avro logical type conversion problem

2022-09-26 Thread 丛搏
Hello, Pulsar community: Now, we use Avro Schema with Logical type, and the client automatically adds logical type conversion. You can see PR https://github.com/apache/pulsar/pull/6704. But AUTO_CONSUME Schema with Avro doesn't add the logical type conversion. It makes the Time type decode to L

[GitHub] [pulsar-dotpulsar] blankensteiner closed issue #106: Why does "pulse changed state to'faulted" appear when I connect to the server version 2.10.0

2022-09-26 Thread GitBox
blankensteiner closed issue #106: Why does "pulse changed state to'faulted" appear when I connect to the server version 2.10.0 URL: https://github.com/apache/pulsar-dotpulsar/issues/106 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[VOTE] Pulsar Release 2.10.2 Candidate 1

2022-09-26 Thread Haiting Jiang
This is the first release candidate for Apache Pulsar, version 2.10.2. This release contains 244 commits by 57 contributors. https://github.com/apache/pulsar/compare/v2.10.1...v2.10.2-candidate-1 Full CI on this release: https://github.com/Jason918/pulsar/pull/8 *** Please download, test and vot

Re: [DISCUSS] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Anon Hxy
+1 LGTM Thanks, Xiaoyu Hou Enrico Olivelli 于2022年9月26日周一 17:39写道: > +100 to this ! > > Thanks for this proposal > > Enrico > > Il giorno lun 26 set 2022 alle ore 10:04 Zike Yang ha > scritto: > > > > +1. Looks good to me. > > > > Do we need to move the `Apache Pulsar / Multi Clients` Github pr

Re: [DISCUSS] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Enrico Olivelli
+100 to this ! Thanks for this proposal Enrico Il giorno lun 26 set 2022 alle ore 10:04 Zike Yang ha scritto: > > +1. Looks good to me. > > Do we need to move the `Apache Pulsar / Multi Clients` Github project > to the corresponding repos? > > Thanks, > Zike Yang > > Zike Yang > > > On Fri, Sep

[GitHub] [pulsar] urfreespace added a comment to the discussion: Lightweight Documentation Translation Solution

2022-09-26 Thread GitBox
GitHub user urfreespace added a comment to the discussion: Lightweight Documentation Translation Solution Crowdin brings a lot of problems, sometimes, it breaks some tag structures in MDX, and these are even unpredictable, problems occur frequently. From a technical point of view, I think it

Re: [VOTE] Pulsar Client Go Release 0.9.0 Candidate 1

2022-09-26 Thread Guangning E
+1(non-binding) - Check sha 512 value for source code - Check go build Thanks, Guangning Rui Fu 于2022年7月7日周四 10:58写道: > Hi everyone, > > Please review and vote on the release candidate #1 for the version 0.9.0, > as follows: > [ ] +1, Approve the release > [ ] -1, Do not approve the release (pl

Re: [DISCUSS] PIP-209: Separate C++/Python clients to own repositories

2022-09-26 Thread Zike Yang
+1. Looks good to me. Do we need to move the `Apache Pulsar / Multi Clients` Github project to the corresponding repos? Thanks, Zike Yang Zike Yang On Fri, Sep 23, 2022 at 7:22 AM Matteo Merli wrote: > > -- > Matteo Merli > > > On Tue, Sep 20, 2022 at 8:14 PM Michael Marshall wrote: > > > >

[GitHub] [pulsar] tuhaihe edited a comment on the discussion: Lightweight Documentation Translation Solution

2022-09-26 Thread GitBox
GitHub user tuhaihe edited a comment on the discussion: Lightweight Documentation Translation Solution > If Crowdin does not work well with Pulsar, it makes sense to choose another > open-source CAT tool. Yes, agree with this. For the CAT tools, we can have some great candidate tools, such a

[GitHub] [pulsar] tuhaihe added a comment to the discussion: Lightweight Documentation Translation Solution

2022-09-26 Thread GitBox
GitHub user tuhaihe added a comment to the discussion: Lightweight Documentation Translation Solution > If Crowdin does not work well with Pulsar, it makes sense to choose another > open-source CAT tool. Yes, agree with this. For the CAT tools, we can have some great candidate tools, such as: