[GitHub] [pulsar] michaeljmarshall commented on a change in pull request #11794: jwks feature for oAuth flow

2021-08-31 Thread GitBox
michaeljmarshall commented on a change in pull request #11794: URL: https://github.com/apache/pulsar/pull/11794#discussion_r699864775 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationProviderTokenJWK.java ## @@ -0,0 +1,358

[GitHub] [pulsar] lhotari closed issue #11768: Failed to restart pulsar standalone

2021-08-31 Thread GitBox
lhotari closed issue #11768: URL: https://github.com/apache/pulsar/issues/11768 -- 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 #11768: Failed to restart pulsar standalone

2021-08-31 Thread GitBox
lhotari commented on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-909915345 > The issue remains that it is unable to start Pulsar standalone after an unclean shutdown. It's more like a mitigation to fix the graceful shutdown by using `exec bin/pulsar` in

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

2021-08-31 Thread GitBox
michaeljmarshall commented on pull request #11872: URL: https://github.com/apache/pulsar/pull/11872#issuecomment-909913704 @sijie, @merlimat, @codelipenghui, @eolivelli, @KannarFr - PTAL -- This is an automated message from the Apache Git Service. To respond to the message, please log on

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

2021-08-31 Thread GitBox
lhotari closed issue #11871: URL: https://github.com/apache/pulsar/issues/11871 -- 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 #11871: Flaky-test: [LockManagerTest].[revalidateLockOnDifferentSession]

2021-08-31 Thread GitBox
lhotari commented on issue #11871: URL: https://github.com/apache/pulsar/issues/11871#issuecomment-909913355 This is a duplicate of #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

[GitHub] [pulsar] michaeljmarshall opened a new pull request #11872: [Broker] Add Authz Check Before Completing Policy Operation

2021-08-31 Thread GitBox
michaeljmarshall opened a new pull request #11872: URL: https://github.com/apache/pulsar/pull/11872 ### Motivation There are several calls in the v2/PeristentTopics endpoint class that are supposed to require authorization but never verify that authorization. These endpoints are

[GitHub] [pulsar] yuruguo commented on pull request #11797: [testclient] Fix IndexOutOfBoundsException when --subscriptions size is less than --num-subscriptions in consumption

2021-08-31 Thread GitBox
yuruguo commented on pull request #11797: URL: https://github.com/apache/pulsar/pull/11797#issuecomment-909904455 If there is no problem, can you merge it? @eolivelli @codelipenghui @hangc0276 Thx! -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] Anonymitaet merged pull request #11851: [website][upgrade]feat: docs migration about Concepts and Architecture (versions: next/2.8.0/2.7.3)

2021-08-31 Thread GitBox
Anonymitaet merged pull request #11851: URL: https://github.com/apache/pulsar/pull/11851 -- 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 closed issue #11766: [website] website upgrade

2021-08-31 Thread GitBox
Anonymitaet closed issue #11766: URL: https://github.com/apache/pulsar/issues/11766 -- 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] michaeljmarshall commented on a change in pull request #11863: [Broker] Refine topic level backlog quota policies warning log

2021-08-31 Thread GitBox
michaeljmarshall commented on a change in pull request #11863: URL: https://github.com/apache/pulsar/pull/11863#discussion_r699821334 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BacklogQuotaManager.java ## @@ -91,9 +91,12 @@ public

