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

2022-09-28 Thread Guangning E
+1(non-binding) - Check sha 512 value for source code - Check go build producer example Thanks, Guangning r...@apache.org 于2022年9月29日周四 14:35写道: > Hi everyone, > > Please review and vote on the release candidate #2 for the version 0.9.0, > as follows: > [ ] +1, Approve the release > [ ] -1, Do

Re: [DISCUSS] Changed pulsar-go-client minimum Go SDK version from 1.15 to 1.16

2022-09-28 Thread r...@apache.org
Hello Matteo: Considering that some users are still using the Go 1.15 version, in the 0.9.0 release, we still use the form of hard code and set the form of pulsar-go-client to the version of 0.9.0. In order to ensure that the 0.9.0 release maintains backward compatibility, we may continue to do th

[VOTE] Pulsar Client Go Release 0.9.0 Candidate 2

2022-09-28 Thread r...@apache.org
Hi everyone, Please review and vote on the release candidate #2 for the version 0.9.0, as follows: [ ] +1, Approve the release [ ] -1, Do not approve the release (please provide specific comments) This is the second release candidate for Apache Pulsar Go client, version 0.9.0. It fixes the follo

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

2022-09-28 Thread r...@apache.org
-1 In https://github.com/apache/pulsar-client-go/pull/777 we introduce the function of AckResponse, but the modification of this function blocks the original unit test, so this piece of logic needs to be refactored. At present, the problem has been fixed, I will close the current Vote and re-lau

Re: [DISCUSS] Changed pulsar-go-client minimum Go SDK version from 1.15 to 1.16

2022-09-28 Thread Rui Fu
+1, according to https://go.dev/doc/devel/release#policy (quoted below), the current live supported go version should be 1.19, 1.18 and 1.17, so I think it should be fine if we move the pulsar-client-go with 1.16 as the minimum Go SDK version. > quote_type > > Release Policy > Each major Go rele

Re: [DISCUSS] Changed pulsar-go-client minimum Go SDK version from 1.15 to 1.16

2022-09-28 Thread Matteo Merli
+1 It would be good to determine and document a compatibility policy to state which versions are going to be supported and for how long. On Wed, Sep 28, 2022 at 8:31 PM 沈家琦 wrote: > +1 > pulsar-go-client from Go 1.15 to Go 1.16 > > Thanks > > r...@apache.org 于2022年9月29日周四 11:11写道: > > > Hello

Re: [DISCUSS] Changed pulsar-go-client minimum Go SDK version from 1.15 to 1.16

2022-09-28 Thread 沈家琦
+1 pulsar-go-client from Go 1.15 to Go 1.16 Thanks r...@apache.org 于2022年9月29日周四 11:11写道: > Hello everyone: > > In the current version of pulsar-go-client, we do not support embedding the > version information of pulsar-go-client into > Client Command to send to the Broker. Now the hard code is

Re: [VOTE] Pulsar Release 2.10.2 Candidate 1

2022-09-28 Thread guo jiwei
+1 (binding) - Checksum and signatures - Apache Rat check passes - Compile from source with JDK11 - Validate Pub/Sub and Java functions - Validate Connectors - Validate Stateful functions Regards Jiwei Guo (Tboy) On Thu, Sep 29, 2022 at 12:26 AM Nicolò Boschi wrote: > +1 (non binding) > > > -

[DISCUSS] Changed pulsar-go-client minimum Go SDK version from 1.15 to 1.16

2022-09-28 Thread r...@apache.org
Hello everyone: In the current version of pulsar-go-client, we do not support embedding the version information of pulsar-go-client into Client Command to send to the Broker. Now the hard code is fixed to 0.1.0. This makes it impossible for us to determine which version of pulsar-go-client the cur

[GitHub] [pulsar] tisonkun added a comment to the discussion: [Doc] Repurpose the Kubernetes chapter with IA improvements

2022-09-28 Thread GitBox
GitHub user tisonkun added a comment to the discussion: [Doc] Repurpose the Kubernetes chapter with IA improvements Related issue: https://github.com/apache/pulsar/issues/17774. GitHub link: https://github.com/apache/pulsar/discussions/17866#discussioncomment-3758068 This is an automati

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

2022-09-28 Thread Matteo Merli
Thanks everyone for voting on this proposal. I'm closing the vote with 4 binding +1s and 11 non-binding +1s. Binding: * Matteo * Enrico * PengHui * Jiwei Non-Binding: * Baodi * Tison * Kai * Zike * Haiting * Zixuan * Yunze * Qiang * Nicolò * Xiaoyu * Max -- Matteo Merli On Wed

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

