Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-12 Thread Yunze Xu
Closing this vote by 3 binding +1s:

- Penghui
- Yunze
- Mattison

Thanks,
Yunze


On Mon, Nov 13, 2023 at 11:14 AM Yunze Xu  wrote:
>
> In addition to Mattison's failure. The
> `LookupServiceTest.testMultiAddresses` failed because the HTTP proxy
> is enabled. See https://github.com/apache/pulsar-client-cpp/issues/343
>
> In addition, I edited the wiki to note proxy should be disabled
> because many failed tests could be caused by the proxy:
> https://github.com/apache/pulsar-client-cpp/wiki/Committer-Release-Guide#run-the-vote
>
> Thanks,
> Yunze
>
> On Mon, Nov 13, 2023 at 10:05 AM mattison chao  wrote:
> >
> > +1(binding)
> >
> > Some failing tests are unstable and state-dependent, and the test passes 
> > after retrying.
> >
> >
> > Best,
> > Mattison
> >
> > > On Nov 12, 2023, at 22:10, mattison chao  wrote:
> > >
> > > Hi, Yunze
> > >
> > > I’ve got several failed tests.
> > >
> > > [--] Global test environment tear-down
> > > [==] 528 tests from 85 test suites ran. (888193 ms total)
> > > [  PASSED  ] 525 tests.
> > > [  FAILED  ] 3 tests, listed below:
> > > [  FAILED  ] CustomLoggerTest.testCustomLogger
> > > [  FAILED  ] LookupServiceTest.testMultiAddresses
> > > [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic
> > >
> > >
> > > After rerun, the second and third tests still failed.
> > >
> > > /Users/mattisonchao/projects/apache/pulsar-client-cpp/tests/c/c_ConsumerTest.cc:182:
> > >  Failure
> > > Expected equality of these values:
> > >   pulsar_result_Timeout
> > > Which is: 3
> > >   res
> > > Which is: 0
> > >
> > > [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic (75 ms)
> > > [--] 1 test from c_ConsumerTest (75 ms total)
> > >
> > > [--] Global test environment tear-down
> > > [==] 1 test from 1 test suite ran. (75 ms total)
> > > [  PASSED  ] 0 tests.
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic
> > >
> > >
> > > Expected equality of these values:
> > >   std::count(results.cbegin(), results.cend(), ResultRetryable)
> > > Which is: 0
> > >   numRequests / 2
> > > Which is: 3
> > >
> > > 2023-11-12 22:10:03.538 INFO  [0x1e483d300] ClientConnection:268 | 
> > > [ -> pulsar://localhost:] Destroyed connection to 
> > > pulsar://localhost:
> > > [  FAILED  ] LookupServiceTest.testMultiAddresses (89 ms)
> > > [--] 1 test from LookupServiceTest (89 ms total)
> > >
> > > [--] Global test environment tear-down
> > > [==] 1 test from 1 test suite ran. (89 ms total)
> > > [  PASSED  ] 0 tests.
> > > [  FAILED  ] 1 test, listed below:
> > > [  FAILED  ] LookupServiceTest.testMultiAddresses
> > >
> > >
> > >
> > > Best,
> > > Mattison
> > >
> > >> On Nov 7, 2023, at 15:03, Yunze Xu  wrote:
> > >>
> > >> This is the second release candidate for Apache Pulsar Client C++,
> > >> version 3.4.0.
> > >>
> > >> It fixes the following issues:
> > >> https://github.com/apache/pulsar-client-cpp/milestone/5?closed=1
> > >>
> > >> *** Please download, test and vote on this release. This vote will stay 
> > >> open
> > >> for at least 72 hours ***
> > >>
> > >> Note that we are voting upon the source (tag), binaries are provided for
> > >> convenience.
> > >>
> > >> Source and binary files:
> > >> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
> > >>
> > >> SHA-512 checksums:
> > >>
> > >> 10517590a2e4296d6767a044e58dd32c79e404a5136cf41126f4cb2416ed0ef8fb1ad7aa7da54c37c12ed71b05a527ed08bfac9d50d3550fa5d475c1e8c00950
> > >> apache-pulsar-client-cpp-3.4.0.tar.gz
> > >>
> > >>
> > >> The tag to be voted upon:
> > >> v3.4.0-candidate-2 (f337eff7caae93730ec1260810655cbb5a345e70)
> > >> https://github.com/apache/pulsar-client-cpp/releases/tag/v3.4.0-candidate-2
> > >>
> > >> Pulsar's KEYS file containing PGP keys you use to sign the release:
> > >> https://downloads.apache.org/pulsar/KEYS
> > >>
> > >> Please download the source package, and follow the README to compile and 
> > >> test.
> > >
> >


Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-12 Thread Yunze Xu
In addition to Mattison's failure. The
`LookupServiceTest.testMultiAddresses` failed because the HTTP proxy
is enabled. See https://github.com/apache/pulsar-client-cpp/issues/343

In addition, I edited the wiki to note proxy should be disabled
because many failed tests could be caused by the proxy:
https://github.com/apache/pulsar-client-cpp/wiki/Committer-Release-Guide#run-the-vote

Thanks,
Yunze

On Mon, Nov 13, 2023 at 10:05 AM mattison chao  wrote:
>
> +1(binding)
>
> Some failing tests are unstable and state-dependent, and the test passes 
> after retrying.
>
>
> Best,
> Mattison
>
> > On Nov 12, 2023, at 22:10, mattison chao  wrote:
> >
> > Hi, Yunze
> >
> > I’ve got several failed tests.
> >
> > [--] Global test environment tear-down
> > [==] 528 tests from 85 test suites ran. (888193 ms total)
> > [  PASSED  ] 525 tests.
> > [  FAILED  ] 3 tests, listed below:
> > [  FAILED  ] CustomLoggerTest.testCustomLogger
> > [  FAILED  ] LookupServiceTest.testMultiAddresses
> > [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic
> >
> >
> > After rerun, the second and third tests still failed.
> >
> > /Users/mattisonchao/projects/apache/pulsar-client-cpp/tests/c/c_ConsumerTest.cc:182:
> >  Failure
> > Expected equality of these values:
> >   pulsar_result_Timeout
> > Which is: 3
> >   res
> > Which is: 0
> >
> > [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic (75 ms)
> > [--] 1 test from c_ConsumerTest (75 ms total)
> >
> > [--] Global test environment tear-down
> > [==] 1 test from 1 test suite ran. (75 ms total)
> > [  PASSED  ] 0 tests.
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic
> >
> >
> > Expected equality of these values:
> >   std::count(results.cbegin(), results.cend(), ResultRetryable)
> > Which is: 0
> >   numRequests / 2
> > Which is: 3
> >
> > 2023-11-12 22:10:03.538 INFO  [0x1e483d300] ClientConnection:268 | [ 
> > -> pulsar://localhost:] Destroyed connection to pulsar://localhost:
> > [  FAILED  ] LookupServiceTest.testMultiAddresses (89 ms)
> > [--] 1 test from LookupServiceTest (89 ms total)
> >
> > [--] Global test environment tear-down
> > [==] 1 test from 1 test suite ran. (89 ms total)
> > [  PASSED  ] 0 tests.
> > [  FAILED  ] 1 test, listed below:
> > [  FAILED  ] LookupServiceTest.testMultiAddresses
> >
> >
> >
> > Best,
> > Mattison
> >
> >> On Nov 7, 2023, at 15:03, Yunze Xu  wrote:
> >>
> >> This is the second release candidate for Apache Pulsar Client C++,
> >> version 3.4.0.
> >>
> >> It fixes the following issues:
> >> https://github.com/apache/pulsar-client-cpp/milestone/5?closed=1
> >>
> >> *** Please download, test and vote on this release. This vote will stay 
> >> open
> >> for at least 72 hours ***
> >>
> >> Note that we are voting upon the source (tag), binaries are provided for
> >> convenience.
> >>
> >> Source and binary files:
> >> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
> >>
> >> SHA-512 checksums:
> >>
> >> 10517590a2e4296d6767a044e58dd32c79e404a5136cf41126f4cb2416ed0ef8fb1ad7aa7da54c37c12ed71b05a527ed08bfac9d50d3550fa5d475c1e8c00950
> >> apache-pulsar-client-cpp-3.4.0.tar.gz
> >>
> >>
> >> The tag to be voted upon:
> >> v3.4.0-candidate-2 (f337eff7caae93730ec1260810655cbb5a345e70)
> >> https://github.com/apache/pulsar-client-cpp/releases/tag/v3.4.0-candidate-2
> >>
> >> Pulsar's KEYS file containing PGP keys you use to sign the release:
> >> https://downloads.apache.org/pulsar/KEYS
> >>
> >> Please download the source package, and follow the README to compile and 
> >> test.
> >
>


Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-12 Thread mattison chao
+1(binding)

Some failing tests are unstable and state-dependent, and the test passes after 
retrying.


Best,
Mattison

> On Nov 12, 2023, at 22:10, mattison chao  wrote:
> 
> Hi, Yunze
> 
> I’ve got several failed tests.
> 
> [--] Global test environment tear-down
> [==] 528 tests from 85 test suites ran. (888193 ms total)
> [  PASSED  ] 525 tests.
> [  FAILED  ] 3 tests, listed below:
> [  FAILED  ] CustomLoggerTest.testCustomLogger
> [  FAILED  ] LookupServiceTest.testMultiAddresses
> [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic
> 
> 
> After rerun, the second and third tests still failed.
> 
> /Users/mattisonchao/projects/apache/pulsar-client-cpp/tests/c/c_ConsumerTest.cc:182:
>  Failure
> Expected equality of these values:
>   pulsar_result_Timeout
> Which is: 3
>   res
> Which is: 0
> 
> [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic (75 ms)
> [--] 1 test from c_ConsumerTest (75 ms total)
> 
> [--] Global test environment tear-down
> [==] 1 test from 1 test suite ran. (75 ms total)
> [  PASSED  ] 0 tests.
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] c_ConsumerTest.testCDeadLetterTopic
> 
> 
> Expected equality of these values:
>   std::count(results.cbegin(), results.cend(), ResultRetryable)
> Which is: 0
>   numRequests / 2
> Which is: 3
> 
> 2023-11-12 22:10:03.538 INFO  [0x1e483d300] ClientConnection:268 | [ -> 
> pulsar://localhost:] Destroyed connection to pulsar://localhost:
> [  FAILED  ] LookupServiceTest.testMultiAddresses (89 ms)
> [--] 1 test from LookupServiceTest (89 ms total)
> 
> [--] Global test environment tear-down
> [==] 1 test from 1 test suite ran. (89 ms total)
> [  PASSED  ] 0 tests.
> [  FAILED  ] 1 test, listed below:
> [  FAILED  ] LookupServiceTest.testMultiAddresses
> 
> 
> 
> Best,
> Mattison
> 
>> On Nov 7, 2023, at 15:03, Yunze Xu  wrote:
>> 
>> This is the second release candidate for Apache Pulsar Client C++,
>> version 3.4.0.
>> 
>> It fixes the following issues:
>> https://github.com/apache/pulsar-client-cpp/milestone/5?closed=1
>> 
>> *** Please download, test and vote on this release. This vote will stay open
>> for at least 72 hours ***
>> 
>> Note that we are voting upon the source (tag), binaries are provided for
>> convenience.
>> 
>> Source and binary files:
>> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
>> 
>> SHA-512 checksums:
>> 
>> 10517590a2e4296d6767a044e58dd32c79e404a5136cf41126f4cb2416ed0ef8fb1ad7aa7da54c37c12ed71b05a527ed08bfac9d50d3550fa5d475c1e8c00950
>> apache-pulsar-client-cpp-3.4.0.tar.gz
>> 
>> 
>> The tag to be voted upon:
>> v3.4.0-candidate-2 (f337eff7caae93730ec1260810655cbb5a345e70)
>> https://github.com/apache/pulsar-client-cpp/releases/tag/v3.4.0-candidate-2
>> 
>> Pulsar's KEYS file containing PGP keys you use to sign the release:
>> https://downloads.apache.org/pulsar/KEYS
>> 
>> Please download the source package, and follow the README to compile and 
>> test.
> 



Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-12 Thread mattison chao
Hi, Yunze

I’ve got several failed tests.

[--] Global test environment tear-down
[==] 528 tests from 85 test suites ran. (888193 ms total)
[  PASSED  ] 525 tests.
[  FAILED  ] 3 tests, listed below:
[  FAILED  ] CustomLoggerTest.testCustomLogger
[  FAILED  ] LookupServiceTest.testMultiAddresses
[  FAILED  ] c_ConsumerTest.testCDeadLetterTopic


After rerun, the second and third tests still failed.

/Users/mattisonchao/projects/apache/pulsar-client-cpp/tests/c/c_ConsumerTest.cc:182:
 Failure
Expected equality of these values:
  pulsar_result_Timeout
Which is: 3
  res
Which is: 0

[  FAILED  ] c_ConsumerTest.testCDeadLetterTopic (75 ms)
[--] 1 test from c_ConsumerTest (75 ms total)

[--] Global test environment tear-down
[==] 1 test from 1 test suite ran. (75 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] c_ConsumerTest.testCDeadLetterTopic


Expected equality of these values:
  std::count(results.cbegin(), results.cend(), ResultRetryable)
Which is: 0
  numRequests / 2
Which is: 3

2023-11-12 22:10:03.538 INFO  [0x1e483d300] ClientConnection:268 | [ -> 
pulsar://localhost:] Destroyed connection to pulsar://localhost:
[  FAILED  ] LookupServiceTest.testMultiAddresses (89 ms)
[--] 1 test from LookupServiceTest (89 ms total)

[--] Global test environment tear-down
[==] 1 test from 1 test suite ran. (89 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] LookupServiceTest.testMultiAddresses



Best,
Mattison

> On Nov 7, 2023, at 15:03, Yunze Xu  wrote:
> 
> This is the second release candidate for Apache Pulsar Client C++,
> version 3.4.0.
> 
> It fixes the following issues:
> https://github.com/apache/pulsar-client-cpp/milestone/5?closed=1
> 
> *** Please download, test and vote on this release. This vote will stay open
> for at least 72 hours ***
> 
> Note that we are voting upon the source (tag), binaries are provided for
> convenience.
> 
> Source and binary files:
> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
> 
> SHA-512 checksums:
> 
> 10517590a2e4296d6767a044e58dd32c79e404a5136cf41126f4cb2416ed0ef8fb1ad7aa7da54c37c12ed71b05a527ed08bfac9d50d3550fa5d475c1e8c00950
> apache-pulsar-client-cpp-3.4.0.tar.gz
> 
> 
> The tag to be voted upon:
> v3.4.0-candidate-2 (f337eff7caae93730ec1260810655cbb5a345e70)
> https://github.com/apache/pulsar-client-cpp/releases/tag/v3.4.0-candidate-2
> 
> Pulsar's KEYS file containing PGP keys you use to sign the release:
> https://downloads.apache.org/pulsar/KEYS
> 
> Please download the source package, and follow the README to compile and test.



Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-09 Thread PengHui Li
+1 (binding)

- Checked the signature
- Build from source and run the tests

Regards,
Penghui

On Thu, Nov 9, 2023 at 5:15 PM Yunze Xu  wrote:

> +1 (binding)
>
> - Verified checksum and signatures
> - Built from source and run unit tests against the test standalone on
> macOS Ventura 13.4 and Apple Clang 14.0.3
>
> A tip for the failed LookupServiceTest.testMultiAddresses I mentioned
> before, it's because this test has a too strict requirement that the
> localhost should be mapped to 127.0.0.1, while on my macOS localhost
> was mapped to IPv6 ::1 first. The test passed after I modified the
> /etc/hosts file.
>
> Thanks,
> Yunze
>
> On Thu, Nov 9, 2023 at 11:42 AM Yunze Xu  wrote:
> >
> > Hi Penghui,
> >
> > It's caused by the relative path and I explained it that issue. Here
> > is an improvement for tests:
> > https://github.com/apache/pulsar-client-cpp/pull/340.
> >
> > Thanks,
> > Yunze
>


Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-09 Thread Yunze Xu
+1 (binding)

- Verified checksum and signatures
- Built from source and run unit tests against the test standalone on
macOS Ventura 13.4 and Apple Clang 14.0.3

A tip for the failed LookupServiceTest.testMultiAddresses I mentioned
before, it's because this test has a too strict requirement that the
localhost should be mapped to 127.0.0.1, while on my macOS localhost
was mapped to IPv6 ::1 first. The test passed after I modified the
/etc/hosts file.

Thanks,
Yunze

On Thu, Nov 9, 2023 at 11:42 AM Yunze Xu  wrote:
>
> Hi Penghui,
>
> It's caused by the relative path and I explained it that issue. Here
> is an improvement for tests:
> https://github.com/apache/pulsar-client-cpp/pull/340.
>
> Thanks,
> Yunze


Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-08 Thread Yunze Xu
Hi Penghui,

It's caused by the relative path and I explained it that issue. Here
is an improvement for tests:
https://github.com/apache/pulsar-client-cpp/pull/340.

Thanks,
Yunze


Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-08 Thread PengHui Li
Hi Yunze,

Thanks for the reminder. I enabled the proxy on my laptop.
After disabling the proxy, the timeout issue is gone.

Now, all the tests get passed except two consistently failed tests related
to the token test.

I have created a GitHub issue.

https://github.com/apache/pulsar-client-cpp/issues/339

Thanks,
Penghui

On Thu, Nov 9, 2023 at 1:33 AM Yunze Xu  wrote:

> I think it's caused by the ClientTest.testConnectTimeout. The
> resources seem not cleaned well in this test so it affected the tests
> later. Could you try the following command that ignores this test?
>
> ```bash
> ./pulsar-tests  --gtest_filter='-ClientTest.testConnectTimeout'
> ```
>
> If the rest of the tests pass, you can open an issue to track this
> flaky test. And I see the local host is "198.18.0.1", did you use any
> proxy when running the tests?
>
>
> > Is it possible to install the CPP client without building from the
> source code?
>
> Currently there is no "true" way to avoid building from source. You
> can edit the libpulsar ruby file to install the 3.4.0-candidate-2 (See
>
> https://github.com/Homebrew/homebrew-core/blob/master/CONTRIBUTING.md#to-contribute-a-fix-to-the-foo-formula
> )
> but it still builds from source. The only advantage is that all 3rd
> party dependencies are guaranteed to be installed from homebrew..
> Maybe we can add a workflow to upload the pre-built libraries for
> macOS in future.
>
> 
>
> BTW, I ran the `pulsar-tests` locally and it failed with two tests:
>
> ```
> [  FAILED  ] 2 tests, listed below:
> [  FAILED  ] CustomLoggerTest.testCustomLogger
> [  FAILED  ] LookupServiceTest.testMultiAddresses
> ```
>
> These tests seem flaky so I reran them individually by:
>
> ```
> ./tests/pulsar-tests --gtest_filter='CustomLoggerTest.*'
> ./tests/pulsar-tests --gtest_filter='*testMultiAddresses'
> ```
>
> `LookupServiceTest.testMultiAddresses` still failed. I will take a look
> soon.
>
> Thanks,
> Yunze
>
>
> On Wed, Nov 8, 2023 at 11:07 PM PengHui Li  wrote:
> >
> > Hi Yunze,
> >
> > I got an error when running the pulsar-tests (I tried multiple times,
> same
> > error)
> >
> > ```
> > 2023-11-08 22:56:43.676 INFO  [0x16db63000] ProducerImpl:216 |
> >
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
> > ] Created producer on broker [[::1]:58479 -> [::1]:6650]
> > 2023-11-08 22:56:43.693 INFO  [0x1f086e080] ProducerImpl:791 |
> >
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0,
> > standalone-0-635] Closing producer for topic
> >
> persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0
> > 2023-11-08 22:56:43.693 INFO  [0x1f086e080] ProducerImpl:791 |
> >
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
> > standalone-0-636] Closing producer for topic
> >
> persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1
> > 2023-11-08 22:56:43.693 INFO  [0x16db63000] ProducerImpl:755 |
> >
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0,
> > standalone-0-635] Closed producer 0
> > 2023-11-08 22:56:43.693 INFO  [0x16db63000] ProducerImpl:755 |
> >
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
> > standalone-0-636] Closed producer 1
> > 2023-11-08 22:56:44.773 WARN  [0x16ba07000] ConnectionPool:91 | Deleting
> > stale connection from pool for pulsar://192.0.2.1:1234-0 use_count: 1 @
> > 0x13000ea00
> > 2023-11-08 22:56:44.773 INFO  [0x16ba07000] ClientConnection:268 | [
> > 198.18.0.1:58467 -> 192.0.2.1:1234] Destroyed connection to pulsar://
> > 192.0.2.1:1234
> > 2023-11-08 22:56:44.773 INFO  [0x16ba07000] ClientConnection:188 |
> [
> > -> pulsar://192.0.2.1:1234] Create ClientConnection, timeout=1000
> > 2023-11-08 22:56:44.773 INFO  [0x16ba07000] ConnectionPool:109 | Created
> > connection for pulsar://192.0.2.1:1234
> > 2023-11-08 22:56:44.775 INFO  [0x16ba07000] ClientConnection:398 | [
> > 198.18.0.1:58482 -> 192.0.2.1:1234] Connected to broker
> > 2023-11-08 22:56:45.774 ERROR [0x16ba07000] ClientConnection:612 | [
> > 198.18.0.1:58482 -> 192.0.2.1:1234] Connection was not established in
> 1000
> > ms, close the socket
> > 2023-11-08 22:56:45.775 INFO  [0x16ba07000] ClientConnection:1319 | [
> > 198.18.0.1:58482 -> 192.0.2.1:1234] Connection disconnected (refCnt: 2)
> > 2023-11-08 22:56:45.775 ERROR [0x16ba07000] ClientImpl:199 | Error
> > Checking/Getting Partition Metadata while creating producer on
> > persistent://public/default/test-connect-timeout -- TimeOut
> > zsh: segmentation fault  ./pulsar-tests
> > ```
> >
> > Reproduce steps:
> >
> > - Build from downloaded source code (passed)
> > - Start pulsar service (./pulsar-test-service-start.sh)
> > - Run the test(./pulsar-tests)
> >
> > Then, I got the above errors after a few minutes.
> > I'm not sure it's just a test 

Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-08 Thread Yunze Xu
I think it's caused by the ClientTest.testConnectTimeout. The
resources seem not cleaned well in this test so it affected the tests
later. Could you try the following command that ignores this test?

```bash
./pulsar-tests  --gtest_filter='-ClientTest.testConnectTimeout'
```

If the rest of the tests pass, you can open an issue to track this
flaky test. And I see the local host is "198.18.0.1", did you use any
proxy when running the tests?


> Is it possible to install the CPP client without building from the source 
> code?

Currently there is no "true" way to avoid building from source. You
can edit the libpulsar ruby file to install the 3.4.0-candidate-2 (See
https://github.com/Homebrew/homebrew-core/blob/master/CONTRIBUTING.md#to-contribute-a-fix-to-the-foo-formula)
but it still builds from source. The only advantage is that all 3rd
party dependencies are guaranteed to be installed from homebrew..
Maybe we can add a workflow to upload the pre-built libraries for
macOS in future.



BTW, I ran the `pulsar-tests` locally and it failed with two tests:

```
[  FAILED  ] 2 tests, listed below:
[  FAILED  ] CustomLoggerTest.testCustomLogger
[  FAILED  ] LookupServiceTest.testMultiAddresses
```

These tests seem flaky so I reran them individually by:

```
./tests/pulsar-tests --gtest_filter='CustomLoggerTest.*'
./tests/pulsar-tests --gtest_filter='*testMultiAddresses'
```

`LookupServiceTest.testMultiAddresses` still failed. I will take a look soon.

Thanks,
Yunze


On Wed, Nov 8, 2023 at 11:07 PM PengHui Li  wrote:
>
> Hi Yunze,
>
> I got an error when running the pulsar-tests (I tried multiple times, same
> error)
>
> ```
> 2023-11-08 22:56:43.676 INFO  [0x16db63000] ProducerImpl:216 |
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
> ] Created producer on broker [[::1]:58479 -> [::1]:6650]
> 2023-11-08 22:56:43.693 INFO  [0x1f086e080] ProducerImpl:791 |
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0,
> standalone-0-635] Closing producer for topic
> persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0
> 2023-11-08 22:56:43.693 INFO  [0x1f086e080] ProducerImpl:791 |
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
> standalone-0-636] Closing producer for topic
> persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1
> 2023-11-08 22:56:43.693 INFO  [0x16db63000] ProducerImpl:755 |
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0,
> standalone-0-635] Closed producer 0
> 2023-11-08 22:56:43.693 INFO  [0x16db63000] ProducerImpl:755 |
> [persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
> standalone-0-636] Closed producer 1
> 2023-11-08 22:56:44.773 WARN  [0x16ba07000] ConnectionPool:91 | Deleting
> stale connection from pool for pulsar://192.0.2.1:1234-0 use_count: 1 @
> 0x13000ea00
> 2023-11-08 22:56:44.773 INFO  [0x16ba07000] ClientConnection:268 | [
> 198.18.0.1:58467 -> 192.0.2.1:1234] Destroyed connection to pulsar://
> 192.0.2.1:1234
> 2023-11-08 22:56:44.773 INFO  [0x16ba07000] ClientConnection:188 | [
> -> pulsar://192.0.2.1:1234] Create ClientConnection, timeout=1000
> 2023-11-08 22:56:44.773 INFO  [0x16ba07000] ConnectionPool:109 | Created
> connection for pulsar://192.0.2.1:1234
> 2023-11-08 22:56:44.775 INFO  [0x16ba07000] ClientConnection:398 | [
> 198.18.0.1:58482 -> 192.0.2.1:1234] Connected to broker
> 2023-11-08 22:56:45.774 ERROR [0x16ba07000] ClientConnection:612 | [
> 198.18.0.1:58482 -> 192.0.2.1:1234] Connection was not established in 1000
> ms, close the socket
> 2023-11-08 22:56:45.775 INFO  [0x16ba07000] ClientConnection:1319 | [
> 198.18.0.1:58482 -> 192.0.2.1:1234] Connection disconnected (refCnt: 2)
> 2023-11-08 22:56:45.775 ERROR [0x16ba07000] ClientImpl:199 | Error
> Checking/Getting Partition Metadata while creating producer on
> persistent://public/default/test-connect-timeout -- TimeOut
> zsh: segmentation fault  ./pulsar-tests
> ```
>
> Reproduce steps:
>
> - Build from downloaded source code (passed)
> - Start pulsar service (./pulsar-test-service-start.sh)
> - Run the test(./pulsar-tests)
>
> Then, I got the above errors after a few minutes.
> I'm not sure it's just a test issue or not.
>
> One more question:
>
> Is it possible to install the CPP client without building from the source
> code?
> From
> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
> I can find the package for Linux and Windows. But no package for macOS.
>
> Regards,
> Penghui
>
> On Wed, Nov 8, 2023 at 10:45 PM Yubiao Feng
>  wrote:
>
> > Hi all
> >
> > Sorry, I'll send another email explaining what tests were done.
> >
> > Please ignore the previous email.
> >
> > Thanks
> > Yubiao Feng
> >
> >
> > On Wed, Nov 8, 2023 at 11:48 AM 

Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-08 Thread PengHui Li
Hi Yunze,

I got an error when running the pulsar-tests (I tried multiple times, same
error)

```
2023-11-08 22:56:43.676 INFO  [0x16db63000] ProducerImpl:216 |
[persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
] Created producer on broker [[::1]:58479 -> [::1]:6650]
2023-11-08 22:56:43.693 INFO  [0x1f086e080] ProducerImpl:791 |
[persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0,
standalone-0-635] Closing producer for topic
persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0
2023-11-08 22:56:43.693 INFO  [0x1f086e080] ProducerImpl:791 |
[persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
standalone-0-636] Closing producer for topic
persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1
2023-11-08 22:56:43.693 INFO  [0x16db63000] ProducerImpl:755 |
[persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-0,
standalone-0-635] Closed producer 0
2023-11-08 22:56:43.693 INFO  [0x16db63000] ProducerImpl:755 |
[persistent://public/default/testPartitionedConsumerUnexpectedAckTimeout1699455403-partition-1,
standalone-0-636] Closed producer 1
2023-11-08 22:56:44.773 WARN  [0x16ba07000] ConnectionPool:91 | Deleting
stale connection from pool for pulsar://192.0.2.1:1234-0 use_count: 1 @
0x13000ea00
2023-11-08 22:56:44.773 INFO  [0x16ba07000] ClientConnection:268 | [
198.18.0.1:58467 -> 192.0.2.1:1234] Destroyed connection to pulsar://
192.0.2.1:1234
2023-11-08 22:56:44.773 INFO  [0x16ba07000] ClientConnection:188 | [
-> pulsar://192.0.2.1:1234] Create ClientConnection, timeout=1000
2023-11-08 22:56:44.773 INFO  [0x16ba07000] ConnectionPool:109 | Created
connection for pulsar://192.0.2.1:1234
2023-11-08 22:56:44.775 INFO  [0x16ba07000] ClientConnection:398 | [
198.18.0.1:58482 -> 192.0.2.1:1234] Connected to broker
2023-11-08 22:56:45.774 ERROR [0x16ba07000] ClientConnection:612 | [
198.18.0.1:58482 -> 192.0.2.1:1234] Connection was not established in 1000
ms, close the socket
2023-11-08 22:56:45.775 INFO  [0x16ba07000] ClientConnection:1319 | [
198.18.0.1:58482 -> 192.0.2.1:1234] Connection disconnected (refCnt: 2)
2023-11-08 22:56:45.775 ERROR [0x16ba07000] ClientImpl:199 | Error
Checking/Getting Partition Metadata while creating producer on
persistent://public/default/test-connect-timeout -- TimeOut
zsh: segmentation fault  ./pulsar-tests
```

Reproduce steps:

- Build from downloaded source code (passed)
- Start pulsar service (./pulsar-test-service-start.sh)
- Run the test(./pulsar-tests)

Then, I got the above errors after a few minutes.
I'm not sure it's just a test issue or not.

One more question:

Is it possible to install the CPP client without building from the source
code?
From
https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
I can find the package for Linux and Windows. But no package for macOS.

Regards,
Penghui

On Wed, Nov 8, 2023 at 10:45 PM Yubiao Feng
 wrote:

> Hi all
>
> Sorry, I'll send another email explaining what tests were done.
>
> Please ignore the previous email.
>
> Thanks
> Yubiao Feng
>
>
> On Wed, Nov 8, 2023 at 11:48 AM Yubiao Feng 
> wrote:
>
> > +1 (no-binding)
> >
> > Thanks
> > Yubiao Feng
> >
> > On Tue, Nov 7, 2023 at 3:03 PM Yunze Xu  wrote:
> >
> >> This is the second release candidate for Apache Pulsar Client C++,
> >> version 3.4.0.
> >>
> >> It fixes the following issues:
> >> https://github.com/apache/pulsar-client-cpp/milestone/5?closed=1
> >>
> >> *** Please download, test and vote on this release. This vote will stay
> >> open
> >> for at least 72 hours ***
> >>
> >> Note that we are voting upon the source (tag), binaries are provided for
> >> convenience.
> >>
> >> Source and binary files:
> >>
> >>
> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
> >>
> >> SHA-512 checksums:
> >>
> >>
> >>
> 10517590a2e4296d6767a044e58dd32c79e404a5136cf41126f4cb2416ed0ef8fb1ad7aa7da54c37c12ed71b05a527ed08bfac9d50d3550fa5d475c1e8c00950
> >>  apache-pulsar-client-cpp-3.4.0.tar.gz
> >>
> >>
> >> The tag to be voted upon:
> >> v3.4.0-candidate-2 (f337eff7caae93730ec1260810655cbb5a345e70)
> >>
> >>
> https://github.com/apache/pulsar-client-cpp/releases/tag/v3.4.0-candidate-2
> >>
> >> Pulsar's KEYS file containing PGP keys you use to sign the release:
> >> https://downloads.apache.org/pulsar/KEYS
> >>
> >> Please download the source package, and follow the README to compile and
> >> test.
> >>
> >
>


Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-08 Thread Yubiao Feng
Hi all

Sorry, I'll send another email explaining what tests were done.

Please ignore the previous email.

Thanks
Yubiao Feng


On Wed, Nov 8, 2023 at 11:48 AM Yubiao Feng 
wrote:

> +1 (no-binding)
>
> Thanks
> Yubiao Feng
>
> On Tue, Nov 7, 2023 at 3:03 PM Yunze Xu  wrote:
>
>> This is the second release candidate for Apache Pulsar Client C++,
>> version 3.4.0.
>>
>> It fixes the following issues:
>> https://github.com/apache/pulsar-client-cpp/milestone/5?closed=1
>>
>> *** Please download, test and vote on this release. This vote will stay
>> open
>> for at least 72 hours ***
>>
>> Note that we are voting upon the source (tag), binaries are provided for
>> convenience.
>>
>> Source and binary files:
>>
>> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
>>
>> SHA-512 checksums:
>>
>>
>> 10517590a2e4296d6767a044e58dd32c79e404a5136cf41126f4cb2416ed0ef8fb1ad7aa7da54c37c12ed71b05a527ed08bfac9d50d3550fa5d475c1e8c00950
>>  apache-pulsar-client-cpp-3.4.0.tar.gz
>>
>>
>> The tag to be voted upon:
>> v3.4.0-candidate-2 (f337eff7caae93730ec1260810655cbb5a345e70)
>>
>> https://github.com/apache/pulsar-client-cpp/releases/tag/v3.4.0-candidate-2
>>
>> Pulsar's KEYS file containing PGP keys you use to sign the release:
>> https://downloads.apache.org/pulsar/KEYS
>>
>> Please download the source package, and follow the README to compile and
>> test.
>>
>


Re: [VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-07 Thread Yubiao Feng
+1 (no-binding)

Thanks
Yubiao Feng

On Tue, Nov 7, 2023 at 3:03 PM Yunze Xu  wrote:

> This is the second release candidate for Apache Pulsar Client C++,
> version 3.4.0.
>
> It fixes the following issues:
> https://github.com/apache/pulsar-client-cpp/milestone/5?closed=1
>
> *** Please download, test and vote on this release. This vote will stay
> open
> for at least 72 hours ***
>
> Note that we are voting upon the source (tag), binaries are provided for
> convenience.
>
> Source and binary files:
>
> https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/
>
> SHA-512 checksums:
>
>
> 10517590a2e4296d6767a044e58dd32c79e404a5136cf41126f4cb2416ed0ef8fb1ad7aa7da54c37c12ed71b05a527ed08bfac9d50d3550fa5d475c1e8c00950
>  apache-pulsar-client-cpp-3.4.0.tar.gz
>
>
> The tag to be voted upon:
> v3.4.0-candidate-2 (f337eff7caae93730ec1260810655cbb5a345e70)
> https://github.com/apache/pulsar-client-cpp/releases/tag/v3.4.0-candidate-2
>
> Pulsar's KEYS file containing PGP keys you use to sign the release:
> https://downloads.apache.org/pulsar/KEYS
>
> Please download the source package, and follow the README to compile and
> test.
>


[VOTE] Pulsar Client C++ Release 3.4.0 Candidate 2

2023-11-06 Thread Yunze Xu
This is the second release candidate for Apache Pulsar Client C++,
version 3.4.0.

It fixes the following issues:
https://github.com/apache/pulsar-client-cpp/milestone/5?closed=1

*** Please download, test and vote on this release. This vote will stay open
for at least 72 hours ***

Note that we are voting upon the source (tag), binaries are provided for
convenience.

Source and binary files:
https://dist.apache.org/repos/dist/dev/pulsar/pulsar-client-cpp/pulsar-client-cpp-3.4.0-candidate-2/

SHA-512 checksums:

10517590a2e4296d6767a044e58dd32c79e404a5136cf41126f4cb2416ed0ef8fb1ad7aa7da54c37c12ed71b05a527ed08bfac9d50d3550fa5d475c1e8c00950
 apache-pulsar-client-cpp-3.4.0.tar.gz


The tag to be voted upon:
v3.4.0-candidate-2 (f337eff7caae93730ec1260810655cbb5a345e70)
https://github.com/apache/pulsar-client-cpp/releases/tag/v3.4.0-candidate-2

Pulsar's KEYS file containing PGP keys you use to sign the release:
https://downloads.apache.org/pulsar/KEYS

Please download the source package, and follow the README to compile and test.