[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-08-31 Thread GitBox
BewareMyPower commented on issue #10721: URL: https://github.com/apache/pulsar/issues/10721#issuecomment-909838082 I still cannot reproduce the bug. For the first log file, it looks like the reconnection logs only happened for `send`, instead of `create_producer`. I ran your code with a

[GitHub] [pulsar] Anonymitaet commented on pull request #11870: [Doc] generate doc set for 2.8.2

2021-08-31 Thread GitBox
Anonymitaet commented on pull request #11870: URL: https://github.com/apache/pulsar/pull/11870#issuecomment-909837276 @eolivelli @Jennifer88huang @Huanli-Meng Could you please help review this PR? Thanks -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] gaoran10 opened a new issue #11871: Flaky-test: [LockManagerTest].[revalidateLockOnDifferentSession]

2021-08-31 Thread GitBox
gaoran10 opened a new issue #11871: URL: https://github.com/apache/pulsar/issues/11871 [LockManagerTest] is flaky. The [revalidateLockOnDifferentSession] test method fails sporadically. Full exception stacktrace Error:

[GitHub] [pulsar] Anonymitaet opened a new pull request #11870: [Doc] generate doc set for 2.8.2

2021-08-31 Thread GitBox
Anonymitaet opened a new pull request #11870: URL: https://github.com/apache/pulsar/pull/11870 Currently, we have doc requirements for 2.8.2 (for example, https://github.com/apache/pulsar/pull/11739). Consequently, I generate the doc set for 2.8.2. I'll update the following files

[GitHub] [pulsar] chgxtony commented on issue #11158: [Pulsar Sql] The number of query result records of pulsar SQL is less than that sent by producer

2021-08-31 Thread GitBox
chgxtony commented on issue #11158: URL: https://github.com/apache/pulsar/issues/11158#issuecomment-909830815 I have the same problem -- 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] beyondyinjl2 opened a new issue #11869: Is there any development documentation for the authentication and authorization plug-in, and what is the address?

2021-08-31 Thread GitBox
beyondyinjl2 opened a new issue #11869: URL: https://github.com/apache/pulsar/issues/11869 Is there any development documentation for the authentication and authorization plug-in, and what is the address? -- This is an automated message from the Apache Git Service. To respond to the

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

2021-08-31 Thread GitBox
BewareMyPower commented on issue #11823: URL: https://github.com/apache/pulsar/issues/11823#issuecomment-909826703 It should only be published to PyPi for stable versions. For master branch, you need to build from source. If the related PR was already cherry-picked to branch of last

[GitHub] [pulsar] 315157973 commented on issue #11783: ConcurrentModificationException occurs when failPendingMessages

2021-08-31 Thread GitBox
315157973 commented on issue #11783: URL: https://github.com/apache/pulsar/issues/11783#issuecomment-909820390 In my opinion, we can only use interfaces to restrict the use of methods, verbal conventions are not a good way -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar] zhanghaou commented on pull request #11852: [Issue 11814] fix pulsar admin method:getMessageById.

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

[GitHub] [pulsar] beyondyinjl2 commented on issue #11860: topic: persisten://public/default/my-topic/# This format defines an error

2021-08-31 Thread GitBox
beyondyinjl2 commented on issue #11860: URL: https://github.com/apache/pulsar/issues/11860#issuecomment-909804319 code: PulsarClient client = PulsarClient.builder() .serviceUrl("pulsar://192.168.146.17:6650,192.168.146.18:6650,192.168.146.19:6650")

[GitHub] [pulsar] Anonymitaet commented on issue #11855: Can not found doc for "Enable pulsar function to send message to external pulsar cluster "

2021-08-31 Thread GitBox
Anonymitaet commented on issue #11855: URL: https://github.com/apache/pulsar/issues/11855#issuecomment-909778988 @liangyuanpeng thanks for your issue. Since the PR https://github.com/apache/pulsar/pull/8434 was already labeled with `doc-required`, there is no need to label this issue with

[GitHub] [pulsar] Anonymitaet closed issue #11855: Can not found doc for "Enable pulsar function to send message to external pulsar cluster "

2021-08-31 Thread GitBox
Anonymitaet closed issue #11855: URL: https://github.com/apache/pulsar/issues/11855 -- 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 a change in pull request #11864: Add basic DOAP file for Pulsar

2021-08-31 Thread GitBox
Anonymitaet commented on a change in pull request #11864: URL: https://github.com/apache/pulsar/pull/11864#discussion_r699769382 ## File path: site2/website/static/doap.rdf ## @@ -0,0 +1,45 @@ + + +http://usefulinc.com/ns/doap#; +

[GitHub] [pulsar] Anonymitaet commented on pull request #11856: [Schema] Schema compatibility strategy in broker level.

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

[GitHub] [pulsar] Anonymitaet commented on pull request #11852: [Issue 11814] fix pulsar admin method:getMessageById.

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

[GitHub] [pulsar] rdhabalia commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-31 Thread GitBox
rdhabalia commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-909757056 it's not a major change or api change so, it won't require documentation. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] zbentley 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-08-31 Thread GitBox
zbentley commented on issue #10721: URL: https://github.com/apache/pulsar/issues/10721#issuecomment-909692933 On master, which has merged https://github.com/apache/pulsar/pull/11029, the issue still occurs. I built _pulsar.so against `master` and re-ran the above snippet twice: once

