[GitHub] [pulsar] lhotari commented on issue #7859: Use Java 11 for Docker Images

2020-09-02 Thread GitBox
lhotari commented on issue #7859: URL: https://github.com/apache/pulsar/issues/7859#issuecomment-686272487 > Java 8 and Docker doesn't work that well together. Some things got backported to Java 8 but only Java 10+ supports Memory and CPU limits inside Docker Containers correctly.

[GitHub] [pulsar] sijie commented on issue #7952: OAuth2 url .well-known not parsed correctly

2020-09-02 Thread GitBox
sijie commented on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-686271862 @jiazhai Can you take a look? This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] sijie commented on issue #7960: AuthenticationOAuth2 as Authentication Plugin in Pulsar Proxy

2020-09-02 Thread GitBox
sijie commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686269458 @jiazhai Can you take a look? This is an automated message from the Apache Git Service. To respond to the message,

[pulsar] branch master updated (c7ac08b -> 452a5f2)

2020-09-02 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 c7ac08b [pulsar-broker] Stop to dispatch when skip message temporally since Key_Shared consumer stuck on

[GitHub] [pulsar] jerrypeng merged pull request #7962: Await thread pool termination when closing Pulsar client

2020-09-02 Thread GitBox
jerrypeng merged pull request #7962: URL: https://github.com/apache/pulsar/pull/7962 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] 315157973 commented on pull request #7940: Fix PR #7818 #7802 does not support admin cli

2020-09-02 Thread GitBox
315157973 commented on pull request #7940: URL: https://github.com/apache/pulsar/pull/7940#issuecomment-686233676 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] 315157973 commented on pull request #7938: Update the pulsar admin cli java doc to support set/get/remove deduplication on topic level

2020-09-02 Thread GitBox
315157973 commented on pull request #7938: URL: https://github.com/apache/pulsar/pull/7938#issuecomment-686233109 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] zymap commented on pull request #7936: Remove python2

2020-09-02 Thread GitBox
zymap commented on pull request #7936: URL: https://github.com/apache/pulsar/pull/7936#issuecomment-686192400 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] aloyszhang removed a comment on pull request #7901: [Bookkeeper] make bookkeeper throttle Configurable

2020-09-02 Thread GitBox
aloyszhang removed a comment on pull request #7901: URL: https://github.com/apache/pulsar/pull/7901#issuecomment-686181066 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] aloyszhang commented on pull request #7901: [Bookkeeper] make bookkeeper throttle Configurable

2020-09-02 Thread GitBox
aloyszhang commented on pull request #7901: URL: https://github.com/apache/pulsar/pull/7901#issuecomment-686181066 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] aloyszhang commented on pull request #7926: fix publish buffer of one ServerCnx caculated multi-times when check broker's publish buffer

2020-09-02 Thread GitBox
aloyszhang commented on pull request #7926: URL: https://github.com/apache/pulsar/pull/7926#issuecomment-686181243 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] jerrypeng opened a new pull request #7965: Various fixes for Batch Source

2020-09-02 Thread GitBox
jerrypeng opened a new pull request #7965: URL: https://github.com/apache/pulsar/pull/7965 ### Modifications 1. Create intermediate topic/subscription prior to function running in case auto topic creation is turned off 2. Fix possible NPE in CronTrigger when calling stop()

[GitHub] [pulsar] rdhabalia opened a new pull request #7964: [pulsar-broker] add info log when broker doesn't find disconnected-consumer in consumer-set

2020-09-02 Thread GitBox
rdhabalia opened a new pull request #7964: URL: https://github.com/apache/pulsar/pull/7964 ### Motivation We have seen frequent instances where topic unloading stuck and we have to restart broker. Unloading topic gives 500 with below broker log ``` 01:26:19.691 [pulsar-web-35-6]

[GitHub] [pulsar] rdhabalia opened a new pull request #7963: [pulsar-proxy] Fix memory leak with debug log-level

2020-09-02 Thread GitBox
rdhabalia opened a new pull request #7963: URL: https://github.com/apache/pulsar/pull/7963 ### Motivation Right now, Proxy has memory leak [debug

[GitHub] [pulsar] ErikJansenIRefact commented on issue #7372: Messages on topic are deleted on restart of standalone cluster

2020-09-02 Thread GitBox
ErikJansenIRefact commented on issue #7372: URL: https://github.com/apache/pulsar/issues/7372#issuecomment-685958856 Does make sense. But should be documented more clearly that this is the expected behaviour. This is an

[GitHub] [pulsar] harissecic commented on issue #7952: OAuth2 url .well-known not parsed correctly

2020-09-02 Thread GitBox
harissecic commented on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-685947635 @cuzyoucant Thank you for detailed description. I'll consider it. However I'm currently using JWT until this gets fixed and I'm trying to make integration with a certain framework

