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

2020-09-03 Thread GitBox
codelipenghui commented on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-686323710 @fracasula Oh, I see. I will build a new image for you. This is an automated message from the Apache Git Service

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

2020-09-03 Thread GitBox
jiazhai commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686335674 @cuzyoucant Thanks for the report. >So the problem seems to be the Authentication Object from the AuthenticationFactory in line 91/92 returning an empty AuthenticationDataOAuth2 O

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

2020-09-03 Thread GitBox
jiazhai commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686339088 ` if (serviceConfig.isTlsEnabledWithBroker())` This means proxy will use tls to communicate with broker, and it expect the `brokerClientAuthenticationPlugin` and `brokerClientAuth

[GitHub] [pulsar] jiazhai removed a comment on issue #7960: AuthenticationOAuth2 as Authentication Plugin in Pulsar Proxy

2020-09-03 Thread GitBox
jiazhai removed a comment on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686339088 ` if (serviceConfig.isTlsEnabledWithBroker())` This means proxy will use tls to communicate with broker, and it expect the `brokerClientAuthenticationPlugin` and `brokerCl

[pulsar] branch master updated: Replace if..else with switch in TypeMessageBuilderImpl (#7947)

2020-09-03 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 7e4ace2 Replace if..else with switch in TypeMe

[GitHub] [pulsar] codelipenghui merged pull request #7947: replace if..else with switch

2020-09-03 Thread GitBox
codelipenghui merged pull request #7947: URL: https://github.com/apache/pulsar/pull/7947 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 g

[pulsar] branch master updated: Fix PR #7818 #7802 does not support admin cli (#7940)

2020-09-03 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 8377396 Fix PR #7818 #7802 does not support ad

[GitHub] [pulsar] codelipenghui merged pull request #7940: Fix PR #7818 #7802 does not support admin cli

2020-09-03 Thread GitBox
codelipenghui merged pull request #7940: URL: https://github.com/apache/pulsar/pull/7940 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 g

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

2020-09-03 Thread GitBox
jiazhai commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686343461 @cuzyoucant How is the tls configured? This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [pulsar] codelipenghui merged pull request #7914: [ISSUE 7758] Support set Max Producer on topic level.

2020-09-03 Thread GitBox
codelipenghui merged pull request #7914: URL: https://github.com/apache/pulsar/pull/7914 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 g

[pulsar] branch master updated (8377396 -> 2e88fdb)

2020-09-03 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 8377396 Fix PR #7818 #7802 does not support admin cli (#7940) add 2e88fdb [ISSUE 7758] Support set Max Produc

[GitHub] [pulsar] jiazhai edited a comment on issue #7960: AuthenticationOAuth2 as Authentication Plugin in Pulsar Proxy

2020-09-03 Thread GitBox
jiazhai edited a comment on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686343461 @cuzyoucant How is the tls configured? I suspected it is enabled for "tlsEnabledWithKeyStore" ? This is an au

[GitHub] [pulsar] codelipenghui merged pull request #7905: fix pulsar metrics providing wrong information

2020-09-03 Thread GitBox
codelipenghui merged pull request #7905: URL: https://github.com/apache/pulsar/pull/7905 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 g

[pulsar] branch master updated (8377396 -> 2e88fdb)

2020-09-03 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 8377396 Fix PR #7818 #7802 does not support admin cli (#7940) add 2e88fdb [ISSUE 7758] Support set Max Produc

[pulsar] branch master updated (2e88fdb -> b9dbb6a)

2020-09-03 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 2e88fdb [ISSUE 7758] Support set Max Producer on topic level. (#7914) add b9dbb6a fix pulsar metrics providin

[GitHub] [pulsar] codelipenghui closed issue #7799: Pulsar metrics providing wrong information

2020-09-03 Thread GitBox
codelipenghui closed issue #7799: URL: https://github.com/apache/pulsar/issues/7799 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] Jennifer88huang merged pull request #7961: update aws

2020-09-03 Thread GitBox
Jennifer88huang merged pull request #7961: URL: https://github.com/apache/pulsar/pull/7961 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: update aws (#7961)

2020-09-03 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 c9026ca update aws (#7961) c9026ca is described be

[GitHub] [pulsar] codelipenghui commented on pull request #7919: check null point before setting auto read

2020-09-03 Thread GitBox
codelipenghui commented on pull request #7919: URL: https://github.com/apache/pulsar/pull/7919#issuecomment-686350110 @yuanboliu You can `checkout -b` a new branch when you work on a bug fix or feature. For more details, you can refer to http://pulsar.apache.org/en/contributing/#create-a-b

[pulsar] branch master updated (c9026ca -> 0ca52f3)

2020-09-03 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 c9026ca update aws (#7961) add 0ca52f3 update node client document (#7966) No new revisions were added by this u

[GitHub] [pulsar] Jennifer88huang merged pull request #7966: [docs] Update Node client document

2020-09-03 Thread GitBox
Jennifer88huang merged pull request #7966: URL: https://github.com/apache/pulsar/pull/7966 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] cuzyoucant commented on issue #7960: AuthenticationOAuth2 as Authentication Plugin in Pulsar Proxy

2020-09-03 Thread GitBox
cuzyoucant commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686351973 @jiazhai The parameters for oauth2 are configured and are working, because as i pass in a dummytoken the proxy starts, notices that the token is invalid and gets a new one, and the

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

2020-09-03 Thread GitBox
cuzyoucant commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686354984 @jiazhai I can update you about the exact config tomorrow. I do not have it by hand right know. This is an automat

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

2020-09-03 Thread GitBox
zymap commented on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-686361630 Hi, @harissecic By this error `java.io.IOException: Cannot obtain authorization metadata from https://***/.well-known/openid-configuration`, it seems the client can not access the confi

[GitHub] [pulsar] gaoran10 commented on a change in pull request #7856: [Transaction] Handle Acknowledge In The Transaction

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

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

2020-09-03 Thread GitBox
codelipenghui commented on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-686384139 @fracasula You can try this one https://hub.docker.com/layers/streamnative/pulsar/2.6.1-sn-hotfix-2/images/sha256-765a82e1601e64cdb8ae8f8d7bd38a4ba936cf5f16557c7f7bb36904813bf0ff

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

2020-09-03 Thread GitBox
jiazhai commented on issue #7960: URL: https://github.com/apache/pulsar/issues/7960#issuecomment-686385381 Thanks @cuzyoucant , also, Are there any suspicious logs in server-side ? This is an automated message from the Apache

[GitHub] [pulsar] gaoran10 commented on a change in pull request #7856: [Transaction] Handle Acknowledge In The Transaction

2020-09-03 Thread GitBox
gaoran10 commented on a change in pull request #7856: URL: https://github.com/apache/pulsar/pull/7856#discussion_r482867544 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/TransactionMetadataStoreService.java ## @@ -207,26 +208,39 @@ public void removeTrans

[GitHub] [pulsar] gaoran10 commented on a change in pull request #7856: [Transaction] Handle Acknowledge In The Transaction

2020-09-03 Thread GitBox
gaoran10 commented on a change in pull request #7856: URL: https://github.com/apache/pulsar/pull/7856#discussion_r482867197 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentSubscription.java ## @@ -1197,8 +1207,8 @@ public void de

[GitHub] [pulsar] gaoran10 commented on a change in pull request #7856: [Transaction] Handle Acknowledge In The Transaction

2020-09-03 Thread GitBox
gaoran10 commented on a change in pull request #7856: URL: https://github.com/apache/pulsar/pull/7856#discussion_r482867294 ## File path: pulsar-transaction/coordinator/src/main/java/org/apache/pulsar/transaction/coordinator/TransactionSubscription.java ## @@ -0,0 +1,61 @@ +/*

[GitHub] [pulsar] 315157973 commented on pull request #7883: Support topicl level offload

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

[GitHub] [pulsar] zhanghaou opened a new pull request #7968: [Issue 7759] Support set Max Consumer on topic level.

2020-09-03 Thread GitBox
zhanghaou opened a new pull request #7968: URL: https://github.com/apache/pulsar/pull/7968 Fix [https://github.com/apache/pulsar/issues/7759](https://github.com/apache/pulsar/issues/7759), and master issue [https://github.com/apache/pulsar/issues/2688](https://github.com/apache/pulsar/iss

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

2020-09-03 Thread GitBox
fracasula commented on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-686429997 > @fracasula You can try this one https://hub.docker.com/layers/streamnative/pulsar/2.6.1-sn-hotfix-2/images/sha256-765a82e1601e64cdb8ae8f8d7bd38a4ba936cf5f16557c7f7bb36904813bf0ff?c

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

2020-09-03 Thread GitBox
fracasula edited a comment on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-686429997 > @fracasula You can try this one https://hub.docker.com/layers/streamnative/pulsar/2.6.1-sn-hotfix-2/images/sha256-765a82e1601e64cdb8ae8f8d7bd38a4ba936cf5f16557c7f7bb36904813

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

2020-09-03 Thread GitBox
harissecic commented on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-686447346 > Hi, @harissecic By this error `java.io.IOException: Cannot obtain authorization metadata from https://***/.well-known/openid-configuration`, it seems the client can not access th

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

2020-09-03 Thread GitBox
harissecic edited a comment on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-686447346 > Hi, @harissecic By this error `java.io.IOException: Cannot obtain authorization metadata from https://***/.well-known/openid-configuration`, it seems the client can not ac

[GitHub] [pulsar] tuteng opened a new pull request #7969: Fixed website admin cli link error

2020-09-03 Thread GitBox
tuteng opened a new pull request #7969: URL: https://github.com/apache/pulsar/pull/7969 Fixes #7915 *(or if this PR is one task of a github issue, please add `Master Issue: #` to link to the master issue.)* ### Motivation * Website admin cli link error

[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-03 Thread GitBox
315157973 commented on pull request #7938: URL: https://github.com/apache/pulsar/pull/7938#issuecomment-686465424 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the messa

[pulsar] branch master updated: [Admin cli]Fixed website admin cli link error (#7969)

2020-09-03 Thread zhaijia
This is an automated email from the ASF dual-hosted git repository. zhaijia 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 5432e86 [Admin cli]Fixed website admin cli lin

[GitHub] [pulsar] jiazhai closed issue #7915: The pulsar-admin page in release 2.6.1 is unavailable.

2020-09-03 Thread GitBox
jiazhai closed issue #7915: URL: https://github.com/apache/pulsar/issues/7915 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 sp

[GitHub] [pulsar] jiazhai merged pull request #7969: [Admin cli]Fixed website admin cli link error

2020-09-03 Thread GitBox
jiazhai merged pull request #7969: URL: https://github.com/apache/pulsar/pull/7969 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 t

[GitHub] [pulsar] MarvinCai commented on pull request #7858: Add config to lazily recover cursors when recovering a managed ledger.

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

[GitHub] [pulsar] MarvinCai commented on pull request #7858: Add config to lazily recover cursors when recovering a managed ledger.

2020-09-03 Thread GitBox
MarvinCai commented on pull request #7858: URL: https://github.com/apache/pulsar/pull/7858#issuecomment-686497096 /pulsarbot rerun unit-tests This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [pulsar] zhanghaou commented on pull request #7968: [Issue 7759] Support set Max Consumer on topic level.

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

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

2020-09-03 Thread GitBox
cuzyoucant commented on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-686508429 @zymap I can agree what @harissecic says. The problem is the line 100 in the DefaultMetadataResolver.java `return new URL(issuerUrl, "/.well-known/openid-configuration");`

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

2020-09-03 Thread GitBox
codelipenghui commented on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-686526204 @fracasula interesting... did you specify the offloader such as `managedLedgerOffloadDriver` in the broker.conf? The default `managedLedgerOffloadDriver` is null, so the broker

[pulsar] branch master updated: Update the pulsar admin cli java doc to support set/get/remove deduplication on topic level (#7938)

2020-09-03 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 1030fbe Update the pulsar admin cli java doc t

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

2020-09-03 Thread GitBox
codelipenghui merged pull request #7938: URL: https://github.com/apache/pulsar/pull/7938 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 g

[GitHub] [pulsar] codelipenghui closed issue #7920: Update the pulsar admin cli java doc to support set/get/remove deduplication on topic level

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

2020-09-03 Thread GitBox
fracasula commented on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-686550588 @codelipenghui yeah I think I did, in fact it works with the usual `apachepulsar/pulsar:2.6.1` image. Here's an exceprt from the deployment: ```yaml containers:

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

2020-09-03 Thread GitBox
fracasula edited a comment on issue #7682: URL: https://github.com/apache/pulsar/issues/7682#issuecomment-686550588 @codelipenghui yeah I think I did, in fact it works with the usual `apachepulsar/pulsar:2.6.1` image. Here's an excerpt from the deployment: ```yaml containers

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

2020-09-03 Thread GitBox
zymap commented on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-686562632 @harissecic @cuzyoucant Sorry for my mistake and thanks for your patience. Would you like to create a patch for the Java implementation? For the CPP client, it will be better

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

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

[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-03 Thread GitBox
aloyszhang commented on pull request #7926: URL: https://github.com/apache/pulsar/pull/7926#issuecomment-686567326 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [pulsar] aloyszhang commented on pull request #7848: take seek time into account in Reader#hasMessageAvailable

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

[GitHub] [pulsar] guyv opened a new issue #7970: pulsar-client vulnerability CVE-2019-17638

2020-09-03 Thread GitBox
guyv opened a new issue #7970: URL: https://github.com/apache/pulsar/issues/7970 Apparently pulsar-client depends on a version of jetty-util which contains known vulnerabilities Failed to execute goal org.owasp:dependency-check-maven:5.3.2:aggregate (default) on project: [ERROR]

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

2020-09-03 Thread GitBox
harissecic edited a comment on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-686447346 > Hi, @harissecic By this error `java.io.IOException: Cannot obtain authorization metadata from https://***/.well-known/openid-configuration`, it seems the client can not ac

[GitHub] [pulsar] guyv opened a new issue #7971: pulsar-client not usable in a jpms (java modules) setup

2020-09-03 Thread GitBox
guyv opened a new issue #7971: URL: https://github.com/apache/pulsar/issues/7971 pulsar-client and pulsar-api both contain the packages org.apache.pulsar.common.schema which causes a split package problem when our application is ran on the module path. ---

[GitHub] [pulsar] MarvinCai commented on pull request #7858: Add config to lazily recover cursors when recovering a managed ledger.

2020-09-03 Thread GitBox
MarvinCai commented on pull request #7858: URL: https://github.com/apache/pulsar/pull/7858#issuecomment-686592307 /pulsarbot rerun unit-tests This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [pulsar] aloyszhang removed a comment on pull request #7848: take seek time into account in Reader#hasMessageAvailable

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

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

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

[GitHub] [pulsar] 315157973 commented on pull request #7883: Support topicl level offload

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

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

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

[GitHub] [pulsar] jerrypeng merged pull request #7965: Various fixes for Batch Source

2020-09-03 Thread GitBox
jerrypeng merged pull request #7965: URL: https://github.com/apache/pulsar/pull/7965 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: Various fixes for Batch Source (#7965)

2020-09-03 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 1632dca Various fixes for Batch Source (#796

[GitHub] [pulsar] zhanghaou commented on pull request #7968: [Issue 7759] Support set Max Consumer on topic level.

2020-09-03 Thread GitBox
zhanghaou commented on pull request #7968: URL: https://github.com/apache/pulsar/pull/7968#issuecomment-686826560 @codelipenghui @jianyun8023 @jiazhai PTAL. Thanks. This is an automated message from the Apache Git Service. To

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

2020-09-03 Thread GitBox
EronWright commented on issue #7952: URL: https://github.com/apache/pulsar/issues/7952#issuecomment-686827234 Regarding the openid configuration document location, @cuzyoucant is correct that the base path of the `issuerUrl` should be preserved, to better interoperate with Keycloak and oth

[GitHub] [pulsar] codelipenghui merged pull request #7883: Support topicl level offload

2020-09-03 Thread GitBox
codelipenghui merged pull request #7883: URL: https://github.com/apache/pulsar/pull/7883 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 g

[pulsar] branch master updated (1632dca -> 62e3df3)

2020-09-03 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 1632dca Various fixes for Batch Source (#7965) add 62e3df3 Support topicl level offload (#7883) No new revis

[GitHub] [pulsar] murong00 commented on issue #7959: Force namespace deletion even if not empty

2020-09-03 Thread GitBox
murong00 commented on issue #7959: URL: https://github.com/apache/pulsar/issues/7959#issuecomment-686841696 We have implemented this feature in our dev branch and I will finish this later when available. This is an automated

[GitHub] [pulsar] sijie commented on issue #7476: Discovery service can't find active broker

2020-09-03 Thread GitBox
sijie commented on issue #7476: URL: https://github.com/apache/pulsar/issues/7476#issuecomment-686851539 @aloyszhang Can you contribute the change back? This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] rudy2steiner commented on issue #7903: AutoTopicCreationType partitioned not supported when passing through proxy

2020-09-03 Thread GitBox
rudy2steiner commented on issue #7903: URL: https://github.com/apache/pulsar/issues/7903#issuecomment-686851784 I'll try to fix it This is an automated message from the Apache Git Service. To respond to the message, please lo

[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-03 Thread GitBox
aloyszhang commented on pull request #7926: URL: https://github.com/apache/pulsar/pull/7926#issuecomment-686853317 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [pulsar] aloyszhang commented on issue #7476: Discovery service can't find active broker

2020-09-03 Thread GitBox
aloyszhang commented on issue #7476: URL: https://github.com/apache/pulsar/issues/7476#issuecomment-686855329 @sijie OK, I'll add a PR later. This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [pulsar] yuku opened a new pull request #7972: [docs] Fix wrong argument name

2020-09-03 Thread GitBox
yuku opened a new pull request #7972: URL: https://github.com/apache/pulsar/pull/7972 ### Motivation Wrong document is confusing. ### Modifications Fixed. Correct argument name is `send_timeout_millis` not `send_timeout_seconds` https://github.com/apache

[GitHub] [pulsar] sijie closed issue #4916: [Transaction] Support storing multiple metadata chunks for on transaction metadata

2020-09-03 Thread GitBox
sijie closed issue #4916: URL: https://github.com/apache/pulsar/issues/4916 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 spec

[GitHub] [pulsar] sijie commented on issue #7750: [Flaky Test] Pulsar SQL integration test

2020-09-03 Thread GitBox
sijie commented on issue #7750: URL: https://github.com/apache/pulsar/issues/7750#issuecomment-686861623 This has been fixed. This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [pulsar] sijie closed issue #7750: [Flaky Test] Pulsar SQL integration test

2020-09-03 Thread GitBox
sijie closed issue #7750: URL: https://github.com/apache/pulsar/issues/7750 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 spec

[GitHub] [pulsar] sijie commented on pull request #7710: [pulsar-functions] Fix message consume ordering issue of function

2020-09-03 Thread GitBox
sijie commented on pull request #7710: URL: https://github.com/apache/pulsar/pull/7710#issuecomment-686862251 @Tiscs Can you rebase the pull request to latest master again? This is an automated message from the Apache Git Ser

[GitHub] [pulsar] Tiscs commented on pull request #7710: [pulsar-functions] Fix message consume ordering issue of function

2020-09-03 Thread GitBox
Tiscs commented on pull request #7710: URL: https://github.com/apache/pulsar/pull/7710#issuecomment-68685 > @Tiscs Can you rebase the pull request to latest master again? @sijie Done. This is an automated message f

[GitHub] [pulsar] zymap opened a new pull request #7973: Update the issuer url in the doc

2020-09-03 Thread GitBox
zymap opened a new pull request #7973: URL: https://github.com/apache/pulsar/pull/7973 --- *Motivation* Fix the issuer url is using the token endpoint. This is an automated message from the Apache Git Servic

[GitHub] [pulsar] sijie commented on issue #7574: Set jvm options to each components

2020-09-03 Thread GitBox
sijie commented on issue #7574: URL: https://github.com/apache/pulsar/issues/7574#issuecomment-686867283 @wenjun Can you check @wolfstudy 's question? This is an automated message from the Apache Git Service. To respond t

[GitHub] [pulsar] Tiscs commented on a change in pull request #7710: [pulsar-functions] Fix message consume ordering issue of function

2020-09-03 Thread GitBox
Tiscs commented on a change in pull request #7710: URL: https://github.com/apache/pulsar/pull/7710#discussion_r483355016 ## File path: pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/instance/JavaInstanceRunnable.java ## @@ -415,28 +415,45 @@ private void s

[GitHub] [pulsar] jiazhai commented on issue #7941: Consumer can't get messages with earliest position subscription type under some circumstances

2020-09-03 Thread GitBox
jiazhai commented on issue #7941: URL: https://github.com/apache/pulsar/issues/7941#issuecomment-686874779 @codelipenghui could this be related with #7682 ? This is an automated message from the Apache Git Service. To res

[GitHub] [pulsar] codelipenghui commented on pull request #7856: [Transaction] Handle Acknowledge In The Transaction

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

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

2020-09-03 Thread GitBox
aloyszhang removed a comment on pull request #7926: URL: https://github.com/apache/pulsar/pull/7926#issuecomment-686567326 This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [pulsar] sijie commented on a change in pull request #7523: [Issue 5720][authz] - add topics authz granularity

2020-09-03 Thread GitBox
sijie commented on a change in pull request #7523: URL: https://github.com/apache/pulsar/pull/7523#discussion_r483377081 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/AuthorizationProvider.java ## @@ -489,4 +490,36 @@ default Boolean a

[GitHub] [pulsar] sijie commented on issue #7258: bookkeeper shell listcookies - a, check that the books nodes are all there, but the actual PS process is not at all

2020-09-03 Thread GitBox
sijie commented on issue #7258: URL: https://github.com/apache/pulsar/issues/7258#issuecomment-686922522 Close this issue since it has been answered. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] sijie closed issue #7258: bookkeeper shell listcookies - a, check that the books nodes are all there, but the actual PS process is not at all

2020-09-03 Thread GitBox
sijie closed issue #7258: URL: https://github.com/apache/pulsar/issues/7258 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 spec

[GitHub] [pulsar] sijie closed issue #7012: Null pointer exception when add ledger fails

2020-09-03 Thread GitBox
sijie closed issue #7012: URL: https://github.com/apache/pulsar/issues/7012 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 spec

[GitHub] [pulsar] sijie commented on issue #7012: Null pointer exception when add ledger fails

2020-09-03 Thread GitBox
sijie commented on issue #7012: URL: https://github.com/apache/pulsar/issues/7012#issuecomment-686925002 @guillaumepitel I fixed the NPE issue in 2.6.0. Please try 2.6.0+ release. This is an automated message from the Apache

[GitHub] [pulsar] lhotari commented on pull request #7937: [Issue 7878][broker]Fix NPE when acknowledge messages at the broker side

2020-09-03 Thread GitBox
lhotari commented on pull request #7937: URL: https://github.com/apache/pulsar/pull/7937#issuecomment-686925505 @Technoboy- @codelipenghui @jiazhai Hi, I was looking at the fix for the NPE. It looks like the fix would silently by-pass any operations when dispatcher is null. Wouldn't skipp

[GitHub] [pulsar] sijie commented on issue #5206: TooLongFrameException : Adjusted frame length exceeds 5253120: 369295620 - discarded => Error while producing message with TLS on standalone setup

2020-09-03 Thread GitBox
sijie commented on issue #5206: URL: https://github.com/apache/pulsar/issues/5206#issuecomment-686928186 Close this issue since it was answered This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [pulsar] gm7y8 commented on issue #6962: Pulsar Admin API / Incomplete OpenAPI Spec

2020-09-03 Thread GitBox
gm7y8 commented on issue #6962: URL: https://github.com/apache/pulsar/issues/6962#issuecomment-686928048 if no one is working on it can you assign this issue to me? This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] sijie closed issue #5206: TooLongFrameException : Adjusted frame length exceeds 5253120: 369295620 - discarded => Error while producing message with TLS on standalone setup

2020-09-03 Thread GitBox
sijie closed issue #5206: URL: https://github.com/apache/pulsar/issues/5206 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 spec

[GitHub] [pulsar] sijie commented on issue #3542: Intermittent test failure in C++ test AuthPluginTest.testTlsDetectHttpsWithHostNameValidation

2020-09-03 Thread GitBox
sijie commented on issue #3542: URL: https://github.com/apache/pulsar/issues/3542#issuecomment-686928590 Close this issue since it was reported a long time ago. This is an automated message from the Apache Git Service. To res

[GitHub] [pulsar] sijie closed issue #3542: Intermittent test failure in C++ test AuthPluginTest.testTlsDetectHttpsWithHostNameValidation

2020-09-03 Thread GitBox
sijie closed issue #3542: URL: https://github.com/apache/pulsar/issues/3542 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 spec

[GitHub] [pulsar] sijie closed issue #3490: Fail to reset-cursor on replicator subscription

2020-09-03 Thread GitBox
sijie closed issue #3490: URL: https://github.com/apache/pulsar/issues/3490 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 spec

[GitHub] [pulsar] sijie commented on issue #3490: Fail to reset-cursor on replicator subscription

2020-09-03 Thread GitBox
sijie commented on issue #3490: URL: https://github.com/apache/pulsar/issues/3490#issuecomment-686929054 Close this issue due to a lack of activities. If the problem still exists, please reopen a new issue. This is an automa

[GitHub] [pulsar] sijie closed issue #4981: [Doc] update config docs

2020-09-03 Thread GitBox
sijie closed issue #4981: URL: https://github.com/apache/pulsar/issues/4981 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 spec

  1   2   >