[GitHub] [pulsar] nlu90 opened a new pull request #11868: [function] enable protobuf-native schema support for function

2021-08-31 Thread GitBox
nlu90 opened a new pull request #11868: URL: https://github.com/apache/pulsar/pull/11868 Fixes #11721 ### Motivation Enable function proces topic with protobuf_native schema ### Modifications update `TopicSchema` ### Verifying this change - [ ]

[pulsar] branch master updated: [PIP-82] [pulsar-broker] Misc fixes: (#11821)

2021-08-31 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 60a5698 [PIP-82] [pulsar-broker] Misc

[GitHub] [pulsar] jerrypeng merged pull request #11821: [PIP-82] [pulsar-broker] Misc fixes:

2021-08-31 Thread GitBox
jerrypeng merged pull request #11821: URL: https://github.com/apache/pulsar/pull/11821 -- 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] eolivelli commented on a change in pull request #11867: Added MetadataStore deleteRecursive operation

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

[GitHub] [pulsar] zbentley 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-08-31 Thread GitBox
zbentley commented on issue #10721: URL: https://github.com/apache/pulsar/issues/10721#issuecomment-909623281 Sorry for taking so long to come back to this. I was able to reproduce the issue on 2.8.0 with the `2.8.0.post0` Python client (MacOS 10.11, Python 3.6.9). Better yet,

[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-08-31 Thread GitBox
zbentley commented on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-909609025 @BewareMyPower I suggest updating this issue's description to reflect affected platforms, e.g. `Pulsar client segfaults Python if any Producer instance outlives the garbage

[GitHub] [pulsar] zbentley commented on issue #11866: Attempting to delete a namespace immediately after deleting all topics in that namespace raises `org.apache.zookeeper.KeeperException$NotEmptyExce

2021-08-31 Thread GitBox
zbentley commented on issue #11866: URL: https://github.com/apache/pulsar/issues/11866#issuecomment-909590977 This occurred against a standalone broker; are there multiple (e.g. thread local) metadata caches in use in standalone as well? -- This is an automated message from the Apache

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

2021-08-31 Thread GitBox
zbentley commented on issue #11635: URL: https://github.com/apache/pulsar/issues/11635#issuecomment-909589405 If it's the same segfault I mentioned [here](https://github.com/apache/pulsar/issues/6463#issuecomment-909587000), that may provide a means to more predictably reproduce it.

[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-08-31 Thread GitBox
zbentley commented on issue #6463: URL: https://github.com/apache/pulsar/issues/6463#issuecomment-909587000 I am also affected by this (pulsar client `2.8.0.post0`, broker 2.8.0 standalone, Python 3.9.6 on MacOS 10.11). The following script reliably reproduces the segfault: ```

[GitHub] [pulsar] merlimat commented on issue #11866: Attempting to delete a namespace immediately after deleting all topics in that namespace raises `org.apache.zookeeper.KeeperException$NotEmptyExce

2021-08-31 Thread GitBox
merlimat commented on issue #11866: URL: https://github.com/apache/pulsar/issues/11866#issuecomment-909558022 The reason is probably related to the caching of metadata in brokers and that the the different requests are hitting different brokers. When you delete all topics, when you

[GitHub] [pulsar] merlimat opened a new pull request #11867: Added MetadataStore deleteRecursive operation

2021-08-31 Thread GitBox
merlimat opened a new pull request #11867: URL: https://github.com/apache/pulsar/pull/11867 ### Motivation There are several cases in which the recursive deletion of metadata is done in the code base. Instead we should just have a single implementation, directly in the

[GitHub] [pulsar] zbentley opened a new issue #11866: Attempting to delete a namespace immediately after deleting all topics in that namespace raises `org.apache.zookeeper.KeeperException$NotEmptyExce

2021-08-31 Thread GitBox
zbentley opened a new issue #11866: URL: https://github.com/apache/pulsar/issues/11866 **Describe the bug** If I delete a namespace (without `force`) via the V2 admin API immediately (less than a second) after issuing deletes for all topics in that namespace, I get a stacktrace and a

[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-08-31 Thread GitBox
zbentley commented on issue #11823: URL: https://github.com/apache/pulsar/issues/11823#issuecomment-909421877 Thanks again for the fix! What's the Pulsar policy on cutting new client releases? Should your bugfix cause publication of updated Python client packages to PyPi, or should

[GitHub] [pulsar] michaeljmarshall commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-31 Thread GitBox
michaeljmarshall commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-909418006 > @michaeljmarshall if it is a major change or affects how to use this feature, maybe you add a note about the **previous usage** and **current changes**

[pulsar] branch master updated (9153e71 -> a68853f)

2021-08-31 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 9153e71 [Python] Handle py::call_method error without mutating internal state (#11840) add a68853f [website]

[GitHub] [pulsar] merlimat commented on pull request #11865: [website] Companies Powered by Pulsar

2021-08-31 Thread GitBox
merlimat commented on pull request #11865: URL: https://github.com/apache/pulsar/pull/11865#issuecomment-909410217 Thanks @dave2wave ! -- 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] merlimat merged pull request #11865: [website] Companies Powered by Pulsar

2021-08-31 Thread GitBox
merlimat merged pull request #11865: URL: https://github.com/apache/pulsar/pull/11865 -- 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: [Python] Handle py::call_method error without mutating internal state (#11840)

2021-08-31 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz 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 9153e71 [Python] Handle py::call_method error

[GitHub] [pulsar] BewareMyPower merged pull request #11840: [Python] Handle py::call_method error without mutating internal state

2021-08-31 Thread GitBox
BewareMyPower merged pull request #11840: URL: https://github.com/apache/pulsar/pull/11840 -- 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 #11823: Using the Python pulsar client with a logger can cause arbitrary/unrelated `async` Python functions to return `None`

2021-08-31 Thread GitBox
BewareMyPower closed issue #11823: URL: https://github.com/apache/pulsar/issues/11823 -- 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 #11823: Using the Python pulsar client with a logger can cause arbitrary/unrelated `async` Python functions to return `None`

2021-08-31 Thread GitBox
BewareMyPower commented on issue #11823: URL: https://github.com/apache/pulsar/issues/11823#issuecomment-909407548 > I think this means that source artifacts are not published to PyPi, which may be a separate issue. Do you think I should open one? You can open an issue or send an

[GitHub] [pulsar] dave2wave commented on a change in pull request #11864: Add basic DOAP file for Pulsar

2021-08-31 Thread GitBox
dave2wave commented on a change in pull request #11864: URL: https://github.com/apache/pulsar/pull/11864#discussion_r699501126 ## File path: site2/website/static/doap.rdf ## @@ -0,0 +1,45 @@ + + +http://usefulinc.com/ns/doap#; +

[GitHub] [pulsar] dave2wave commented on a change in pull request #11864: Add basic DOAP file for Pulsar

2021-08-31 Thread GitBox
dave2wave commented on a change in pull request #11864: URL: https://github.com/apache/pulsar/pull/11864#discussion_r699501126 ## File path: site2/website/static/doap.rdf ## @@ -0,0 +1,45 @@ + + +http://usefulinc.com/ns/doap#; +

[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-08-31 Thread GitBox
zbentley commented on issue #11823: URL: https://github.com/apache/pulsar/issues/11823#issuecomment-909399309 > How did you build the C++ library? I didn't; `pip install pulsar-client` does not by default invoke the compiler. Instead, it downloads

[GitHub] [pulsar] dave2wave opened a new pull request #11865: [website] Companies Powered by Pulsar

2021-08-31 Thread GitBox
dave2wave opened a new pull request #11865: URL: https://github.com/apache/pulsar/pull/11865 ### Motivation Improve the powered by hyperlink language on the main page ### Modifications Changed the hyperlink text. ### Verifying this change No tests

[pulsar] 01/01: [website] Companies Powered by Pulsar

2021-08-31 Thread wave
This is an automated email from the ASF dual-hosted git repository. wave pushed a commit to branch dave2wave-patch-2 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit ca315556ba17469281e0d26befe3d28eaf2de917 Author: Dave Fisher AuthorDate: Tue Aug 31 09:31:18 2021 -0700

[pulsar] branch dave2wave-patch-2 created (now ca31555)

2021-08-31 Thread wave
This is an automated email from the ASF dual-hosted git repository. wave pushed a change to branch dave2wave-patch-2 in repository https://gitbox.apache.org/repos/asf/pulsar.git. at ca31555 [website] Companies Powered by Pulsar This branch includes the following new commits: new

[GitHub] [pulsar] avirlrma commented on issue #11507: Pulsar 2.6: GET persistent/{tenant}/{namespace} with invalid namespace returns 200

2021-08-31 Thread GitBox
avirlrma commented on issue #11507: URL: https://github.com/apache/pulsar/issues/11507#issuecomment-909385351 Hi @eolivelli , Can we close this? -- 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] avirlrma edited a comment on issue #11780: producer.sendAsync is interrupted by client.close()

2021-08-31 Thread GitBox
avirlrma edited a comment on issue #11780: URL: https://github.com/apache/pulsar/issues/11780#issuecomment-909383320 Hi folks, Can I take this up? -- 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] avirlrma commented on issue #11780: producer.sendAsync is interrupted by client.close()

2021-08-31 Thread GitBox
avirlrma commented on issue #11780: URL: https://github.com/apache/pulsar/issues/11780#issuecomment-909383320 Hi folks, Can I take this up? @sijie -- 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] merlimat commented on a change in pull request #11864: Add basic DOAP file for Pulsar

2021-08-31 Thread GitBox
merlimat commented on a change in pull request #11864: URL: https://github.com/apache/pulsar/pull/11864#discussion_r699472313 ## File path: site2/website/static/doap.rdf ## @@ -0,0 +1,45 @@ + + +http://usefulinc.com/ns/doap#; +

[GitHub] [pulsar] merlimat commented on a change in pull request #11864: Add basic DOAP file for Pulsar

2021-08-31 Thread GitBox
merlimat commented on a change in pull request #11864: URL: https://github.com/apache/pulsar/pull/11864#discussion_r699447486 ## File path: site2/website/static/doap.rdf ## @@ -0,0 +1,45 @@ + + +http://usefulinc.com/ns/doap#; +

[GitHub] [pulsar] Shoothzj commented on issue #11783: ConcurrentModificationException occurs when failPendingMessages

2021-08-31 Thread GitBox
Shoothzj commented on issue #11783: URL: https://github.com/apache/pulsar/issues/11783#issuecomment-909347986 I just figure out why, because the client code retry to send messages when exception occurred to avoid network "Jitter"(Translate on google). the sample code are like this

[GitHub] [pulsar] eolivelli commented on pull request #10487: Add the Maven wrapper

2021-08-31 Thread GitBox
eolivelli commented on pull request #10487: URL: https://github.com/apache/pulsar/pull/10487#issuecomment-909343926 @sijie @codelipenghui it would be good to merge this before 2.9.0 release -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] liangyuanpeng commented on issue #11850: Java Client not allow newConsumer().receiverQueueSize(0)

2021-08-31 Thread GitBox
liangyuanpeng commented on issue #11850: URL: https://github.com/apache/pulsar/issues/11850#issuecomment-908857553 > unsupported received batch-message with zero receiver queue size You can close batch from producer. ```java pulsarClient.newProducer()

[GitHub] [pulsar] lhotari merged pull request #11818: [config] Remove unnecessary spaces from conf files

2021-08-31 Thread GitBox
lhotari merged pull request #11818: URL: https://github.com/apache/pulsar/pull/11818 -- 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] liangyuanpeng edited a comment on issue #11850: Java Client not allow newConsumer().receiverQueueSize(0)

2021-08-31 Thread GitBox
liangyuanpeng edited a comment on issue #11850: URL: https://github.com/apache/pulsar/issues/11850#issuecomment-908857553 > unsupported received batch-message with zero receiver queue size You can close batch from producer. ```java pulsarClient.newProducer()

[GitHub] [pulsar] massakam commented on a change in pull request #11839: [stats] Add Key_Shared metadata to topic stats

2021-08-31 Thread GitBox
massakam commented on a change in pull request #11839: URL: https://github.com/apache/pulsar/pull/11839#discussion_r699412556 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/stats/SubscriptionStatsImpl.java ## @@ -103,6 +103,12 @@ /**

[GitHub] [pulsar] hangc0276 commented on pull request #11862: [Branch-2.8]Fix pulsar_standalone docker image build failed

2021-08-31 Thread GitBox
hangc0276 commented on pull request #11862: URL: https://github.com/apache/pulsar/pull/11862#issuecomment-909086192 > LGTM > in 2.9.0 we dropped Standalone. > it is fine to keep it in 2.8.0 at the moment > thanks @eolivelli Yes, it just tag it into 2.8.1 -- This is an

[GitHub] [pulsar] codelipenghui merged pull request #11739: Expose compaction metrics to Prometheus

2021-08-31 Thread GitBox
codelipenghui merged pull request #11739: URL: https://github.com/apache/pulsar/pull/11739 -- 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 #11823: Using the Python pulsar client with a logger can cause arbitrary/unrelated `async` Python functions to return `None`

2021-08-31 Thread GitBox
BewareMyPower commented on issue #11823: URL: https://github.com/apache/pulsar/issues/11823#issuecomment-909320468 -- 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

[GitHub] [pulsar] graceon commented on issue #11850: Java Client not allow newConsumer().receiverQueueSize(0)

2021-08-31 Thread GitBox
graceon commented on issue #11850: URL: https://github.com/apache/pulsar/issues/11850#issuecomment-908926532 it works 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 to go to the specific

[GitHub] [pulsar] lhotari commented on issue #11628: Flaky-test: ElasticSearchClientTests.testBulkRetry

2021-08-31 Thread GitBox
lhotari commented on issue #11628: URL: https://github.com/apache/pulsar/issues/11628#issuecomment-908627170 ElasticSearchClientTests.testBulkRetry is failing frequently. @eolivelli @vroyer any chance to reduce the flakiness? [recent

[GitHub] [pulsar] MathiasHaudgaard commented on pull request #11794: jwks feature for oAuth flow

2021-08-31 Thread GitBox
MathiasHaudgaard commented on pull request #11794: URL: https://github.com/apache/pulsar/pull/11794#issuecomment-909041052 > What about creating a separate Maven Module and use the Maven shade plugin to include all the third party deps in the .jar file for this Provider ? > This way we

[GitHub] [pulsar] eolivelli commented on pull request #11794: jwks feature for oAuth flow

2021-08-31 Thread GitBox
eolivelli commented on pull request #11794: URL: https://github.com/apache/pulsar/pull/11794#issuecomment-908995352 What about creating a separate Maven Module and use the Maven shade plugin to include all the third party deps in the .jar file for this Provider ? This way we are not

[GitHub] [pulsar] freeznet commented on pull request #11853: [pulsar-functions-go] sync to the latest function proto

2021-08-31 Thread GitBox
freeznet commented on pull request #11853: URL: https://github.com/apache/pulsar/pull/11853#issuecomment-908860516 /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

[GitHub] [pulsar] lhotari merged pull request #11820: [Functions] ConcurrentHashMap should be used for caching producers

2021-08-31 Thread GitBox
lhotari merged pull request #11820: URL: https://github.com/apache/pulsar/pull/11820 -- 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 #11836: after split bundle,what happened next

2021-08-31 Thread GitBox
lhotari closed issue #11836: URL: https://github.com/apache/pulsar/issues/11836 -- 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 #11844: [Website] Remove Company Logos From Main Page

2021-08-31 Thread GitBox
merlimat commented on a change in pull request #11844: URL: https://github.com/apache/pulsar/pull/11844#discussion_r698799705 ## File path: site2/website/pages/en/index.js ## @@ -168,30 +168,7 @@ const UsersBlock = props => ( id={props.id}

[GitHub] [pulsar] kaushik-develop commented on pull request #11821: [PIP-82] [pulsar-broker] Misc fixes:

2021-08-31 Thread GitBox
kaushik-develop commented on pull request #11821: URL: https://github.com/apache/pulsar/pull/11821#issuecomment-908680649 /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

[GitHub] [pulsar] yuruguo edited a comment on pull request #11797: [testclient] Fix IndexOutOfBoundsException when --subscriptions size is less than --num-subscriptions in consumption

2021-08-31 Thread GitBox
yuruguo edited a comment on pull request #11797: URL: https://github.com/apache/pulsar/pull/11797#issuecomment-906799519 -- 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

[GitHub] [pulsar] freeznet commented on a change in pull request #11831: [pulsar-functions] Pass `SubscriptionPosition` from `FunctionDetails` to `FunctionConfig` / `SinkConfig`

2021-08-31 Thread GitBox
freeznet commented on a change in pull request #11831: URL: https://github.com/apache/pulsar/pull/11831#discussion_r698935643 ## File path: pulsar-functions/utils/src/main/java/org/apache/pulsar/functions/utils/FunctionConfigUtils.java ## @@ -373,6 +373,14 @@ public static

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

2021-08-31 Thread GitBox
Technoboy- commented on pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#issuecomment-908821889 -- 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

[GitHub] [pulsar] gaoran10 commented on a change in pull request #11856: [Schema] Schema compatibility strategy in broker level.

2021-08-31 Thread GitBox
gaoran10 commented on a change in pull request #11856: URL: https://github.com/apache/pulsar/pull/11856#discussion_r698959315 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java ## @@ -1955,6 +1957,13 @@

[GitHub] [pulsar] congbobo184 commented on a change in pull request #11856: [Schema] Schema compatibility strategy in broker level.

2021-08-31 Thread GitBox
congbobo184 commented on a change in pull request #11856: URL: https://github.com/apache/pulsar/pull/11856#discussion_r698966658 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java ## @@ -1955,6 +1957,13 @@

[GitHub] [pulsar] eolivelli commented on pull request #11859: [Branch-2.8]Fix jetty-alpn-java-client license

2021-08-31 Thread GitBox
eolivelli commented on pull request #11859: URL: https://github.com/apache/pulsar/pull/11859#issuecomment-909055195 -- 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

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

2021-08-31 Thread GitBox
zhanghaou commented on a change in pull request #11852: URL: https://github.com/apache/pulsar/pull/11852#discussion_r699006800 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -2300,6 +2300,12 @@ protected void

[GitHub] [pulsar] michaeljmarshall commented on pull request #11820: [Functions] ConcurrentHashMap should be used for caching producers

2021-08-31 Thread GitBox
michaeljmarshall commented on pull request #11820: URL: https://github.com/apache/pulsar/pull/11820#issuecomment-908672692 @lhotari - I took a look, and I think the current design is intentional to allow for the `PulsarClientException` to be thrown without special handling in the

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #11821: [PIP-82] [pulsar-broker] Misc fixes:

2021-08-31 Thread GitBox
kaushik-develop commented on a change in pull request #11821: URL: https://github.com/apache/pulsar/pull/11821#discussion_r698653201 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroup.java ## @@ -321,6 +321,12 @@ protected

[GitHub] [pulsar] Anonymitaet commented on pull request #11828: [testclient] deprecate option --subscriber-name and substitute --subscriptions first element for it

2021-08-31 Thread GitBox
Anonymitaet commented on pull request #11828: URL: https://github.com/apache/pulsar/pull/11828#issuecomment-908823973 Thanks for your contribution. Does this affect only master or other versioned docs? If latter, could you please help update all affected versions? Thanks -- This is

[GitHub] [pulsar] urfreespace commented on pull request #11851: [website][upgrade]feat: docs migration about Concepts and Architecture (versions: next/2.8.0)

2021-08-31 Thread GitBox
urfreespace commented on pull request #11851: URL: https://github.com/apache/pulsar/pull/11851#issuecomment-908849872 @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 to

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

2021-08-31 Thread GitBox
gaoran10 commented on a change in pull request #11852: URL: https://github.com/apache/pulsar/pull/11852#discussion_r698966718 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -2300,6 +2300,12 @@ protected void

[GitHub] [pulsar] MathiasHaudgaard commented on a change in pull request #11794: jwks feature for oAuth flow

2021-08-31 Thread GitBox
MathiasHaudgaard commented on a change in pull request #11794: URL: https://github.com/apache/pulsar/pull/11794#discussion_r698503213 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationProviderToken.java ## @@ -237,6 +280,69

[GitHub] [pulsar] eolivelli merged pull request #11844: [Website] Remove Company Logos From Main Page

2021-08-31 Thread GitBox
eolivelli merged pull request #11844: URL: https://github.com/apache/pulsar/pull/11844 -- 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] beyondyinjl2 commented on issue #11857: java.lang.NoSuchMethodError: org.apache.pulsar.broker.ServiceConfigurationUtils.getAppliedAdvertisedAddress(Lorg/apache/pulsar/broker/ServiceC

2021-08-31 Thread GitBox
beyondyinjl2 commented on issue #11857: URL: https://github.com/apache/pulsar/issues/11857#issuecomment-908947448 Thank you very much, 2.8.0.8 can be used -- 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] Anonymitaet commented on pull request #11839: [stats] Add Key_Shared metadata to topic stats

2021-08-31 Thread GitBox
Anonymitaet commented on pull request #11839: URL: https://github.com/apache/pulsar/pull/11839#issuecomment-908823557 @massakam thanks for adding docs! -- 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] hangc0276 closed issue #10917: Cannot build Pulsar from Source Distribution Bundle (2.8.0rc3)

2021-08-31 Thread GitBox
hangc0276 closed issue #10917: URL: https://github.com/apache/pulsar/issues/10917 -- 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] hangc0276 merged pull request #11862: [Branch-2.8]Fix pulsar_standalone docker image build failed

2021-08-31 Thread GitBox
hangc0276 merged pull request #11862: URL: https://github.com/apache/pulsar/pull/11862 -- 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 #11856: [Schema] Schema compatibility strategy in broker level.

2021-08-31 Thread GitBox
merlimat commented on a change in pull request #11856: URL: https://github.com/apache/pulsar/pull/11856#discussion_r698987994 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java ## @@ -1955,6 +1957,13 @@

[GitHub] [pulsar] ivankelly merged pull request #11841: [admin] Allow http service URIs with path

2021-08-31 Thread GitBox
ivankelly merged pull request #11841: URL: https://github.com/apache/pulsar/pull/11841 -- 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 #10437: pulsar java reactor based client support

2021-08-31 Thread GitBox
lhotari commented on issue #10437: URL: https://github.com/apache/pulsar/issues/10437#issuecomment-909270873 > @lhotari > Thank a lot for the invite. Do you have smth production ready? And also is there any issues that I can help with& =) > I implemented reactive facade in out

[GitHub] [pulsar] merlimat merged pull request #11854: [Website] fix incorrect logo URLs

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

  1   2   >