[GitHub] [pulsar] jerrypeng opened a new pull request #7962: Await thread pool termination when closing Pulsar client

2020-09-02 Thread GitBox
jerrypeng opened a new pull request #7962: URL: https://github.com/apache/pulsar/pull/7962 ### Motivation Let's wait for thread pool termination in the Pulsar client so we make more of an effort to make sure all Pulsar threads are shutdown when the pulsar client is closed.

[GitHub] [pulsar] cuzyoucant commented on issue #7952: OAuth2 url .well-known not parsed correctly

2020-09-02 Thread GitBox
cuzyoucant commented on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-685904145 Hey. We ran into the same problem as you described above. The better solution would be to have a config somewhere where we can set what gets appended to the issuerURL.

[GitHub] [pulsar] fracasula commented on issue #7682: Stuck consumer: terminate called after throwing an instance of 'std::bad_alloc'

2020-09-02 Thread GitBox
fracasula commented on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-685823123 @codelipenghui I replaced `apachepulsar/pulsar:2.6.1` with `streamnative/pulsar:2.6.1-sn-hotfix-1` in my Kubernetes configuration but now I have the pods failing on startup with

[GitHub] [pulsar] cuzyoucant commented on issue #7960: AuthenticationOAuth2 as Authentication Plugin in Pulsar Proxy

2020-09-02 Thread GitBox
cuzyoucant commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-685795777 After some even more debugging i narrowed it done even more. In the same file ServiceChannelInitializer.java line 85 to 89 when the AuthenticationFactory builds the

[GitHub] [pulsar] sijia-w opened a new pull request #7961: update aws

2020-09-02 Thread GitBox
sijia-w opened a new pull request #7961: URL: https://github.com/apache/pulsar/pull/7961 Motivation: There is an empty line in the table, which influences the following lines. Modification: The empty line is deleted.

