[pulsar] branch master updated: remove pulsar-standalone image (#11657)

2021-08-12 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 05827ae remove pulsar-standalone image (#11657)

[GitHub] [pulsar] sijie merged pull request #11657: remove pulsar-standalone image

2021-08-12 Thread GitBox
sijie merged pull request #11657: URL: https://github.com/apache/pulsar/pull/11657 -- 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] Technoboy- commented on pull request #11650: [Broker Interceptor] Fix Pulsar didn't respond error messages when throw InterceptException

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

[pulsar] branch master updated (0748ecb -> e36233e)

2021-08-12 Thread hjf
This is an automated email from the ASF dual-hosted git repository. hjf pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 0748ecb [C++] Fixing use-after-free and constructor bugs in UnAckedMessageTrackerEnabled (#11630) add e36233e

[GitHub] [pulsar] Jennifer88huang merged pull request #11658: [docs] Remove broken links

2021-08-12 Thread GitBox
Jennifer88huang merged pull request #11658: URL: https://github.com/apache/pulsar/pull/11658 -- 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] Technoboy- commented on pull request #11655: Check null or empty instead of catch NPE

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

[GitHub] [pulsar] Jennifer88huang opened a new pull request #11658: [docs] Remove broken links

2021-08-12 Thread GitBox
Jennifer88huang opened a new pull request #11658: URL: https://github.com/apache/pulsar/pull/11658 ### Modifications Remove the broken links and link to the third part website. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] freeznet commented on pull request #11623: Fix unnecessary user interactions when building pulsar-standalone image

2021-08-12 Thread GitBox
freeznet commented on pull request #11623: URL: https://github.com/apache/pulsar/pull/11623#issuecomment-898138663 @eolivelli I just created a PR https://github.com/apache/pulsar/pull/11657 to remove the pulsar-standalone image, PTAL when you have time, thanks. -- This is an automated

[GitHub] [pulsar] freeznet opened a new pull request #11657: remove pulsar-standalone image

2021-08-12 Thread GitBox
freeznet opened a new pull request #11657: URL: https://github.com/apache/pulsar/pull/11657 ### Motivation Currently, the pulsar-dashboard has been removed in https://github.com/apache/pulsar/pull/11284, as the pulsar-standalone image is required pulsar-dashboard and provide

[GitHub] [pulsar] Technoboy- opened a new pull request #11656: Use getIfValid to avoid handleMetadataStoreNotification swallow exception

2021-08-12 Thread GitBox
Technoboy- opened a new pull request #11656: URL: https://github.com/apache/pulsar/pull/11656 ### Motivation getNamespaceFromPoliciesPath(n.getPath()) may return empty, and result NamespaceName.get throw IllegalArgumentException ### Documentation This is code-optimized, no

[GitHub] [pulsar] gaoran10 commented on issue #11576: Avro or KeyValue Schema throws exception in PulsarRecordCursor

2021-08-12 Thread GitBox
gaoran10 commented on issue #11576: URL: https://github.com/apache/pulsar/issues/11576#issuecomment-898109632 Maybe you could refer to Java client logic, the producer will register schema before producing messages to make sure the schema saved in BookKeeper, the register schema command

[GitHub] [pulsar] Anonymitaet commented on pull request #11648: [Doc] [ISSUE 11291] Update Pulsar Dashboard relevant content

2021-08-12 Thread GitBox
Anonymitaet commented on pull request #11648: URL: https://github.com/apache/pulsar/pull/11648#issuecomment-898101919 @sijia-w @Jennifer88huang how about https://opensourcelibs.com/lib/pulsar-java-tutorial? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] Technoboy- opened a new pull request #11655: Check null or empty instead of catch NPE

2021-08-12 Thread GitBox
Technoboy- opened a new pull request #11655: URL: https://github.com/apache/pulsar/pull/11655 ### Motivation - Check variable first instead of catch NPE. ### Documentation This is code-optimized, no need to update the doc. -- This is an automated message from the

[GitHub] [pulsar] Sunny-Island commented on issue #11561: Request for Timeout of hasMessageAvailable API in Pulsar CPP Reader

2021-08-12 Thread GitBox
Sunny-Island commented on issue #11561: URL: https://github.com/apache/pulsar/issues/11561#issuecomment-898087304 This issue can be closed. https://github.com/apache/pulsar/pull/11600 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [pulsar] Technoboy- commented on pull request #11650: [Broker Interceptor] Fix Pulsar didn't respond error messages when throw InterceptException

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

[GitHub] [pulsar] jerrypeng commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
jerrypeng commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r688085063 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -243,7 +245,7 @@ protected

[GitHub] [pulsar] jerrypeng commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
jerrypeng commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r688085063 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -243,7 +245,7 @@ protected

[GitHub] [pulsar] jerrypeng commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
jerrypeng commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r688085063 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -243,7 +245,7 @@ protected

[GitHub] [pulsar] jerrypeng commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
jerrypeng commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r688081880 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -883,16 +887,21 @@ private void

[GitHub] [pulsar] jerrypeng commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
jerrypeng commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r688077951 ## File path: pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java ## @@ -83,6 +86,49 @@ public

[GitHub] [pulsar] jerrypeng commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
jerrypeng commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r688076935 ## File path: pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java ## @@ -83,6 +86,49 @@ public

[GitHub] [pulsar] jerrypeng commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
jerrypeng commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r688064193 ## File path: pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java ## @@ -83,6 +86,49 @@ public

[GitHub] [pulsar] acjohnson opened a new issue #11654: [Functions worker] Topic has an existing exclusive producer - self-deadlock issue on Pulsar 2.8.0

2021-08-12 Thread GitBox
acjohnson opened a new issue #11654: URL: https://github.com/apache/pulsar/issues/11654 **Describe the bug** After upgrading to Official Apache Pulsar 2.8.0 docker image we are now seeing a flood of messages like this: ``` {"line":"22:48:42.086 [pulsar-client-io-51-1] ERROR

[GitHub] [pulsar] sijia-w commented on issue #11380: CLI in TTL config is not clear

2021-08-12 Thread GitBox
sijia-w commented on issue #11380: URL: https://github.com/apache/pulsar/issues/11380#issuecomment-897947070 @Jennifer88huang I checked this page https://pulsar.apache.org/docs/en/cookbooks-retention-expiry/#clear-backlog and the `N/A`s seem to be empty returned messages. If they are not,

[GitHub] [pulsar] sijia-w commented on pull request #11653: [docs] [ISSUE 11380] Update code example and description

2021-08-12 Thread GitBox
sijia-w commented on pull request #11653: URL: https://github.com/apache/pulsar/pull/11653#issuecomment-897946101 @Jennifer88huang Please help review the changes. Thank you. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[pulsar] branch master updated (bf62bd9 -> 0748ecb)

2021-08-12 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 bf62bd9 [PIP-82] [pulsar-broker] Use the qualified NS-name in RG-register/unregisters (#11643) add 0748ecb

[GitHub] [pulsar] sijia-w opened a new pull request #11653: [docs] [ISSUE 11380] Update code example and description

2021-08-12 Thread GitBox
sijia-w opened a new pull request #11653: URL: https://github.com/apache/pulsar/pull/11653 Fixes #11380 It seems that the changed two file share common content on the usage of message retention policies. Do we need to keep the examples of the commands in both files? -- This is

[GitHub] [pulsar] merlimat merged pull request #11630: [C++] Fixing use-after-free and constructor bugs in UnAckedMessageTrackerEnabled

2021-08-12 Thread GitBox
merlimat merged pull request #11630: URL: https://github.com/apache/pulsar/pull/11630 -- 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 (683c2d6a -> bf62bd9)

2021-08-12 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 683c2d6a Java Client: remove usage of reflection while using Pulsar Implementation classes (#11636) add

[GitHub] [pulsar] jerrypeng merged pull request #11643: [PIP-82] [pulsar-broker] Use the qualified NS-name in RG-register/unr…

2021-08-12 Thread GitBox
jerrypeng merged pull request #11643: URL: https://github.com/apache/pulsar/pull/11643 -- 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] oversearch commented on pull request #11630: [C++] Fixing use-after-free and constructor bugs in UnAckedMessageTrackerEnabled

2021-08-12 Thread GitBox
oversearch commented on pull request #11630: URL: https://github.com/apache/pulsar/pull/11630#issuecomment-897906422 I wasn't able to reproduce the stack dump in the tests in my local environment. Everything passes except for an obscure error about a bytes/string type mismatch in the

[GitHub] [pulsar] ivankelly commented on pull request #11570: [Issue 11496][C++] Allow partitioned producers to start lazily

2021-08-12 Thread GitBox
ivankelly commented on pull request #11570: URL: https://github.com/apache/pulsar/pull/11570#issuecomment-897892377 > For authn/authz backward compatibility reason, I think the internal producer should be connected to broker before completing the creation of partitioned producer(

[GitHub] [pulsar] kaushik-develop commented on pull request #11643: [PIP-82] [pulsar-broker] Use the qualified NS-name in RG-register/unr…

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

[pulsar] branch master updated (eabf52f -> 683c2d6a)

2021-08-12 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 eabf52f [Blog] Update Pulsar summit link (#11651) add 683c2d6a Java Client: remove usage of reflection while

[GitHub] [pulsar] merlimat merged pull request #11636: Java Client: remove usage of reflection while using Pulsar Implementation classes

2021-08-12 Thread GitBox
merlimat merged pull request #11636: URL: https://github.com/apache/pulsar/pull/11636 -- 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 #11636: Java Client: remove usage of reflection while using Pulsar Implementation classes

2021-08-12 Thread GitBox
merlimat commented on a change in pull request #11636: URL: https://github.com/apache/pulsar/pull/11636#discussion_r687966915 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarClientImplementationBindingImpl.java ## @@ -0,0 +1,348 @@ +/** + *

[GitHub] [pulsar] eaba closed issue #11576: Avro or KeyValue Schema throws exception in PulsarRecordCursor

2021-08-12 Thread GitBox
eaba closed issue #11576: URL: https://github.com/apache/pulsar/issues/11576 -- 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] eaba commented on issue #11576: Avro or KeyValue Schema throws exception in PulsarRecordCursor

2021-08-12 Thread GitBox
eaba commented on issue #11576: URL: https://github.com/apache/pulsar/issues/11576#issuecomment-897831708 @gaoran10 thanks. Null schemaversion was the problem. I have it fixed now. Thanks a lot!! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] oversearch commented on pull request #11630: [C++] Fixing use-after-free and constructor bugs in UnAckedMessageTrackerEnabled

2021-08-12 Thread GitBox
oversearch commented on pull request #11630: URL: https://github.com/apache/pulsar/pull/11630#issuecomment-897821212 Ah yes, sorry about leaving out the docs section. I had assumed leaving it out would imply "no" but I'll make that explicit. I'm also looking into the python test

[GitHub] [pulsar] ivankelly commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
ivankelly commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r687941161 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java ## @@ -327,6 +333,14 @@ public boolean

[GitHub] [pulsar] oversearch commented on issue #11632: [C++] pulsar-client-cpp stability enhancements and code cleanup

2021-08-12 Thread GitBox
oversearch commented on issue #11632: URL: https://github.com/apache/pulsar/issues/11632#issuecomment-897818966 Thanks for the feedback guys. I'll proceed with the changes as time allows, starting with the improved warnings. With respect to the boost compatibility stuff: you're

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #11643: [PIP-82] [pulsar-broker] Use the qualified NS-name in RG-register/unr…

2021-08-12 Thread GitBox
kaushik-develop commented on a change in pull request #11643: URL: https://github.com/apache/pulsar/pull/11643#discussion_r687935502 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceQuotaCalculatorImpl.java ## @@ -60,7 +68,11 @@ public

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #11643: [PIP-82] [pulsar-broker] Use the qualified NS-name in RG-register/unr…

2021-08-12 Thread GitBox
kaushik-develop commented on a change in pull request #11643: URL: https://github.com/apache/pulsar/pull/11643#discussion_r687935424 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java ## @@ -233,66 +235,80 @@ public void

[GitHub] [pulsar] eolivelli commented on issue #11070: Zookeeper in 3 node Kubernetes cluster does not pass heath check in 2.8.0

2021-08-12 Thread GitBox
eolivelli commented on issue #11070: URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897759754 I have run my tests and with this helm chart ZooKeeper works well with 3 pods https://github.com/datastax/pulsar-helm-chart -- This is an automated message from the Apache

[GitHub] [pulsar] LeBW commented on a change in pull request #11553: Exposing the broker entry metadata to client

2021-08-12 Thread GitBox
LeBW commented on a change in pull request #11553: URL: https://github.com/apache/pulsar/pull/11553#discussion_r687870069 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/PulsarCommandSenderImpl.java ## @@ -243,8 +243,12 @@ public ChannelPromise

[GitHub] [pulsar] srouthu1 opened a new issue #11652: deadLetterPolicy is not working with key shared subscription

2021-08-12 Thread GitBox
srouthu1 opened a new issue #11652: URL: https://github.com/apache/pulsar/issues/11652 Expected behavior I have created consumer with the following code. Consumer consumer = client.newConsumer() .topic("persistent://mytenant/myns/mytopic")

[GitHub] [pulsar] eolivelli commented on issue #11070: Zookeeper in 3 node Kubernetes cluster does not pass heath check in 2.8.0

2021-08-12 Thread GitBox
eolivelli commented on issue #11070: URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897695832 @andrekramer1 this line seems relevant for upgrading ZK and Pulsar 2.8

[GitHub] [pulsar] eolivelli commented on issue #11070: Zookeeper in 3 node Kubernetes cluster does not pass heath check in 2.8.0

2021-08-12 Thread GitBox
eolivelli commented on issue #11070: URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897690248 I am testing it now with my helm chartstay tuned -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [pulsar] eolivelli commented on issue #11070: Zookeeper in 3 node Kubernetes cluster does not pass heath check in 2.8.0

2021-08-12 Thread GitBox
eolivelli commented on issue #11070: URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897683866 which helm chart are you using ? the "official" Apache Pulsar Helm chart was never updated to Pulsar 2.8.0 see this PR that has not been merged

[GitHub] [pulsar] hangc0276 commented on issue #11070: Zookeeper in 3 node Kubernetes cluster does not pass heath check in 2.8.0

2021-08-12 Thread GitBox
hangc0276 commented on issue #11070: URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897676723 Thanks for @eolivelli 's help, from the exception stack, i looks like something wrong in zookeeper or the helm chart. Would you please paste detail logs of zookeeper?

[pulsar] branch master updated (36d5738 -> eabf52f)

2021-08-12 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 36d5738 Add Docs for the new ElasticSearch sink features (#11602) add eabf52f [Blog] Update Pulsar summit

[GitHub] [pulsar] eolivelli merged pull request #11651: [Blog] Update Pulsar summit link

2021-08-12 Thread GitBox
eolivelli merged pull request #11651: URL: https://github.com/apache/pulsar/pull/11651 -- 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 issue #11070: Zookeeper in 3 node Kubernetes cluster does not pass heath check in 2.8.0

2021-08-12 Thread GitBox
eolivelli commented on issue #11070: URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897664698 the NPE is here

[GitHub] [pulsar] Technoboy- commented on a change in pull request #11650: [Broker Interceptor] Fix Pulsar didn't respond error messages when throw InterceptException

2021-08-12 Thread GitBox
Technoboy- commented on a change in pull request #11650: URL: https://github.com/apache/pulsar/pull/11650#discussion_r687717002 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/web/ExceptionHandlerTest.java ## @@ -0,0 +1,55 @@ +/** + * Licensed to the Apache

[GitHub] [pulsar] Technoboy- commented on a change in pull request #11650: [Broker Interceptor] Fix Pulsar didn't respond error messages when throw InterceptException

2021-08-12 Thread GitBox
Technoboy- commented on a change in pull request #11650: URL: https://github.com/apache/pulsar/pull/11650#discussion_r687715617 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/web/WebService.java ## @@ -159,11 +159,12 @@ public void addServlet(String path,

[GitHub] [pulsar] Technoboy- commented on a change in pull request #11650: [Broker Interceptor] Fix Pulsar didn't respond error messages when throw InterceptException

2021-08-12 Thread GitBox
Technoboy- commented on a change in pull request #11650: URL: https://github.com/apache/pulsar/pull/11650#discussion_r687715269 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/web/WebService.java ## @@ -159,11 +159,12 @@ public void addServlet(String path,

[GitHub] [pulsar] eolivelli commented on a change in pull request #11650: Fix [Broker Interceptor] Pulsar didn't respond error messages when throw InterceptException

2021-08-12 Thread GitBox
eolivelli commented on a change in pull request #11650: URL: https://github.com/apache/pulsar/pull/11650#discussion_r687710188 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/web/WebService.java ## @@ -159,11 +159,12 @@ public void addServlet(String path,

[GitHub] [pulsar] Technoboy- opened a new pull request #11650: Fix [Broker Interceptor] Pulsar didn't respond error messages when throw InterceptException

2021-08-12 Thread GitBox
Technoboy- opened a new pull request #11650: URL: https://github.com/apache/pulsar/pull/11650 Fixes #11649 ### Documentation This is a bug fix, and do not need to update doc. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] Technoboy- opened a new issue #11649: [Broker Interceptor] Pulsar didn't respond error messages when throw InterceptException

2021-08-12 Thread GitBox
Technoboy- opened a new issue #11649: URL: https://github.com/apache/pulsar/issues/11649 **Describe the bug** Cluster didn't respond error messages when hitting the resource limit **To Reproduce** Set the limit of number of tenants to 4, create a new tenant when there're

[GitHub] [pulsar] HistoryGift commented on issue #5099: "ConnectException: Connection refused: /127.0.0.1:49695" when running python example function

2021-08-12 Thread GitBox
HistoryGift commented on issue #5099: URL: https://github.com/apache/pulsar/issues/5099#issuecomment-897591799 I have meet the same problem with python script example, but it's ok with java script someone say it's because of downloadDIrectory in functions_worker.yaml has no execute

[GitHub] [pulsar] 315157973 commented on a change in pull request #11553: Exposing the broker entry metadata to client

2021-08-12 Thread GitBox
315157973 commented on a change in pull request #11553: URL: https://github.com/apache/pulsar/pull/11553#discussion_r687646847 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/PulsarCommandSenderImpl.java ## @@ -243,8 +243,12 @@ public ChannelPromise

[GitHub] [pulsar] sijia-w commented on issue #11291: [Doc] Remove Pulsar Dashboard contents

2021-08-12 Thread GitBox
sijia-w commented on issue #11291: URL: https://github.com/apache/pulsar/issues/11291#issuecomment-897583889 @Jennifer88huang @Anonymitaet Thanks for your feedback. Please help review the pr linked. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] sijia-w commented on pull request #11648: [Doc] [ISSUE 11291] Update Pulsar Dashboard relevant content

2021-08-12 Thread GitBox
sijia-w commented on pull request #11648: URL: https://github.com/apache/pulsar/pull/11648#issuecomment-897581541 @Anonymitaet @Jennifer88huang Please review the changes and I look forward to your recommendation for the links. Thanks. -- This is an automated message from the Apache Git

[GitHub] [pulsar] sijia-w opened a new pull request #11648: [Doc] [ISSUE 11291] Update Pulsar Dashboard relevant content

2021-08-12 Thread GitBox
sijia-w opened a new pull request #11648: URL: https://github.com/apache/pulsar/pull/11648 Fixes #11291 The links for `[Pulsar Java tutorial](https://github.com/streamlio/pulsar-java-tutorial)` and relevant content are broken. Is there any recommendations for what we should use

[GitHub] [pulsar] eolivelli commented on pull request #10485: Update command descriptions from old 'property/cluster/namespace' format to current 'tenant/namespace' format

2021-08-12 Thread GitBox
eolivelli commented on pull request #10485: URL: https://github.com/apache/pulsar/pull/10485#issuecomment-897568305 @merlimat precommit check passed -- 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

[GitHub] [pulsar] eolivelli commented on issue #11070: Zookeeper in 3 node Kubernetes cluster does not pass heath check in 2.8.0

2021-08-12 Thread GitBox
eolivelli commented on issue #11070: URL: https://github.com/apache/pulsar/issues/11070#issuecomment-897567591 @hangc0276 I believe that we should fix this problem before cutting 2.8.1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [pulsar] codelipenghui commented on pull request #11553: Exposing the broker entry metadata to client

2021-08-12 Thread GitBox
codelipenghui commented on pull request #11553: URL: https://github.com/apache/pulsar/pull/11553#issuecomment-897539348 @eolivelli Please help review the PR again. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [pulsar] abhilashmandaliya opened a new issue #11647: Inconsistency in backlog quota configuration

2021-08-12 Thread GitBox
abhilashmandaliya opened a new issue #11647: URL: https://github.com/apache/pulsar/issues/11647 **Describe the bug** A backlog quota in the broker is configured differently from the broker's configuration file than the admin CLI. A configuration file accepts the limit in GB only as

[GitHub] [pulsar] wuzhanpeng commented on pull request #11640: [Enhancement] avoid duplicate deletion of schema

2021-08-12 Thread GitBox
wuzhanpeng commented on pull request #11640: URL: https://github.com/apache/pulsar/pull/11640#issuecomment-897528300 > Good work > Can you add a test case? Sorry for the late reply! I will add a test later. -- This is an automated message from the Apache Git Service. To respond

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

2021-08-12 Thread GitBox
BewareMyPower commented on issue #11635: URL: https://github.com/apache/pulsar/issues/11635#issuecomment-897527855 I also found this problem recently. It's flaky but the cause is hard to know currently. Maybe a temporary solution is adding the retry logic for `python pulsar_test.py`? I'm

[pulsar] branch master updated: Add Docs for the new ElasticSearch sink features (#11602)

2021-08-12 Thread eolivelli
This is an automated email from the ASF dual-hosted git repository. eolivelli 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 36d5738 Add Docs for the new ElasticSearch

[GitHub] [pulsar] eolivelli commented on pull request #11602: Add Docs for the new ElasticSearch sink features

2021-08-12 Thread GitBox
eolivelli commented on pull request #11602: URL: https://github.com/apache/pulsar/pull/11602#issuecomment-897496847 thank you @Anonymitaet ! -- 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] eolivelli merged pull request #11602: Add Docs for the new ElasticSearch sink features

2021-08-12 Thread GitBox
eolivelli merged pull request #11602: URL: https://github.com/apache/pulsar/pull/11602 -- 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] naveenkusakula commented on issue #11646: [Pulsar Java client] Not able to load "deadLetterPolicy" using loadConf(Map config) in ConsumerBuilder.

2021-08-12 Thread GitBox
naveenkusakula commented on issue #11646: URL: https://github.com/apache/pulsar/issues/11646#issuecomment-897492852 I loaded deadLetterPolicy() using normal builder and than loadConfi() without deadLetterPolicy still not able to load configuration like I mentioned in option 2. -- This

[GitHub] [pulsar] eolivelli commented on issue #11646: [Pulsar Java client] Not able to load "deadLetterPolicy" using loadConf(Map config) in ConsumerBuilder.

2021-08-12 Thread GitBox
eolivelli commented on issue #11646: URL: https://github.com/apache/pulsar/issues/11646#issuecomment-897490867 Unfortunately there are some other configuration options that are not configurable via loadConf, for instance Authentication in the PulsarClientBuilder. I would like to

[GitHub] [pulsar] eolivelli commented on pull request #11636: Java Client: remove usage of reflection while using Pulsar Implementation classes

2021-08-12 Thread GitBox
eolivelli commented on pull request #11636: URL: https://github.com/apache/pulsar/pull/11636#issuecomment-897475256 @merlimat I have removed the usage of Fully qualified class names. I have answered to your question about classloading (I had to verify this in the specs). After

[GitHub] [pulsar] eolivelli commented on a change in pull request #11636: Java Client: remove usage of reflection while using Pulsar Implementation classes

2021-08-12 Thread GitBox
eolivelli commented on a change in pull request #11636: URL: https://github.com/apache/pulsar/pull/11636#discussion_r687529763 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarClientImplementationBindingImpl.java ## @@ -0,0 +1,348 @@ +/** + *

[GitHub] [pulsar] BewareMyPower closed issue #11549: The CI of cpp tests is broken

2021-08-12 Thread GitBox
BewareMyPower closed issue #11549: URL: https://github.com/apache/pulsar/issues/11549 -- 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 #11549: The CI of cpp tests is broken

2021-08-12 Thread GitBox
BewareMyPower commented on issue #11549: URL: https://github.com/apache/pulsar/issues/11549#issuecomment-897472062 It's fixed by #11575 -- 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] BewareMyPower commented on issue #11632: [C++] pulsar-client-cpp stability enhancements and code cleanup

2021-08-12 Thread GitBox
BewareMyPower commented on issue #11632: URL: https://github.com/apache/pulsar/issues/11632#issuecomment-897470434 LGTM, here're my comments. > - Enable many more warnings (-Wall, -Wvla, -Wformat-security, etc) > - Enforce warnings as errors to prevent changes being submitted with

[GitHub] [pulsar] equanz commented on pull request #11570: [Issue 11496][C++] Allow partitioned producers to start lazily

2021-08-12 Thread GitBox
equanz commented on pull request #11570: URL: https://github.com/apache/pulsar/pull/11570#issuecomment-897463605 Thank you for your comments about PIP-79. I have a few comments. I think these comments don't block merging. For authn/authz backward compatibility reason, I think the

[GitHub] [pulsar] eolivelli commented on pull request #11636: Java Client: remove usage of reflection while using Pulsar Implementation classes

2021-08-12 Thread GitBox
eolivelli commented on pull request #11636: URL: https://github.com/apache/pulsar/pull/11636#issuecomment-897456337 > @eolivelli thanks for providing doc-related info. If your PR does not need to update doc, could you please help label the PR with `no-need-doc` in the future? Thanks

[GitHub] [pulsar] eolivelli commented on a change in pull request #11636: Java Client: remove usage of reflection while using Pulsar Implementation classes

2021-08-12 Thread GitBox
eolivelli commented on a change in pull request #11636: URL: https://github.com/apache/pulsar/pull/11636#discussion_r687509897 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarClientImplementationBindingImpl.java ## @@ -0,0 +1,348 @@ +/** + *

[GitHub] [pulsar] naveenkusakula opened a new issue #11646: [Pulsar Java client] Not able to load "deadLetterPolicy" using loadConf(Map config) in ConsumerBuilder.

2021-08-12 Thread GitBox
naveenkusakula opened a new issue #11646: URL: https://github.com/apache/pulsar/issues/11646 Not able to load "deadLetterPolicy" when using loadConf(Map config) and not able to loadConf(Map config), after deadLetterPolicy() is called in ConsumerBuilder. **Steps to Reporoduce:**

[pulsar] branch master updated (9088294 -> 402a88d)

2021-08-12 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 9088294 Sync up C error code with C++ (#11633) add 402a88d Fix flaky tests for ElasticSearch Sink (#11638)

[GitHub] [pulsar] eolivelli merged pull request #11638: [Issue #11628][ES-Sink] Fix flaky tests

2021-08-12 Thread GitBox
eolivelli merged pull request #11638: URL: https://github.com/apache/pulsar/pull/11638 -- 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 closed issue #11628: Flaky-test: ElasticSearchClientTests.testBulkRetry

2021-08-12 Thread GitBox
eolivelli closed issue #11628: URL: https://github.com/apache/pulsar/issues/11628 -- 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] ivankelly commented on pull request #11627: PIP-91: Separate lookup timeout from operation timeout

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

[GitHub] [pulsar] ivankelly commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
ivankelly commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r687477622 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -883,16 +887,21 @@ private void

[GitHub] [pulsar] ivankelly commented on a change in pull request #11627: PIP-91: Separate lookup timeout from operation timeout

2021-08-12 Thread GitBox
ivankelly commented on a change in pull request #11627: URL: https://github.com/apache/pulsar/pull/11627#discussion_r687476062 ## File path: pulsar-client-api/src/main/java/org/apache/pulsar/client/api/PulsarClientException.java ## @@ -83,6 +86,49 @@ public

[GitHub] [pulsar] gaoran10 edited a comment on issue #11576: Avro or KeyValue Schema throws exception in PulsarRecordCursor

2021-08-12 Thread GitBox
gaoran10 edited a comment on issue #11576: URL: https://github.com/apache/pulsar/issues/11576#issuecomment-897407506 I think the root cause is the message metadata is the lake of `schemaVersion` property. You could change the message serialization method to add schema version in message

[pulsar] branch branch-2.8 updated: fix flaky test in AdminApiOffloadTest (#11028)

2021-08-12 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.8 by this push: new dd796d8 fix flaky test in

[pulsar] branch branch-2.8 updated: [Branch-2.8]fix license mismatch (#11645)

2021-08-12 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.8 by this push: new f918370 [Branch-2.8]fix license

[GitHub] [pulsar] hangc0276 merged pull request #11645: [Branch-2.8]fix license mismatch

2021-08-12 Thread GitBox
hangc0276 merged pull request #11645: URL: https://github.com/apache/pulsar/pull/11645 -- 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 opened a new pull request #11645: [Branch-2.8]fix license mismatch

2021-08-12 Thread GitBox
hangc0276 opened a new pull request #11645: URL: https://github.com/apache/pulsar/pull/11645 Related to #11644 -- 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] LeBW commented on a change in pull request #11553: Exposing the broker entry metadata to client

2021-08-12 Thread GitBox
LeBW commented on a change in pull request #11553: URL: https://github.com/apache/pulsar/pull/11553#discussion_r687458847 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/service/BrokerEntryMetadataE2ETest.java ## @@ -273,6 +274,78 @@ public void

[GitHub] [pulsar] gaoran10 commented on issue #11576: Avro or KeyValue Schema throws exception in PulsarRecordCursor

2021-08-12 Thread GitBox
gaoran10 commented on issue #11576: URL: https://github.com/apache/pulsar/issues/11576#issuecomment-897407506 I think the root cause is the message metadata is lake of schemaVersion property. You could change the message serialization method to add schema version in message metadata. --

[GitHub] [pulsar] frankxieke commented on a change in pull request #11555: [Offload] Add offload performance metrics for tiered storage.

2021-08-12 Thread GitBox
frankxieke commented on a change in pull request #11555: URL: https://github.com/apache/pulsar/pull/11555#discussion_r687427116 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/LedgerOffloaderMXBeanImpl.java ## @@ -0,0 +1,269 @@ +/** + * Licensed

[GitHub] [pulsar] frankxieke commented on a change in pull request #11555: [Offload] Add offload performance metrics for tiered storage.

2021-08-12 Thread GitBox
frankxieke commented on a change in pull request #11555: URL: https://github.com/apache/pulsar/pull/11555#discussion_r687426921 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/LedgerOffloaderMXBeanImpl.java ## @@ -0,0 +1,269 @@ +/** + * Licensed

[GitHub] [pulsar] frankxieke commented on a change in pull request #11555: [Offload] Add offload performance metrics for tiered storage.

2021-08-12 Thread GitBox
frankxieke commented on a change in pull request #11555: URL: https://github.com/apache/pulsar/pull/11555#discussion_r687425798 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/LedgerOffloaderMXBean.java ## @@ -0,0 +1,139 @@ +/** + * Licensed to the

  1   2   >