2022-09-28 Thread guo jiwei
+1 (binding) Regards Jiwei Guo (Tboy) On Thu, Sep 29, 2022 at 8:38 AM PengHui Li wrote: > +1 (binding) > > Penghui > > On Tue, Sep 27, 2022 at 5:17 PM Enrico Olivelli > wrote: > > > +1 (binding) > > > > Enrico > > > > Il Mar 27 Set 2022, 11:13 Max Xu ha scritto: > > > > > +1 (non-binding) >

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

2022-09-28 Thread PengHui Li
+1 (binding) Penghui On Tue, Sep 27, 2022 at 5:17 PM Enrico Olivelli wrote: > +1 (binding) > > Enrico > > Il Mar 27 Set 2022, 11:13 Max Xu ha scritto: > > > +1 (non-binding) > > > > Best, > > Max Xu > > > > > > On Tue, Sep 27, 2022 at 4:50 PM Anon Hxy wrote: > > > > > +1 (non binding) > > > >

[GitHub] [pulsar-dotpulsar] aintJoshinya opened a new pull request, #110: modified the "hello world" to fix first run issue

2022-09-28 Thread GitBox
aintJoshinya opened a new pull request, #110: URL: https://github.com/apache/pulsar-dotpulsar/pull/110 the current "hello world" will not show a consumed message the very first time it is run. On subsequent runs, it will work, but it isn't entirely clear from the code alone why it works on

[GitHub] [pulsar-manager] rdhabalia merged pull request #490: Redirect notifications to the commits@ list instead of dev

2022-09-28 Thread GitBox
rdhabalia merged PR #490: URL: https://github.com/apache/pulsar-manager/pull/490 -- 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: dev-unsubscr...@pulsar.

Re: [VOTE] Pulsar Release 2.10.2 Candidate 1

2022-09-28 Thread Nicolò Boschi
+1 (non binding) - Checksum and signatures - Apache Rat check passes - Compile from source w JDK11 - Build docker image from source - Run Pulsar standalone and produce-consume from CLI - Tested a couple of connectors (ElasticSink, JDBC sink) in K8S env Nicolò Boschi Il giorno lun 26 set 2

[GitHub] [pulsar-manager] merlimat opened a new pull request, #490: Redirect notifications to the commits@ list instead of dev

2022-09-28 Thread GitBox
merlimat opened a new pull request, #490: URL: https://github.com/apache/pulsar-manager/pull/490 ### Motivation Redirect notifications to the commits@ list instead of dev -- This is an automated message from the Apache Git Service. To respond to the message, please log o

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

2022-09-28 Thread Dave Fisher
+1 (binding) Sent from my iPhone > On Sep 26, 2022, at 2:38 AM, Enrico Olivelli wrote: > > +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 Pulsa

Re: [DISCUSS] Apache Pulsar 2.11.0 Release

2022-09-28 Thread Nicolò Boschi
I've found another release blocker about client compatibility with jdk8/11, please review https://github.com/apache/pulsar/pull/17873 Nicolò Boschi Il giorno ven 16 set 2022 alle ore 10:54 Nicolò Boschi ha scritto: > Actually Yong found the root cause of the issue, see > https://lists.apache.o

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

2022-09-28 Thread Qiang Huang
+1 Zixuan Liu 于2022年9月26日周一 23:42写道: > +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

[GitHub] [pulsar-manager] sumingsoftware opened a new issue, #489: apache-pulsar-manager-0.3.0 Management menu always return 404

2022-09-28 Thread GitBox
sumingsoftware opened a new issue, #489: URL: https://github.com/apache/pulsar-manager/issues/489 **Describtion:** When I click the Management menu on the top Management menu , It always return 404 page. **environment:** 1. apache-pulsar-manager version is 0.3.0,

[GitHub] [pulsar-dotpulsar] kandersen82 opened a new pull request, #109: Interface for enqueueing

2022-09-28 Thread GitBox
kandersen82 opened a new pull request, #109: URL: https://github.com/apache/pulsar-dotpulsar/pull/109 WIP Added User API method, and rearranged interface dependencies for more flexibility in SubProducer TODO: Implement queue in SubProducer Add configuration option for que

[GitHub] [pulsar-test-infra] mangoGoForward commented on pull request #75: feat: add action of check-pr-title to ensure PR title matches Naming Convention Guide

2022-09-28 Thread GitBox
mangoGoForward commented on PR #75: URL: https://github.com/apache/pulsar-test-infra/pull/75#issuecomment-1260653955 > Got~ -- 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 co