[pulsar] branch master updated: [pulsar-broker] Stop to dispatch when skip message temporally since Key_Shared consumer stuck on delivery (#7553)

2020-09-02 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 c7ac08b [pulsar-broker] Stop to dispatch when

[GitHub] [pulsar] codelipenghui merged pull request #7553: [pulsar-broker] Stop to dispatch when skip message temporally since Key_Shared consumer stuck on delivery

2020-09-02 Thread GitBox
codelipenghui merged pull request #7553: URL: https://github.com/apache/pulsar/pull/7553 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: [Transaction] Support transaction abort on partition (#7953)

2020-09-02 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 3ac98d8 [Transaction] Support transaction

[GitHub] [pulsar] codelipenghui merged pull request #7953: [Transaction] Support transaction abort on partition

2020-09-02 Thread GitBox
codelipenghui merged pull request #7953: URL: https://github.com/apache/pulsar/pull/7953 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] codelipenghui commented on a change in pull request #7856: [Transaction] Handle Acknowledge In The Transaction

2020-09-02 Thread GitBox
codelipenghui commented on a change in pull request #7856: URL: https://github.com/apache/pulsar/pull/7856#discussion_r482094605 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/transaction/TransactionCoordinatorClientImpl.java ## @@ -176,6 +178,30 @@

[GitHub] [pulsar] zhanghaou commented on a change in pull request #7914: [ISSUE 7758] Support set Max Producer on topic level.

2020-09-02 Thread GitBox
zhanghaou commented on a change in pull request #7914: URL: https://github.com/apache/pulsar/pull/7914#discussion_r482098110 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -2368,6 +2371,53 @@ protected void

[pulsar-dotpulsar] branch master updated: Eat errors in SetupChannel (#52)

2020-09-02 Thread blankensteiner
This is an automated email from the ASF dual-hosted git repository. blankensteiner pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-dotpulsar.git The following commit(s) were added to refs/heads/master by this push: new 7b1eaf6 Eat errors in

[pulsar-dotpulsar] branch master updated: Eat errors in SetupChannel (#52)

2020-09-02 Thread blankensteiner
This is an automated email from the ASF dual-hosted git repository. blankensteiner pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-dotpulsar.git The following commit(s) were added to refs/heads/master by this push: new 7b1eaf6 Eat errors in

[GitHub] [pulsar] fracasula commented on issue #7682: Stuck consumer: terminate called after throwing an instance of 'std::bad_alloc'

2020-09-02 Thread GitBox
fracasula commented on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-685734170 @codelipenghui thanks! I'll give it a try :+1: This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] codelipenghui commented on pull request #7908: Add pulsar version info metric

2020-09-02 Thread GitBox
codelipenghui commented on pull request #7908: URL: https://github.com/apache/pulsar/pull/7908#issuecomment-685733600 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[pulsar] branch master updated: [Issue 7864][docs]Update document for how to pick active consumer for failover subscription. (#7917)

2020-09-02 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 6a2b574 [Issue 7864][docs]Update document for

[GitHub] [pulsar] codelipenghui merged pull request #7917: [Issue 7864][docs]Update document for how to pick active consumer for failover subscription.

2020-09-02 Thread GitBox
codelipenghui merged pull request #7917: URL: https://github.com/apache/pulsar/pull/7917 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] codelipenghui closed issue #7864: The master consumer is not chosen correctly in failover mode

2020-09-02 Thread GitBox
codelipenghui closed issue #7864: URL: https://github.com/apache/pulsar/issues/7864 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] codelipenghui commented on pull request #7919: check null point before setting auto read

2020-09-02 Thread GitBox
codelipenghui commented on pull request #7919: URL: https://github.com/apache/pulsar/pull/7919#issuecomment-685731916 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[pulsar] branch master updated: Add python oauth2 document (#7946)

2020-09-02 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 cd8b7c7 Add python oauth2 document (#7946)

[GitHub] [pulsar] codelipenghui merged pull request #7946: Add python oauth2 document

2020-09-02 Thread GitBox
codelipenghui merged pull request #7946: URL: https://github.com/apache/pulsar/pull/7946 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] codelipenghui commented on pull request #7947: replace if..else with switch

2020-09-02 Thread GitBox
codelipenghui commented on pull request #7947: URL: https://github.com/apache/pulsar/pull/7947#issuecomment-685729602 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] codelipenghui commented on pull request #7950: [Issue 7949][Tiered Storage] support aws creds per offload policies

2020-09-02 Thread GitBox
codelipenghui commented on pull request #7950: URL: https://github.com/apache/pulsar/pull/7950#issuecomment-685728348 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] codelipenghui closed issue #7933: Add some logging to improve Authentication debugging and Fix typos in code "occured" -> "occurred"

2020-09-02 Thread GitBox
codelipenghui closed issue #7933: URL: https://github.com/apache/pulsar/issues/7933 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

[pulsar] branch master updated: [Issue 7933][pulsar-broker-common] Add some logging to improve Authentication debugging and Fix typos in code "occured" -> "occurred" (#7934)

2020-09-02 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 7ab29d8 [Issue 7933][pulsar-broker-common]

[GitHub] [pulsar] codelipenghui merged pull request #7934: [Issue 7933][pulsar-broker-common] Add some logging to improve Authentication debugging and Fix typos in code "occured" -> "occurred"

2020-09-02 Thread GitBox
codelipenghui merged pull request #7934: URL: https://github.com/apache/pulsar/pull/7934 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] codelipenghui commented on pull request #7938: Update the pulsar admin cli java doc to support set/get/remove deduplication on topic level

2020-09-02 Thread GitBox
codelipenghui commented on pull request #7938: URL: https://github.com/apache/pulsar/pull/7938#issuecomment-685727396 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[pulsar] branch master updated: support message publish rate on topic level (#7948)

2020-09-02 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 66688d1 support message publish rate on topic

[GitHub] [pulsar] codelipenghui merged pull request #7948: support message publish rate on topic level

2020-09-02 Thread GitBox
codelipenghui merged pull request #7948: URL: https://github.com/apache/pulsar/pull/7948 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] codelipenghui commented on pull request #7951: Allow to build Pulsar on JDK14

2020-09-02 Thread GitBox
codelipenghui commented on pull request #7951: URL: https://github.com/apache/pulsar/pull/7951#issuecomment-685725712 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] codelipenghui merged pull request #7955: Ignore unit-tests ci when the pull request only change doc

2020-09-02 Thread GitBox
codelipenghui merged pull request #7955: URL: https://github.com/apache/pulsar/pull/7955 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: Ignore unit-tests ci when the pull requeat only change doc (#7955)

2020-09-02 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 5cee214 Ignore unit-tests ci when the pull

[pulsar] branch master updated: [blog] Add image and update file name (#7958)

2020-09-02 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 cacded2 [blog] Add image and update file name

[GitHub] [pulsar] codelipenghui merged pull request #7958: [blog] Add image and update file name

2020-09-02 Thread GitBox
codelipenghui merged pull request #7958: URL: https://github.com/apache/pulsar/pull/7958 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] codelipenghui commented on issue #7682: Stuck consumer: terminate called after throwing an instance of 'std::bad_alloc'

2020-09-02 Thread GitBox
codelipenghui commented on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-685713536 @fracasula I have pushed a new docker image to the docker hub

[GitHub] [pulsar] cuzyoucant commented on issue #7960: AuthenticationOAuth2 as Authentication Plugin in Pulsar Proxy

2020-09-02 Thread GitBox
cuzyoucant commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-685684259 After some more debugging we found some additional information about the bug. We got it to run in the standalone after disabling TLS connections in the config. It would be

[GitHub] [pulsar] Huanli-Meng commented on pull request #7956: [Doc] Remove stateful function note from the Deploy a cluster on bare metal document

2020-09-02 Thread GitBox
Huanli-Meng commented on pull request #7956: URL: https://github.com/apache/pulsar/pull/7956#issuecomment-685675764 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] hangc0276 commented on pull request #7948: support message publish rate on topic level

2020-09-02 Thread GitBox
hangc0276 commented on pull request #7948: URL: https://github.com/apache/pulsar/pull/7948#issuecomment-685664703 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] cuzyoucant opened a new issue #7960: AuthenticationOAuth2 as Authentication Plugin in Pulsar Proxy

2020-09-02 Thread GitBox
cuzyoucant opened a new issue #7960: URL: https://github.com/apache/pulsar/issues/7960 **Describe the bug** When setting AuthenticationOAuth2 as brokerClientAuthenticationPlugin the proxy crashes on start-up with an uncaught Exception with message "null". Relevant config:

[GitHub] [pulsar] KannarFr opened a new issue #7959: Force namespace deletion even if not empty

2020-09-02 Thread GitBox
KannarFr opened a new issue #7959: URL: https://github.com/apache/pulsar/issues/7959 **Is your enhancement request related to a problem? Please describe.** Currently namespace needs to be empty to be deleted. **Describe the solution you'd like** I think we should have a

[GitHub] [pulsar] Huanli-Meng commented on pull request #7956: [Doc] Remove stateful function note from the Deploy a cluster on bare metal document

2020-09-02 Thread GitBox
Huanli-Meng commented on pull request #7956: URL: https://github.com/apache/pulsar/pull/7956#issuecomment-685454368 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] Jennifer88huang merged pull request #7886: [blog] Add Pulsar Summit Asia 2020 CFP blog

2020-09-02 Thread GitBox
Jennifer88huang merged pull request #7886: URL: https://github.com/apache/pulsar/pull/7886 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

[pulsar] branch master updated: [blog] Add Pulsar Summit Asia 2020 CFP blog (#7886)

2020-09-02 Thread hjf
This is an automated email from the ASF dual-hosted git repository. hjf 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 2248a2e [blog] Add Pulsar Summit Asia 2020 CFP

[GitHub] [pulsar] Renkai opened a new pull request #7957: [Issue 5739][Client-Java] Use message bytes instead of message number for producer pending messages

2020-09-02 Thread GitBox
Renkai opened a new pull request #7957: URL: https://github.com/apache/pulsar/pull/7957 *(If this PR fixes a github issue, please add `Fixes #`.)* Fixes #5739 ### Motivation Add a AtomicLong to keep bytessize of a queue ### Modifications *Describe

[GitHub] [pulsar] Jennifer88huang commented on pull request #7958: [blog] Add image and update file name

2020-09-02 Thread GitBox
Jennifer88huang commented on pull request #7958: URL: https://github.com/apache/pulsar/pull/7958#issuecomment-685415256 Check the preview locally, and it works well. This is an automated message from the Apache Git Service.

[GitHub] [pulsar] Jennifer88huang opened a new pull request #7958: [blog] Add image and update file name

2020-09-02 Thread GitBox
Jennifer88huang opened a new pull request #7958: URL: https://github.com/apache/pulsar/pull/7958 ### Motivation To avoid conflicts on my master branch, I've merged https://github.com/apache/pulsar/pull/7886 ### Modifications Add image for this blog, and update the file

[pulsar-dotpulsar] branch master updated: Update test method names to follow Feature/MethodName_Given[StateUnderTest]_Should[ExpectedBehavior]

2020-09-02 Thread blankensteiner
This is an automated email from the ASF dual-hosted git repository. blankensteiner pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-dotpulsar.git The following commit(s) were added to refs/heads/master by this push: new ac1e228 Update test method

[pulsar-dotpulsar] branch master updated: Fix checking MessageId against null throwing NRE (#50)

2020-09-02 Thread blankensteiner
This is an automated email from the ASF dual-hosted git repository. blankensteiner pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-dotpulsar.git The following commit(s) were added to refs/heads/master by this push: new 1f4e498 Fix checking

[GitHub] [pulsar] Huanli-Meng opened a new pull request #7956: [Doc] Remove stateful function note from the Deploy a cluster on bare metal document

2020-09-02 Thread GitBox
Huanli-Meng opened a new pull request #7956: URL: https://github.com/apache/pulsar/pull/7956 ### Motivation The state is not production-ready. ### Modifications Remove the note about stateful function from the Deploy a cluster on bare metal document.