[GitHub] [pulsar] 97arushisharma closed issue #7488: ERROR org.apache.bookkeeper.client.BKException$ZKException: Error while using ZooKeeper

2020-07-14 Thread GitBox
97arushisharma closed issue #7488: URL: https://github.com/apache/pulsar/issues/7488 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] 97arushisharma commented on issue #7488: ERROR org.apache.bookkeeper.client.BKException$ZKException: Error while using ZooKeeper

2020-07-14 Thread GitBox
97arushisharma commented on issue #7488: URL: https://github.com/apache/pulsar/issues/7488#issuecomment-658559832 Yup. Thanks @sijie . This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] sijie commented on issue #7488: ERROR org.apache.bookkeeper.client.BKException$ZKException: Error while using ZooKeeper

2020-07-14 Thread GitBox
sijie commented on issue #7488: URL: https://github.com/apache/pulsar/issues/7488#issuecomment-658554731 @97arushisharma the configurations look good to me. Based on your last comment, it seems that you have addressed your problem by setting `zkLedgersRootPath`. Is that correct? Any other

[pulsar-helm-chart] branch master updated: Add optional user provided zookeeper as metadata store for other components (#38)

2020-07-14 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git The following commit(s) were added to refs/heads/master by this push: new 135868c Add optional user provided

[GitHub] [pulsar] srkukarni opened a new pull request #7537: Fix Broken AdminApiTest2

2020-07-14 Thread GitBox
srkukarni opened a new pull request #7537: URL: https://github.com/apache/pulsar/pull/7537 *(If this PR fixes a github issue, please add `Fixes #`.)* Fixes # *(or if this PR is one task of a github issue, please add `Master Issue: #` to link to the master issue.)*

[pulsar-helm-chart] branch master updated (8f9c1d1 -> 714ff41)

2020-07-14 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-helm-chart.git. from 8f9c1d1 Update appVersion to 2.6.0 (#36) add 714ff41 add targetport for grafana nad manager service

[GitHub] [pulsar] devinbost commented on issue #6054: Catastrophic frequent random topic freezes, especially on high-traffic topics.

2020-07-14 Thread GitBox
devinbost commented on issue #6054: URL: https://github.com/apache/pulsar/issues/6054#issuecomment-658548718 @sijie Unfortunately, I don't. What's the best way for us to get a heap dump when this happens again? This is an

[GitHub] [pulsar] sijie commented on issue #6054: Catastrophic frequent random topic freezes, especially on high-traffic topics.

2020-07-14 Thread GitBox
sijie commented on issue #6054: URL: https://github.com/apache/pulsar/issues/6054#issuecomment-658537755 @devinbost did you happen to have a heap dump? This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar-client-go] taiyang-li commented on issue #318: Some messages are missing when reopen consumer

2020-07-14 Thread GitBox
taiyang-li commented on issue #318: URL: https://github.com/apache/pulsar-client-go/issues/318#issuecomment-658527623 > > @taiyang-li Are you running consumers before running producers? > > Run producer first and then consumer

[GitHub] [pulsar-client-go] taiyang-li removed a comment on issue #318: Some messages are missing when reopen consumer

2020-07-14 Thread GitBox
taiyang-li removed a comment on issue #318: URL: https://github.com/apache/pulsar-client-go/issues/318#issuecomment-658527623 > > @taiyang-li Are you running consumers before running producers? > > Run producer first and then consumer

[GitHub] [pulsar-client-go] taiyang-li closed issue #318: Some messages are missing when reopen consumer

2020-07-14 Thread GitBox
taiyang-li closed issue #318: URL: https://github.com/apache/pulsar-client-go/issues/318 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] Anonymitaet edited a comment on pull request #7534: [docs] Improve configuration details of functions worker

2020-07-14 Thread GitBox
Anonymitaet edited a comment on pull request #7534: URL: https://github.com/apache/pulsar/pull/7534#issuecomment-658506639 @addisonj thank you so much for your tech inputs. @wolfstudy could u pls help review? Thanks. FYI: @addisonj has added the following contents (in blue)

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #7534: [docs] Improve configuration details of functions worker

2020-07-14 Thread GitBox
Anonymitaet commented on a change in pull request #7534: URL: https://github.com/apache/pulsar/pull/7534#discussion_r454761678 ## File path: site2/docs/functions-runtime.md ## @@ -138,20 +149,75 @@ roleRef: apiGroup: rbac.authorization.k8s.io kind: ClusterRole name:

[GitHub] [pulsar] Anonymitaet edited a comment on pull request #7534: [docs] Improve configuration details of functions worker

2020-07-14 Thread GitBox
Anonymitaet edited a comment on pull request #7534: URL: https://github.com/apache/pulsar/pull/7534#issuecomment-658506639 @wolfstudy could u pls help review? Thanks FYI @addisonj has added the following contents (in blue)

[GitHub] [pulsar-client-go] zymap commented on a change in pull request #313: Support oauth2 authentication for pulsar-client-go

2020-07-14 Thread GitBox
zymap commented on a change in pull request #313: URL: https://github.com/apache/pulsar-client-go/pull/313#discussion_r454761641 ## File path: go.mod ## @@ -1,20 +1,25 @@ module github.com/apache/pulsar-client-go -go 1.12 +go 1.13 require (

[GitHub] [pulsar-client-go] zymap opened a new issue #325: Using a real oauth2 service to test the oauth2 auth plugin

2020-07-14 Thread GitBox
zymap opened a new issue #325: URL: https://github.com/apache/pulsar-client-go/issues/325 *Motivation* The PR #313 supports the oauth2 auth plugin and I mock an oauth2 service for integration test. It is better to use a real oauth2 service to make an integration test.

[GitHub] [pulsar] Jennifer88huang opened a new issue #7536: "Proxy support with SNI routing" is not displayed in the "Concepts and Architecture" chapter

2020-07-14 Thread GitBox
Jennifer88huang opened a new issue #7536: URL: https://github.com/apache/pulsar/issues/7536 **Describe the bug** I pull request in [PR-7471](https://github.com/apache/pulsar/pull/7471) and add sidebar for "Pulasr SNI routing with ATS". I've tested locally, the sidebar works. However,

[GitHub] [pulsar] wolfstudy commented on pull request #6752: Support different docker images in Kubernetes runtime of Pulsar Functions

2020-07-14 Thread GitBox
wolfstudy commented on pull request #6752: URL: https://github.com/apache/pulsar/pull/6752#issuecomment-658512237 ping @jerrypeng please take a look, thanks This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] Anonymitaet edited a comment on pull request #7534: [docs] Improve configuration details of functions worker

2020-07-14 Thread GitBox
Anonymitaet edited a comment on pull request #7534: URL: https://github.com/apache/pulsar/pull/7534#issuecomment-658506639 @wolfstudy could u pls help review? Thanks FYI @addisonj has added the following contents (in blue)

[GitHub] [pulsar-client-go] shohi commented on a change in pull request #323: add Logger field to ClientOptions

2020-07-14 Thread GitBox
shohi commented on a change in pull request #323: URL: https://github.com/apache/pulsar-client-go/pull/323#discussion_r454753639 ## File path: pulsar/client.go ## @@ -91,6 +92,9 @@ type ClientOptions struct { // Max number of connections to a single broker that will

[GitHub] [pulsar] xuesong172500 opened a new issue #7535: Modify the create/update function mode

2020-07-14 Thread GitBox
xuesong172500 opened a new issue #7535: URL: https://github.com/apache/pulsar/issues/7535 **Is your feature request related to a problem? Please describe.** Can the create / update function be used through a storm like framework storm.sh To manage the start and stop of function,

[GitHub] [pulsar] Anonymitaet commented on pull request #7534: [docs] Improve configuration details of functions worker

2020-07-14 Thread GitBox
Anonymitaet commented on pull request #7534: URL: https://github.com/apache/pulsar/pull/7534#issuecomment-658506639 @wolfstudy could u pls help review? Thanks This is an automated message from the Apache Git Service. To

[pulsar] branch master updated (eef63de -> 2ab520b)

2020-07-14 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from eef63de Fix: function BC issue introduced in 2.6 (#7528) add 2ab520b Metadata compact config (#7532) No

[GitHub] [pulsar] srkukarni merged pull request #7532: Metadata compact config

2020-07-14 Thread GitBox
srkukarni merged pull request #7532: URL: https://github.com/apache/pulsar/pull/7532 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] addisonj commented on pull request #7534: [docs] Improve configuration details of functions worker

2020-07-14 Thread GitBox
addisonj commented on pull request #7534: URL: https://github.com/apache/pulsar/pull/7534#issuecomment-658415855 @Anonymitaet here are the docs for #6086, let me know who else should review with you being out :) This is an

[GitHub] [pulsar] addisonj opened a new pull request #7534: [docs] Improve configuration details of functions worker

2020-07-14 Thread GitBox
addisonj opened a new pull request #7534: URL: https://github.com/apache/pulsar/pull/7534 Fixes #6806 ### Motivation The documentation for functions worker were lacking, as it didn't cover how to configure the functions worker in a variety of possible configurations of

[pulsar] branch master updated (b197962 -> eef63de)

2020-07-14 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 b197962 Add async state manupulation methods in java functions (#7468) add eef63de Fix: function BC issue

[GitHub] [pulsar] jerrypeng merged pull request #7528: Fix: function BC issue introduced in 2.6

2020-07-14 Thread GitBox
jerrypeng merged pull request #7528: URL: https://github.com/apache/pulsar/pull/7528 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] devinbost commented on issue #6054: Catastrophic frequent random topic freezes, especially on high-traffic topics.

2020-07-14 Thread GitBox
devinbost commented on issue #6054: URL: https://github.com/apache/pulsar/issues/6054#issuecomment-658404866 If it's a clue, it only happens on topics involving functions. This is an automated message from the Apache Git

[GitHub] [pulsar] devinbost commented on issue #6054: Catastrophic frequent random topic freezes, especially on high-traffic topics.

2020-07-14 Thread GitBox
devinbost commented on issue #6054: URL: https://github.com/apache/pulsar/issues/6054#issuecomment-658404472 @sijie @codelipenghui We have confirmed that this is actually still an issue in Pulsar 2.5.2. This is an

[GitHub] [pulsar] srkukarni opened a new pull request #7532: Metadata compact config

2020-07-14 Thread GitBox
srkukarni opened a new pull request #7532: URL: https://github.com/apache/pulsar/pull/7532 *(If this PR fixes a github issue, please add `Fixes #`.)* Fixes # *(or if this PR is one task of a github issue, please add `Master Issue: #` to link to the master issue.)*

[GitHub] [pulsar] horsteff opened a new pull request #7533: [Issue 7492][pulsar-broker] Cleanup configuration process when using PulsarStandaloneBuilder

2020-07-14 Thread GitBox
horsteff opened a new pull request #7533: URL: https://github.com/apache/pulsar/pull/7533 Fixes #7492 ### Motivation `PulsarStandaloneBuilder.withConfig()` does effectively nothing because provided config will be overwritten in `PulsarStandaloneBuilder.build()`. Any

[GitHub] [pulsar] merlimat commented on a change in pull request #7528: Fix: function BC issue introduced in 2.6

2020-07-14 Thread GitBox
merlimat commented on a change in pull request #7528: URL: https://github.com/apache/pulsar/pull/7528#discussion_r454518289 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/FunctionWorkerStarter.java ## @@ -66,11 +66,12 @@ public static

[GitHub] [pulsar-client-go] EronWright commented on a change in pull request #313: Support oauth2 authentication for pulsar-client-go

2020-07-14 Thread GitBox
EronWright commented on a change in pull request #313: URL: https://github.com/apache/pulsar-client-go/pull/313#discussion_r454491977 ## File path: go.mod ## @@ -1,20 +1,25 @@ module github.com/apache/pulsar-client-go -go 1.12 +go 1.13 require (

[GitHub] [pulsar-client-go] merlimat commented on a change in pull request #323: add Logger field to ClientOptions

2020-07-14 Thread GitBox
merlimat commented on a change in pull request #323: URL: https://github.com/apache/pulsar-client-go/pull/323#discussion_r454487155 ## File path: pulsar/client.go ## @@ -91,6 +92,9 @@ type ClientOptions struct { // Max number of connections to a single broker that

[GitHub] [pulsar] Lagranmoon opened a new issue #7531: prebuild protoc incompatible for build project

2020-07-14 Thread GitBox
Lagranmoon opened a new issue #7531: URL: https://github.com/apache/pulsar/issues/7531 **Describe the bug** I modify the PlusarApi.proto and then use the generate_protobuf_docker.sh to regenerate Java files.But when I rebuild the project using "mvn clean package -DskipTests",it throws

[GitHub] [pulsar] xuesong172500 opened a new issue #7530: In some cases, the workerport does not listening after the functions worker process is started.

2020-07-14 Thread GitBox
xuesong172500 opened a new issue #7530: URL: https://github.com/apache/pulsar/issues/7530 **Describe the bug** In some cases, after the functions worker process is started, the workerport does not listen, the process exists, and no error information is found,The broker and functions

[GitHub] [pulsar] aahmed-se opened a new pull request #7529: Add test group tags for functions and state integration tests

2020-07-14 Thread GitBox
aahmed-se opened a new pull request #7529: URL: https://github.com/apache/pulsar/pull/7529 * Add additional tags to functions and state integration tests. This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] anvinjain commented on issue #7481: Default value and in-line doc for bookkeeper config: openLedgerRereplicationGracePeriod is incorrect

2020-07-14 Thread GitBox
anvinjain commented on issue #7481: URL: https://github.com/apache/pulsar/issues/7481#issuecomment-658013060 @codelipenghui Sure, I will take this up. This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar-client-go] LvBay commented on issue #281: producer.Send() blocks for too long time

2020-07-14 Thread GitBox
LvBay commented on issue #281: URL: https://github.com/apache/pulsar-client-go/issues/281#issuecomment-658008123 @merlimat @liangyuanpeng I'm sorry I can't reproduce it in a test environment. But in a production environment, it will always appear at an irregular time. What is

[GitHub] [pulsar] jerrypeng opened a new pull request #7528: Fix: function BC issue introduced in 2.6

2020-07-14 Thread GitBox
jerrypeng opened a new pull request #7528: URL: https://github.com/apache/pulsar/pull/7528 ### Motivation There was a backwards compatibility breakage introduced in: https://github.com/apache/pulsar/pull/5985 If running function workers separately from brokers,