[GitHub] [pulsar] shibd edited a comment on issue #11834: 500 error and stacktrace when attempting to terminate a non-persistent topic via the admin API

2021-09-01 Thread GitBox
shibd edited a comment on issue #11834: URL: https://github.com/apache/pulsar/issues/11834#issuecomment-911224227 @gaoran10 I can fix it. Please assign to me -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [pulsar] shibd commented on issue #11834: 500 error and stacktrace when attempting to terminate a non-persistent topic via the admin API

2021-09-01 Thread GitBox
shibd commented on issue #11834: URL: https://github.com/apache/pulsar/issues/11834#issuecomment-911224227 @gaoran10 I can fix it. -- 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

[GitHub] [pulsar] eolivelli commented on a change in pull request #11852: [Issue 11814] fix pulsar admin method:getMessageById.

2021-09-01 Thread GitBox
eolivelli commented on a change in pull request #11852: URL: https://github.com/apache/pulsar/pull/11852#discussion_r700763721 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -2300,6 +2300,13 @@ protected void

[pulsar] branch master updated (1303e7dc -> 1d5403e)

2021-09-01 Thread eolivelli
This is an automated email from the ASF dual-hosted git repository. eolivelli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 1303e7dc Reduce redundant FLOW requests for non-durable multi-topics consumer (#11802) add 1d5403e

[GitHub] [pulsar] eolivelli merged pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-09-01 Thread GitBox
eolivelli merged pull request #11777: URL: https://github.com/apache/pulsar/pull/11777 -- 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:

[pulsar] 01/02: Fix using partitioned topic name to get Policy (#11294)

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 46d8e9068046f7d6811292247ccac46571a20d73 Author: feynmanlin AuthorDate: Wed Jul 14 14:43:00 2021 +0800 Fix

[pulsar] 02/02: Fix cherry-pick issue after cherry-pick #11294

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 0448c27ecad7de1f318bfb16fde6ce869965c0b1 Author: penghui AuthorDate: Thu Sep 2 13:23:05 2021 +0800 Fix

[pulsar] branch branch-2.8 updated (19b2b69 -> 0448c27)

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 19b2b69 Fix the checkstyle issue during the cherry-pick new 46d8e90 Fix using partitioned topic name to

[pulsar] branch master updated (d0e5d96 -> 1303e7dc)

2021-09-01 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from d0e5d96 [function] enable protobuf-native schema support for function (#11868) add 1303e7dc Reduce redundant

[GitHub] [pulsar] BewareMyPower merged pull request #11802: [Client] Reduce redundant FLOW requests for non-durable multi-topics consumer

2021-09-01 Thread GitBox
BewareMyPower merged pull request #11802: URL: https://github.com/apache/pulsar/pull/11802 -- 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:

[GitHub] [pulsar] lhotari edited a comment on issue #9936: test fails on AArch64, Fedora 33

2021-09-01 Thread GitBox
lhotari edited a comment on issue #9936: URL: https://github.com/apache/pulsar/issues/9936#issuecomment-911172357 Running `mvn test` has issues. Please check the README instructions to build Pulsar. Running all tests requires running the tests the same way CI does. Please check

[GitHub] [pulsar] lhotari commented on issue #9936: test fails on AArch64, Fedora 33

2021-09-01 Thread GitBox
lhotari commented on issue #9936: URL: https://github.com/apache/pulsar/issues/9936#issuecomment-911172357 Please retest and discuss general testing failures on #testing channel in Pulsar Slack. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] lhotari closed issue #9936: test fails on AArch64, Fedora 33

2021-09-01 Thread GitBox
lhotari closed issue #9936: URL: https://github.com/apache/pulsar/issues/9936 -- 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:

[GitHub] [pulsar] lhotari commented on issue #11032: Flaky-test: ZKSessionTest.testReacquireLeadershipAfterSessionLost

2021-09-01 Thread GitBox
lhotari commented on issue #11032: URL: https://github.com/apache/pulsar/issues/11032#issuecomment-911170852 fixed by #11886 -- 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

[GitHub] [pulsar] lhotari closed issue #11032: Flaky-test: ZKSessionTest.testReacquireLeadershipAfterSessionLost

2021-09-01 Thread GitBox
lhotari closed issue #11032: URL: https://github.com/apache/pulsar/issues/11032 -- 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:

[GitHub] [pulsar] lhotari commented on issue #11143: Flaky-test: [org.apache.pulsar.metadata.ZKSessionTest].[testReacquireLeadershipAfterSessionLost]

2021-09-01 Thread GitBox
lhotari commented on issue #11143: URL: https://github.com/apache/pulsar/issues/11143#issuecomment-911170729 duplicates #11032, fixed by #11886 -- 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

[GitHub] [pulsar] lhotari closed issue #11143: Flaky-test: [org.apache.pulsar.metadata.ZKSessionTest].[testReacquireLeadershipAfterSessionLost]

2021-09-01 Thread GitBox
lhotari closed issue #11143: URL: https://github.com/apache/pulsar/issues/11143 -- 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:

[pulsar] branch master updated: [function] enable protobuf-native schema support for function (#11868)

2021-09-01 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new d0e5d96 [function] enable protobuf-native

[GitHub] [pulsar] sijie closed issue #11721: [function] support protobuf_native schema type in pulsar functions

2021-09-01 Thread GitBox
sijie closed issue #11721: URL: https://github.com/apache/pulsar/issues/11721 -- 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:

[GitHub] [pulsar] sijie merged pull request #11868: [function] enable protobuf-native schema support for function

2021-09-01 Thread GitBox
sijie merged pull request #11868: URL: https://github.com/apache/pulsar/pull/11868 -- 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:

[GitHub] [pulsar] lhotari commented on pull request #11884: [Client] Fix ConcurrentModificationException in sendAsync

2021-09-01 Thread GitBox
lhotari commented on pull request #11884: URL: https://github.com/apache/pulsar/pull/11884#issuecomment-911169103 > We can't just guarantee the thread safety of the queue, so we can use a Concurrent queue. The reason why the ProducerImpl object was locked before is because there are other

[GitHub] [pulsar] lhotari closed issue #11857: java.lang.NoSuchMethodError: org.apache.pulsar.broker.ServiceConfigurationUtils.getAppliedAdvertisedAddress(Lorg/apache/pulsar/broker/ServiceConfiguratio

2021-09-01 Thread GitBox
lhotari closed issue #11857: URL: https://github.com/apache/pulsar/issues/11857 -- 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:

[GitHub] [pulsar] lhotari commented on issue #11696: Flaky-test: test_producer_routing_mode (__main__.PulsarTest)

2021-09-01 Thread GitBox
lhotari commented on issue #11696: URL: https://github.com/apache/pulsar/issues/11696#issuecomment-911164068 This might be fixed by #11887 . Closing. -- 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

[GitHub] [pulsar] lhotari closed issue #11696: Flaky-test: test_producer_routing_mode (__main__.PulsarTest)

2021-09-01 Thread GitBox
lhotari closed issue #11696: URL: https://github.com/apache/pulsar/issues/11696 -- 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:

[GitHub] [pulsar] lhotari closed issue #11635: [Tests] "python pulsar_test.py" failed with segmentation fault and core dump

2021-09-01 Thread GitBox
lhotari closed issue #11635: URL: https://github.com/apache/pulsar/issues/11635 -- 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:

[GitHub] [pulsar] lhotari commented on issue #11635: [Tests] "python pulsar_test.py" failed with segmentation fault and core dump

2021-09-01 Thread GitBox
lhotari commented on issue #11635: URL: https://github.com/apache/pulsar/issues/11635#issuecomment-911163162 This might be fixed by #11887 . Closing. We can reopen if this appears in the future. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] casuallc commented on a change in pull request #11813: [ISSUE 11796] throw NPE when readEntry

2021-09-01 Thread GitBox
casuallc commented on a change in pull request #11813: URL: https://github.com/apache/pulsar/pull/11813#discussion_r700732109 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java ## @@ -622,9 +622,11 @@ public void

[GitHub] [pulsar] 315157973 commented on a change in pull request #11813: [ISSUE 11796] throw NPE when readEntry

2021-09-01 Thread GitBox
315157973 commented on a change in pull request #11813: URL: https://github.com/apache/pulsar/pull/11813#discussion_r700726597 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java ## @@ -622,9 +622,11 @@ public void

[GitHub] [pulsar] BewareMyPower opened a new pull request #11889: [C++] Fix incorrect connect timeout implementation

2021-09-01 Thread GitBox
BewareMyPower opened a new pull request #11889: URL: https://github.com/apache/pulsar/pull/11889 Fixes #10721 ### Motivation Currently the connection timer stops after the TCP connection is established immediately (the state becomes `TcpConnected`). However, the connect

[GitHub] [pulsar] BewareMyPower commented on pull request #11802: [Client] Reduce redundant FLOW requests for non-durable multi-topics consumer

2021-09-01 Thread GitBox
BewareMyPower commented on pull request #11802: URL: https://github.com/apache/pulsar/pull/11802#issuecomment-93750 @sijie @codelipenghui @hangc0276 Could you also take a look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] tomscut edited a comment on pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

2021-09-01 Thread GitBox
tomscut edited a comment on pull request #11880: URL: https://github.com/apache/pulsar/pull/11880#issuecomment-90189 > @tomscut I'm so glad to receive your very positive feedback! As you see, Pulsar is skyrocketing globally. It is impossible to achieve such great results without every

[GitHub] [pulsar] tomscut commented on pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

2021-09-01 Thread GitBox
tomscut commented on pull request #11880: URL: https://github.com/apache/pulsar/pull/11880#issuecomment-90189 > @tomscut I'm so glad to receive your very positive feedback! As you see, Pulsar is skyrocketing globally. It is impossible to achieve such great results without every

[GitHub] [pulsar] Anonymitaet edited a comment on pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

2021-09-01 Thread GitBox
Anonymitaet edited a comment on pull request #11880: URL: https://github.com/apache/pulsar/pull/11880#issuecomment-911103252 @tomscut I'm so glad to receive your very positive feedback! As you see, Pulsar is skyrocketing globally. It is impossible to achieve such great results without

[GitHub] [pulsar] Anonymitaet commented on pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

2021-09-01 Thread GitBox
Anonymitaet commented on pull request #11880: URL: https://github.com/apache/pulsar/pull/11880#issuecomment-911103252 @tomscut I'm so glad to receive your very positive feedback! As you see, Pulsar is skyrocketing globally, It is impossible to achieve such great results without every

[GitHub] [pulsar] fewrwee2 commented on issue #11873: 使用avro格式传递数据时报错

2021-09-01 Thread GitBox
fewrwee2 commented on issue #11873: URL: https://github.com/apache/pulsar/issues/11873#issuecomment-911098910 @congbobo184 source的配置 ![image](https://user-images.githubusercontent.com/88083754/131776291-87d527e1-f884-457f-944d-838a34ad47a6.png) avro配置 { "type": "AVRO",

[pulsar] 05/05: Fix the checkstyle issue during the cherry-pick

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 19b2b69983ed4ac141fe037dfbbc43ac50267fca Author: penghui AuthorDate: Thu Sep 2 11:04:02 2021 +0800 Fix the

[pulsar] 03/05: Expose compaction metrics to Prometheus (#11739)

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit a80f1e8774d1c8f6505f19ed8ec1b97d75fa47d9 Author: GuoJiwei AuthorDate: Tue Aug 31 22:58:44 2021 +0800 Expose

[pulsar] 04/05: [Broker] Refine topic level backlog quota policies warning log (#11863)

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit bbed232f10bfdf44bf288c27610a71d99a922aa9 Author: ran AuthorDate: Thu Sep 2 09:44:17 2021 +0800 [Broker]

[pulsar] 02/05: Add compacted topic metrics for TopicStats in CLI (#11564)

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 71cc13be016646da47ba7298c585ae1f484507a5 Author: GuoJiwei AuthorDate: Sat Aug 21 22:58:20 2021 +0800 Add

[pulsar] 01/05: [python-client] Fixed crash when using Python logger (#10981)

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 875bfdfc879bdef09be3c6cebeaa43fdce192ebb Author: Livio Benčik AuthorDate: Wed Jul 14 09:02:34 2021 +0200

[pulsar] branch branch-2.8 updated (6bc1e0d -> 19b2b69)

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 6bc1e0d Source tarball: apply executable file permissions to shell scripts (fixes #10917) (#11858) new

[GitHub] [pulsar] Anonymitaet edited a comment on pull request #11739: Expose compaction metrics to Prometheus

2021-09-01 Thread GitBox
Anonymitaet edited a comment on pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#issuecomment-911083313 Hi @sijia-w, in this PR, @Technoboy- has already added docs to`master`. - Could you please add the same contents to `2.8.2`? - If you find any inaccurate

[GitHub] [pulsar] Anonymitaet commented on pull request #11739: Expose compaction metrics to Prometheus

2021-09-01 Thread GitBox
Anonymitaet commented on pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#issuecomment-911083313 Hi @sijia-w, in this PR, @Technoboy- has already added docs to`master`. - Could you please add the same contents to `2.8.2`? - If you find any inaccurate

[GitHub] [pulsar] congbobo184 commented on issue #11873: 使用avro格式传递数据时报错

2021-09-01 Thread GitBox
congbobo184 commented on issue #11873: URL: https://github.com/apache/pulsar/issues/11873#issuecomment-911082087 @feiyuyitiao could you please provide mysql schema and pulsar avro schema? What the type of the `K`? ```org.apache.avro.AvroTypeException: Unknown type: K``` -- This is

[GitHub] [pulsar] casuallc commented on issue #11796: throw NPE when readEntry

2021-09-01 Thread GitBox
casuallc commented on issue #11796: URL: https://github.com/apache/pulsar/issues/11796#issuecomment-911068781 > Did you see this error in production ? Nope. When I add some other functions in broker, this error appeared. ```java 11:19:30.843

[GitHub] [pulsar] casuallc commented on a change in pull request #11813: [ISSUE 11796] throw NPE when readEntry

2021-09-01 Thread GitBox
casuallc commented on a change in pull request #11813: URL: https://github.com/apache/pulsar/pull/11813#discussion_r700700625 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java ## @@ -762,6 +764,9 @@ public void

[GitHub] [pulsar] gaoran10 commented on issue #11834: 500 error and stacktrace when attempting to terminate a non-persistent topic via the admin API

2021-09-01 Thread GitBox
gaoran10 commented on issue #11834: URL: https://github.com/apache/pulsar/issues/11834#issuecomment-911064886 I add a help wanted label first. -- 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

[GitHub] [pulsar] casuallc commented on a change in pull request #11813: [ISSUE 11796] throw NPE when readEntry

2021-09-01 Thread GitBox
casuallc commented on a change in pull request #11813: URL: https://github.com/apache/pulsar/pull/11813#discussion_r700700625 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java ## @@ -762,6 +764,9 @@ public void

[GitHub] [pulsar] tomscut commented on pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

2021-09-01 Thread GitBox
tomscut commented on pull request #11880: URL: https://github.com/apache/pulsar/pull/11880#issuecomment-911060230 > Thanks for your contribution. For this PR, do we need to update docs? > > (The [PR template contains info about

[pulsar] branch master updated (f0e20cb -> 235e678)

2021-09-01 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from f0e20cb [Test] Fix managed cursor metrics test (#11879) add 235e678 [Python] Ensure producer is keeping the

[GitHub] [pulsar] BewareMyPower merged pull request #11887: [Python] Ensure producer is keeping the client object alive

2021-09-01 Thread GitBox
BewareMyPower merged pull request #11887: URL: https://github.com/apache/pulsar/pull/11887 -- 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:

[GitHub] [pulsar] BewareMyPower closed issue #6463: pulsar-client 2.5 - Segmentation fault: 11 on Mac OS 10.14.6 using Python 3.7.6

2021-09-01 Thread GitBox
BewareMyPower closed issue #6463: URL: https://github.com/apache/pulsar/issues/6463 -- 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:

[GitHub] [pulsar] Anonymitaet commented on issue #11869: Is there any development documentation for the authentication and authorization plug-in, and what is the address?

2021-09-01 Thread GitBox
Anonymitaet commented on issue #11869: URL: https://github.com/apache/pulsar/issues/11869#issuecomment-911049195 Hi @beyondyinjl2 thanks for your question. Before raising an issue, please make sure you have investigated it thoroughly. When you opening a doc issue, please share your

[GitHub] [pulsar] Anonymitaet commented on pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

2021-09-01 Thread GitBox
Anonymitaet commented on pull request #11880: URL: https://github.com/apache/pulsar/pull/11880#issuecomment-911042150 Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about

[GitHub] [pulsar] tomscut commented on pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

2021-09-01 Thread GitBox
tomscut commented on pull request #11880: URL: https://github.com/apache/pulsar/pull/11880#issuecomment-911032463 Hi @merlimat , I made some changes. Please take a look. Thank you. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[pulsar] branch master updated (9bfb3db -> f0e20cb)

2021-09-01 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 9bfb3db [Issue 11814] fix pulsar admin method:getMessageById. (#11852) add f0e20cb [Test] Fix managed cursor

[GitHub] [pulsar] sijie merged pull request #11879: [Test] Fix managed cursor metrics test

2021-09-01 Thread GitBox
sijie merged pull request #11879: URL: https://github.com/apache/pulsar/pull/11879 -- 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:

[pulsar] branch master updated (44becbf -> 9bfb3db)

2021-09-01 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 44becbf [pulsar-client-tools] fix packages tool parameter desc (#11809) add 9bfb3db [Issue 11814] fix pulsar

[GitHub] [pulsar] sijie merged pull request #11852: [Issue 11814] fix pulsar admin method:getMessageById.

2021-09-01 Thread GitBox
sijie merged pull request #11852: URL: https://github.com/apache/pulsar/pull/11852 -- 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:

[GitHub] [pulsar] sijie closed issue #11814: [Pulsar admin] admin command 'get-message-by-id' can get message by messageId regardless of topic name

2021-09-01 Thread GitBox
sijie closed issue #11814: URL: https://github.com/apache/pulsar/issues/11814 -- 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:

[pulsar] branch master updated: [pulsar-client-tools] fix packages tool parameter desc (#11809)

2021-09-01 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 44becbf [pulsar-client-tools] fix packages

[GitHub] [pulsar] sijie merged pull request #11809: [pulsar-client-tools] fix packages tool parameter desc

2021-09-01 Thread GitBox
sijie merged pull request #11809: URL: https://github.com/apache/pulsar/pull/11809 -- 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:

[pulsar] branch master updated (2f4b890 -> d370d00)

2021-09-01 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 2f4b890 [Doc] generate doc set for 2.8.2 (#11870) add d370d00 [Broker] Refine topic level backlog quota

[GitHub] [pulsar] sijie merged pull request #11863: [Broker] Refine topic level backlog quota policies warning log

2021-09-01 Thread GitBox
sijie merged pull request #11863: URL: https://github.com/apache/pulsar/pull/11863 -- 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:

[GitHub] [pulsar] Shoothzj edited a comment on pull request #11884: [Client] Fix ConcurrentModificationException in sendAsync

2021-09-01 Thread GitBox
Shoothzj edited a comment on pull request #11884: URL: https://github.com/apache/pulsar/pull/11884#issuecomment-911011520 > @merlimat @codelipenghui does this PR make sense? any recommendation how to add tests? Can we do that? 1. init producer 2. close broker 3. send

[GitHub] [pulsar] Shoothzj edited a comment on pull request #11884: [Client] Fix ConcurrentModificationException in sendAsync

2021-09-01 Thread GitBox
Shoothzj edited a comment on pull request #11884: URL: https://github.com/apache/pulsar/pull/11884#issuecomment-911011520 > @merlimat @codelipenghui does this PR make sense? any recommendation how to add tests? Can we do that? 1. init producer 2. close broker 3. send

[GitHub] [pulsar] Shoothzj commented on pull request #11884: [Client] Fix ConcurrentModificationException in sendAsync

2021-09-01 Thread GitBox
Shoothzj commented on pull request #11884: URL: https://github.com/apache/pulsar/pull/11884#issuecomment-911011520 > @merlimat @codelipenghui does this PR make sense? any recommendation how to add tests? Can we do that? 1. init producer 2. close broker 3. send messages

[GitHub] [pulsar] nlu90 commented on pull request #11868: [function] enable protobuf-native schema support for function

2021-09-01 Thread GitBox
nlu90 commented on pull request #11868: URL: https://github.com/apache/pulsar/pull/11868#issuecomment-910999091 /pulsarbot run-failure-checks -- 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

[GitHub] [pulsar] urfreespace commented on pull request #11888: [website][upgrade]feat: docs migration about Pulsar Schema (versions: next/2.8.0/2.7.3)

2021-09-01 Thread GitBox
urfreespace commented on pull request #11888: URL: https://github.com/apache/pulsar/pull/11888#issuecomment-910992328 @tuteng @Anonymitaet PTAL, thanks. -- 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

[GitHub] [pulsar] tomscut commented on a change in pull request #11880: [Broker] Remove redundant log guards for pulsar-broker

2021-09-01 Thread GitBox
tomscut commented on a change in pull request #11880: URL: https://github.com/apache/pulsar/pull/11880#discussion_r700670687 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/PulsarClusterMetadataTeardown.java ## @@ -146,9 +146,7 @@ public static void main(String[]

[pulsar] branch master updated (bc49191 -> 2f4b890)

2021-09-01 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from bc49191 Fixed ZKSessionTest.testReacquireLocksAfterSessionLost (#11886) add 2f4b890 [Doc] generate doc set

[GitHub] [pulsar] codelipenghui merged pull request #11870: [Doc] generate doc set for 2.8.2

2021-09-01 Thread GitBox
codelipenghui merged pull request #11870: URL: https://github.com/apache/pulsar/pull/11870 -- 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:

[GitHub] [pulsar] urfreespace opened a new pull request #11888: [website][upgrade]feat: docs migration about Pulsar Schema (versions: next)

2021-09-01 Thread GitBox
urfreespace opened a new pull request #11888: URL: https://github.com/apache/pulsar/pull/11888 Master Issue: https://github.com/apache/pulsar/issues/11766 ### Motivation website upgrade ### Modifications feat: docs migration about Pulsar Schema (versions:

[GitHub] [pulsar] nlu90 commented on pull request #11868: [function] enable protobuf-native schema support for function

2021-09-01 Thread GitBox
nlu90 commented on pull request #11868: URL: https://github.com/apache/pulsar/pull/11868#issuecomment-910863558 > Could we add a test for the schema `PROTOBUF_NATIVE`? added a test class for `TopicSchema`. PTAL -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] zbentley commented on issue #11823: Using the Python pulsar client with a logger can cause arbitrary/unrelated `async` Python functions to return `None`

2021-09-01 Thread GitBox
zbentley commented on issue #11823: URL: https://github.com/apache/pulsar/issues/11823#issuecomment-910843883 I did not know about the weekly releases, thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [pulsar] michaeljmarshall commented on pull request #11872: [Broker] Add Authz Check Before Completing Topic Level Policy Operation

2021-09-01 Thread GitBox
michaeljmarshall commented on pull request #11872: URL: https://github.com/apache/pulsar/pull/11872#issuecomment-910828509 @gaoran10 and @eolivelli - thank you for your feedback. In writing the tests, I discovered that my initial comments were overstated. There is an authorization check

[GitHub] [pulsar] merlimat opened a new pull request #11887: [Python] Ensure producer is keeping the client object alive

2021-09-01 Thread GitBox
merlimat opened a new pull request #11887: URL: https://github.com/apache/pulsar/pull/11887 ### Motivation Fix #6463. When a producer is created, keep a reference on the client object so that the Python GC will not destroy it. This is similar to what we were already doing

[pulsar] branch master updated: Fixed ZKSessionTest.testReacquireLocksAfterSessionLost (#11886)

2021-09-01 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new bc49191 Fixed

[GitHub] [pulsar] lhotari merged pull request #11886: Fixed ZKSessionTest.testReacquireLocksAfterSessionLost

2021-09-01 Thread GitBox
lhotari merged pull request #11886: URL: https://github.com/apache/pulsar/pull/11886 -- 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:

[pulsar] branch master updated: Fixed ResourceLockTest.revalidateLockOnDifferentSession() (#11885)

2021-09-01 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new f42dd55 Fixed

[GitHub] [pulsar] lhotari merged pull request #11885: Fixed ResourceLockTest.revalidateLockOnDifferentSession()

2021-09-01 Thread GitBox
lhotari merged pull request #11885: URL: https://github.com/apache/pulsar/pull/11885 -- 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:

[GitHub] [pulsar] lhotari closed issue #11690: Flaky-test: LockManagerTest.revalidateLockOnDifferentSession

2021-09-01 Thread GitBox
lhotari closed issue #11690: URL: https://github.com/apache/pulsar/issues/11690 -- 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:

[GitHub] [pulsar] merlimat commented on issue #6463: pulsar-client 2.5 - Segmentation fault: 11 on Mac OS 10.14.6 using Python 3.7.6

2021-09-01 Thread GitBox
merlimat commented on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-910786724 > The cyclic GC works "all right", not perfectly, in Python 2, but is much improved in 3. I think this can be done without changes in the native code. I wouldn't even need to

[GitHub] [pulsar] merlimat commented on issue #6463: pulsar-client 2.5 - Segmentation fault: 11 on Mac OS 10.14.6 using Python 3.7.6

2021-09-01 Thread GitBox
merlimat commented on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-910783869 > @merlimat I suggest solving it in Python: have producer objects, when created, store a Python-side reference to the Client that created them. Python's cyclic garbage collector

[GitHub] [pulsar] merlimat commented on issue #6463: pulsar-client 2.5 - Segmentation fault: 11 on Mac OS 10.14.6 using Python 3.7.6

2021-09-01 Thread GitBox
merlimat commented on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-910782962 > Though I guess that raises a larger question: for consumers of the C++ API, it might be desirable in some cases to provide a producer object without providing the ability to

[GitHub] [pulsar] zbentley edited a comment on issue #6463: pulsar-client 2.5 - Segmentation fault: 11 on Mac OS 10.14.6 using Python 3.7.6

2021-09-01 Thread GitBox
zbentley edited a comment on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-910747646 Though I guess that raises a larger question: for consumers of the C++ API, it might be desirable in some cases to provide a producer object without providing the ability to

[GitHub] [pulsar] zbentley commented on issue #6463: pulsar-client 2.5 - Segmentation fault: 11 on Mac OS 10.14.6 using Python 3.7.6

2021-09-01 Thread GitBox
zbentley commented on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-910747646 Though I guess that raises a larger question: for consumers of the C++ API, it might be desirable in some cases to provide a producer object without providing the ability to create

[GitHub] [pulsar] zbentley edited a comment on issue #6463: pulsar-client 2.5 - Segmentation fault: 11 on Mac OS 10.14.6 using Python 3.7.6

2021-09-01 Thread GitBox
zbentley edited a comment on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-910743817 @merlimat I suggest solving it in Python: have producer objects, when created, store a Python-side reference to the Client that created them. Python's cyclic garbage

[GitHub] [pulsar] zbentley commented on issue #6463: pulsar-client 2.5 - Segmentation fault: 11 on Mac OS 10.14.6 using Python 3.7.6

2021-09-01 Thread GitBox
zbentley commented on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-910743817 @merlimat I suggest solving it in Python: have producer objects, when created, store a Python-side reference to the Client that created them. Python's cyclic garbage collector

[GitHub] [pulsar] merlimat opened a new pull request #11886: Fixed ZKSessionTest.testReacquireLocksAfterSessionLost

2021-09-01 Thread GitBox
merlimat opened a new pull request #11886: URL: https://github.com/apache/pulsar/pull/11886 ### Motivation Fixes the flakiness in `ZKSessionTest.testReacquireLocksAfterSessionLost` test: ``` Error:

[GitHub] [pulsar] merlimat commented on pull request #11815: fix flaky test testReacquireLocksAfterSessionLost

2021-09-01 Thread GitBox
merlimat commented on pull request #11815: URL: https://github.com/apache/pulsar/pull/11815#issuecomment-910557271 Yes, the error is still happening and as @lhotari mentioned, we shouldn't even need to use Awaitily here: we're testing that the future is not completed, so if the assertion

[pulsar] branch master updated: [Tests] Fix BaseMetadataStoreTest retries (#11778)

2021-09-01 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 499713c [Tests] Fix BaseMetadataStoreTest

[GitHub] [pulsar] merlimat merged pull request #11778: [Tests] Fix BaseMetadataStoreTest retries

2021-09-01 Thread GitBox
merlimat merged pull request #11778: URL: https://github.com/apache/pulsar/pull/11778 -- 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:

[GitHub] [pulsar] merlimat commented on a change in pull request #11867: Added MetadataStore deleteRecursive operation

2021-09-01 Thread GitBox
merlimat commented on a change in pull request #11867: URL: https://github.com/apache/pulsar/pull/11867#discussion_r700445608 ## File path: pulsar-metadata/src/main/java/org/apache/pulsar/metadata/api/MetadataStore.java ## @@ -112,6 +112,15 @@ */ CompletableFuture

[GitHub] [pulsar] merlimat commented on a change in pull request #11867: Added MetadataStore deleteRecursive operation

2021-09-01 Thread GitBox
merlimat commented on a change in pull request #11867: URL: https://github.com/apache/pulsar/pull/11867#discussion_r700445165 ## File path: pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataStoreTest.java ## @@ -300,4 +300,30 @@ public void

[pulsar] branch master updated: Upgrade Netty to 4.1.67.Final (#11875)

2021-09-01 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 0a99206 Upgrade Netty to 4.1.67.Final (#11875)

[GitHub] [pulsar] merlimat merged pull request #11875: Upgrade Netty to 4.1.67.Final

2021-09-01 Thread GitBox
merlimat merged pull request #11875: URL: https://github.com/apache/pulsar/pull/11875 -- 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:

[GitHub] [pulsar] BewareMyPower commented on issue #10721: (2.7.2) Python client blocks forever in "create_producer" during a broker failure, even if "operation_timeout_seconds" is set

2021-09-01 Thread GitBox
BewareMyPower commented on issue #10721: URL: https://github.com/apache/pulsar/issues/10721#issuecomment-910517590 It's a bug of connect timeout. I'll push the fix soon. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] merlimat opened a new pull request #11885: Fixed ResourceLockTest.revalidateLockOnDifferentSession()

2021-09-01 Thread GitBox
merlimat opened a new pull request #11885: URL: https://github.com/apache/pulsar/pull/11885 ### Motivation Fixes #11690 The main issue in the test is that there are 2 `ResourceLock` instances that are competing. * `RL_1` acquires the lock * `RL_2` tries to

[GitHub] [pulsar] BewareMyPower edited a comment on issue #10721: (2.7.2) Python client blocks forever in "create_producer" during a broker failure, even if "operation_timeout_seconds" is set

2021-09-01 Thread GitBox
BewareMyPower edited a comment on issue #10721: URL: https://github.com/apache/pulsar/issues/10721#issuecomment-910490369 I found the reason why I can't reproduce it. Because I used to just type Ctrl+C to stop the standalone (the signal is SIGINT but not SIGSTOP, my fault). With your

  1   2   3   >