[pulsar] branch master updated (2bd614a -> 67adf0a)

2021-05-21 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 2bd614a feat: update website docker image (#10678) add 67adf0a [Issue 9204][Python/C++]Expose

[GitHub] [pulsar] codelipenghui merged pull request #10243: [Issue 9204][Python/C++]Expose replicateSubscriptionState setting for python/c++ consumer

2021-05-21 Thread GitBox
codelipenghui merged pull request #10243: URL: https://github.com/apache/pulsar/pull/10243 -- 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. For queries about this

[GitHub] [pulsar] codelipenghui closed issue #9204: [Python Client] Expose "replicateSubscriptionState" setting for python consumer

2021-05-21 Thread GitBox
codelipenghui closed issue #9204: URL: https://github.com/apache/pulsar/issues/9204 -- 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. For queries about this service,

[pulsar] branch master updated (f807fed -> 2bd614a)

2021-05-21 Thread guangning
This is an automated email from the ASF dual-hosted git repository. guangning pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from f807fed Issue 6478: Fix the misleading setting in presto configuration (#8549) add 2bd614a feat: update

[GitHub] [pulsar] tuteng merged pull request #10678: feat: update website docker image(upgrade node version)

2021-05-21 Thread GitBox
tuteng merged pull request #10678: URL: https://github.com/apache/pulsar/pull/10678 -- 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. For queries about this service,

[GitHub] [pulsar] tuteng closed issue #10676: Node version compatibility error

2021-05-21 Thread GitBox
tuteng closed issue #10676: URL: https://github.com/apache/pulsar/issues/10676 -- 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. For queries about this service, please

[GitHub] [pulsar] urfreespace opened a new pull request #10678: feat: update website docker image(upgrade node version)

2021-05-21 Thread GitBox
urfreespace opened a new pull request #10678: URL: https://github.com/apache/pulsar/pull/10678 Fix: #10676 Node version has been upgraded from 10.24.0 to 14.17.0 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] fantapsody opened a new pull request #10677: [broker] Setup transaction metadata with metadata store

2021-05-21 Thread GitBox
fantapsody opened a new pull request #10677: URL: https://github.com/apache/pulsar/pull/10677 ### Motivation Refactor the `initialize-transaction-coordinator-metadata` command to use the new `MetadataStore` API. ### Modifications - Refactor the

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

2021-05-21 Thread GitBox
rudy2steiner commented on issue #7903: URL: https://github.com/apache/pulsar/issues/7903#issuecomment-846348093 > Would this PR fix this issue also? > #10449 yes -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] Jennifer88huang closed pull request #6757: [#6748][sql] Fix is misleading in presto configuration

2021-05-21 Thread GitBox
Jennifer88huang closed pull request #6757: URL: https://github.com/apache/pulsar/pull/6757 -- 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. For queries about this

[GitHub] [pulsar] Jennifer88huang commented on pull request #6757: [#6748][sql] Fix is misleading in presto configuration

2021-05-21 Thread GitBox
Jennifer88huang commented on pull request #6757: URL: https://github.com/apache/pulsar/pull/6757#issuecomment-846344140 Since it's fixed in https://github.com/apache/pulsar/pull/8549, I assume we can close this PR. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] urfreespace closed pull request #10675: Fix: Node version has been upgraded from 10.24.0 to 14.17.0

2021-05-21 Thread GitBox
urfreespace closed pull request #10675: URL: https://github.com/apache/pulsar/pull/10675 -- 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. For queries about this service,

[GitHub] [pulsar] urfreespace opened a new issue #10676: Node version compatibility error

2021-05-21 Thread GitBox
urfreespace opened a new issue #10676: URL: https://github.com/apache/pulsar/issues/10676 Node version compatibility error: https://github.com/apache/pulsar/runs/2643679216?check_suite_focus=true -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] urfreespace opened a new pull request #10675: feat: Node version has been upgraded from 10.24.0 to 14.17.0

2021-05-21 Thread GitBox
urfreespace opened a new pull request #10675: URL: https://github.com/apache/pulsar/pull/10675 Fix: https://github.com/apache/pulsar/runs/2643679216?check_suite_focus=true -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] codelipenghui commented on pull request #10254: [Broker] Remove possible error case for system topic name checking

2021-05-21 Thread GitBox
codelipenghui commented on pull request #10254: URL: https://github.com/apache/pulsar/pull/10254#issuecomment-846326365 Hi @michaeljmarshall, Could you please help confirm if https://github.com/apache/pulsar/pull/10529 has fixed the problem? Sorry, sorry we missed your PR. -- This is

[pulsar] branch master updated (bad185a -> f807fed)

2021-05-21 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 bad185a [Issue-10611] consumer related topic stats only available while consumer or reader are connected (#10644)

[GitHub] [pulsar] codelipenghui merged pull request #8549: Issue 6478: Fix the misleading setting in presto configuration

2021-05-21 Thread GitBox
codelipenghui merged pull request #8549: URL: https://github.com/apache/pulsar/pull/8549 -- 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. For queries about this service,

[GitHub] [pulsar] codelipenghui closed issue #6748: `pulsar.broker-service-url` is misleading in presto configuration

2021-05-21 Thread GitBox
codelipenghui closed issue #6748: URL: https://github.com/apache/pulsar/issues/6748 -- 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. For queries about this service,

[GitHub] [pulsar] jerrypeng commented on pull request #10667: Fix issues in advanceNonDurableCursors

2021-05-21 Thread GitBox
jerrypeng commented on pull request #10667: URL: https://github.com/apache/pulsar/pull/10667#issuecomment-846310838 @eolivelli yes in general but don't have a test to replicate this specific issue. Attempted to replicate issue in test but no a clear way how to -- This is an automated

[GitHub] [pulsar] merlimat opened a new pull request #10674: Revert "Creating a topic does not wait for creating cursor of replicators"

2021-05-21 Thread GitBox
merlimat opened a new pull request #10674: URL: https://github.com/apache/pulsar/pull/10674 This reverts #6364 commit 336e971f4d41d6ffb26b3b53a20f36a360c070e8. ### Motivation There are few issues that were introduced in that commit: 1. If a new replication cursor

[GitHub] [pulsar] rdhabalia commented on issue #10654: [Performance] DimensionStats implementation uses inefficient Prometheus client method CollectorRegistry.getSampleValue which is only intended for

2021-05-21 Thread GitBox
rdhabalia commented on issue #10654: URL: https://github.com/apache/pulsar/issues/10654#issuecomment-846298941 Yes, `CollectorRegistry::getSampleValue()` can be inefficient and we can improve it but just wanted to understand performance impact. It is used to generate Prometheus stats and

[pulsar-client-go] branch master updated (320b42c -> 6607ef8)

2021-05-21 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-client-go.git. from 320b42c update jwt-go dependency to resolve vulnerabilities (#524) add 6607ef8 Fixed Athenz

[GitHub] [pulsar-client-go] merlimat merged pull request #522: Fixed Athenz repository name

2021-05-21 Thread GitBox
merlimat merged pull request #522: URL: https://github.com/apache/pulsar-client-go/pull/522 -- 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. For queries about this

[GitHub] [pulsar] david-streamlio removed a comment on pull request #10593: [Issue-10109] [admin client] Add --batch-source-config switch to the Pulsar Admin Source API

2021-05-21 Thread GitBox
david-streamlio removed a comment on pull request #10593: URL: https://github.com/apache/pulsar/pull/10593#issuecomment-843289624 /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

[GitHub] [pulsar] david-streamlio removed a comment on pull request #10593: [Issue-10109] [admin client] Add --batch-source-config switch to the Pulsar Admin Source API

2021-05-21 Thread GitBox
david-streamlio removed a comment on pull request #10593: URL: https://github.com/apache/pulsar/pull/10593#issuecomment-846156058 /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

[GitHub] [pulsar] david-streamlio commented on pull request #10593: [Issue-10109] [admin client] Add --batch-source-config switch to the Pulsar Admin Source API

2021-05-21 Thread GitBox
david-streamlio commented on pull request #10593: URL: https://github.com/apache/pulsar/pull/10593#issuecomment-846278194 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] complone commented on pull request #10544: [plusar-10479] Resolved Allow Java client to create consumer with rec…

2021-05-21 Thread GitBox
complone commented on pull request #10544: URL: https://github.com/apache/pulsar/pull/10544#issuecomment-846273832 Good two, this function has been refactored, can you take time to have a look? @linlinnn @zymap -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] complone commented on a change in pull request #10544: [plusar-10479] Resolved Allow Java client to create consumer with rec…

2021-05-21 Thread GitBox
complone commented on a change in pull request #10544: URL: https://github.com/apache/pulsar/pull/10544#discussion_r637244804 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerBase.java ## @@ -172,8 +172,17 @@ protected

[GitHub] [pulsar] anvinjain commented on issue #10673: update-partitioned-topic fails for newly created geo-replicated topic

2021-05-21 Thread GitBox
anvinjain commented on issue #10673: URL: https://github.com/apache/pulsar/issues/10673#issuecomment-846200321 **Full stacktrace in broker logs** ``` 18:56:12.405 [AsyncHttpClient-60-1] WARN org.apache.pulsar.broker.admin.impl.PersistentTopicsBase - [AUTHN/viesti-super-user-1]

[GitHub] [pulsar] anvinjain opened a new issue #10673: update-partitioned-topic fails for newly created geo-replicated topic

2021-05-21 Thread GitBox
anvinjain opened a new issue #10673: URL: https://github.com/apache/pulsar/issues/10673 **Describe the bug** If a partitioned topic is created from one cluster, C1 where namespace cluster-list=(C1,C2), then update-partitioned-topic fails from remote cluster(C2) unless produce/consume

[GitHub] [pulsar] merlimat commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
merlimat commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637120866 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupConfigListener.java ## @@ -0,0 +1,137 @@ +/** + *

[GitHub] [pulsar] ravi-vaidyanathan commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
ravi-vaidyanathan commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637147345 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroup.java ## @@ -70,9 +70,10 @@ // Default

[GitHub] [pulsar] bharanic-dev commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
bharanic-dev commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637140932 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceUsageTransportManagerTest.java ## @@ -113,8 +113,8 @@

[GitHub] [pulsar] bharanic-dev commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
bharanic-dev commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637140554 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java ## @@ -539,29 +549,32 @@ private void

[GitHub] [pulsar] bharanic-dev commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
bharanic-dev commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637140032 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java ## @@ -117,19 +122,24 @@ public

[GitHub] [pulsar] bharanic-dev commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
bharanic-dev commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637138909 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java ## @@ -539,29 +549,32 @@ private void

[GitHub] [pulsar] bharanic-dev commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
bharanic-dev commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637135851 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroup.java ## @@ -70,9 +70,10 @@ // Default ctor:

[GitHub] [pulsar] david-streamlio commented on pull request #10593: [Issue-10109] [admin client] Add --batch-source-config switch to the Pulsar Admin Source API

2021-05-21 Thread GitBox
david-streamlio commented on pull request #10593: URL: https://github.com/apache/pulsar/pull/10593#issuecomment-846156058 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
kaushik-develop commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637123495 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/resourcegroup/ResourceUsageTransportManagerTest.java ## @@ -113,8 +113,8 @@

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
kaushik-develop commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637121030 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java ## @@ -117,19 +122,24 @@ public

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
kaushik-develop commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637122625 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java ## @@ -539,29 +549,32 @@ private

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
kaushik-develop commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637121030 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java ## @@ -117,19 +122,24 @@ public

[GitHub] [pulsar] cckellogg commented on a change in pull request #10593: [Issue-10109] [admin client] Add --batch-source-config switch to the Pulsar Admin Source API

2021-05-21 Thread GitBox
cckellogg commented on a change in pull request #10593: URL: https://github.com/apache/pulsar/pull/10593#discussion_r637120933 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdSources.java ## @@ -315,6 +316,8 @@ protected void

[pulsar-client-go] branch master updated: update jwt-go dependency to resolve vulnerabilities (#524)

2021-05-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 320b42c update jwt-go dependency to

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
kaushik-develop commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637118409 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroupService.java ## @@ -539,29 +549,32 @@ private

[GitHub] [pulsar-client-go] merlimat merged pull request #524: update jwt-go dependency to resolve vulnerabilities

2021-05-21 Thread GitBox
merlimat merged pull request #524: URL: https://github.com/apache/pulsar-client-go/pull/524 -- 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. For queries about this

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
kaushik-develop commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637115027 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroup.java ## @@ -179,7 +181,7 @@ protected

[GitHub] [pulsar] kaushik-develop commented on a change in pull request #10657: [PIP-82] [pulsar-broker] Add resource-group configuration listener.

2021-05-21 Thread GitBox
kaushik-develop commented on a change in pull request #10657: URL: https://github.com/apache/pulsar/pull/10657#discussion_r637114635 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/resourcegroup/ResourceGroup.java ## @@ -70,9 +70,10 @@ // Default

[GitHub] [pulsar-client-go] merlimat merged pull request #523: Update changelog to 0.5.0

2021-05-21 Thread GitBox
merlimat merged pull request #523: URL: https://github.com/apache/pulsar-client-go/pull/523 -- 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. For queries about this

[pulsar-client-go] branch master updated (af9d58e -> dd7cc9a)

2021-05-21 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-client-go.git. from af9d58e Always check connection close channell, before attempting to put requests (#521) add

[GitHub] [pulsar] gaoran10 commented on a change in pull request #10604: [Schema] Support consume multiple schema types messages by AutoConsumeSchema

2021-05-21 Thread GitBox
gaoran10 commented on a change in pull request #10604: URL: https://github.com/apache/pulsar/pull/10604#discussion_r637068217 ## File path: pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/LookupProxyHandler.java ## @@ -357,6 +358,11 @@ public void

[GitHub] [pulsar-client-go] xdingsplk opened a new pull request #524: update jwt-go dependency to resolve vulnerabilities

2021-05-21 Thread GitBox
xdingsplk opened a new pull request #524: URL: https://github.com/apache/pulsar-client-go/pull/524 <-- ### Contribution Checklist - Name the pull request in the form "[Issue XYZ][component] Title of the pull request", where *XYZ* should be replaced by the actual issue number.

[GitHub] [pulsar] eolivelli opened a new pull request #10672: Branch-2.7: getListOfNamespaces does not fail with non-existant tenant

2021-05-21 Thread GitBox
eolivelli opened a new pull request #10672: URL: https://github.com/apache/pulsar/pull/10672 on branch-2.7 NamespacesTest.getNamespaces fails ``` java.lang.AssertionError: should have failed at org.testng.Assert.fail(Assert.java:99) at

[GitHub] [pulsar] david-streamlio commented on a change in pull request #10593: [Issue-10109] [admin client] Add --batch-source-config switch to the Pulsar Admin Source API

2021-05-21 Thread GitBox
david-streamlio commented on a change in pull request #10593: URL: https://github.com/apache/pulsar/pull/10593#discussion_r637028453 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdSources.java ## @@ -315,6 +316,8 @@ protected void

[GitHub] [pulsar] flowchartsman commented on issue #10560: [Go Functions] support state storage API

2021-05-21 Thread GitBox
flowchartsman commented on issue #10560: URL: https://github.com/apache/pulsar/issues/10560#issuecomment-846050108 So you're saying this needs a dependent issue for pulsar-client-go to support bookeeper interaction? If so, I can file one. -- This is an automated message from the Apache

[GitHub] [pulsar] eolivelli commented on pull request #10604: [Schema] Support consume multiple schema types messages by AutoConsumeSchema

2021-05-21 Thread GitBox
eolivelli commented on pull request #10604: URL: https://github.com/apache/pulsar/pull/10604#issuecomment-846000321 @gaoran10 I would postpone fixing those problems to a follow up patch this patch is already quite big. -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar] eolivelli commented on a change in pull request #10604: [Schema] Support consume multiple schema types messages by AutoConsumeSchema

2021-05-21 Thread GitBox
eolivelli commented on a change in pull request #10604: URL: https://github.com/apache/pulsar/pull/10604#discussion_r636976987 ## File path: pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/LookupProxyHandler.java ## @@ -357,6 +358,11 @@ public void

[GitHub] [pulsar] megfigura opened a new issue #10671: Race condition with Reader.seek()

2021-05-21 Thread GitBox
megfigura opened a new issue #10671: URL: https://github.com/apache/pulsar/issues/10671 **Describe the bug** Consuming a message with the Reader interface immediately after seeking can consume the wrong message. This only seems to happen in a pretty specific situation (details below).

[GitHub] [pulsar] codelipenghui closed issue #7378: [docs] Add docs for "Dispatch batch messages according consumer permits"

2021-05-21 Thread GitBox
codelipenghui closed issue #7378: URL: https://github.com/apache/pulsar/issues/7378 -- 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. For queries about this service,

[GitHub] [pulsar] codelipenghui commented on pull request #7042: [Issue 6957][pulsar-io-debezium] Support CDC Connector for SqlServer

2021-05-21 Thread GitBox
codelipenghui commented on pull request #7042: URL: https://github.com/apache/pulsar/pull/7042#issuecomment-845967447 @huangdx0726 Could you please resolve the conflicts? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] codelipenghui commented on issue #7378: [docs] Add docs for "Dispatch batch messages according consumer permits"

2021-05-21 Thread GitBox
codelipenghui commented on issue #7378: URL: https://github.com/apache/pulsar/issues/7378#issuecomment-845968935 @Jennifer88huang Seems we don't need to add a document for #7266 Since the PR does not introduce any new features and configurations, so I will close it first, feel free to

[GitHub] [pulsar] gaoran10 commented on pull request #10604: [Schema] Support consume multiple schema types messages by AutoConsumeSchema

2021-05-21 Thread GitBox
gaoran10 commented on pull request #10604: URL: https://github.com/apache/pulsar/pull/10604#issuecomment-845954875 There are still two problems that need to be resolved when consuming keyValue schema data. 1. The keyValue schema data encoding does not consider the

[GitHub] [pulsar] codelipenghui closed issue #2664: PIP-31: Transactional Streaming

2021-05-21 Thread GitBox
codelipenghui closed issue #2664: URL: https://github.com/apache/pulsar/issues/2664 -- 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. For queries about this service,

[GitHub] [pulsar] codelipenghui commented on issue #2664: PIP-31: Transactional Streaming

2021-05-21 Thread GitBox
codelipenghui commented on issue #2664: URL: https://github.com/apache/pulsar/issues/2664#issuecomment-845953706 Close this issue, #8633 is tracking the remaining tasks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[pulsar] branch master updated (85aee95 -> bad185a)

2021-05-21 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 85aee95 [Doc] Add descriptions for broker (#10605) add bad185a [Issue-10611] consumer related topic stats

[GitHub] [pulsar] codelipenghui merged pull request #10644: [Issue-10611] consumer related topic stats only available while consumer or reader are connected

2021-05-21 Thread GitBox
codelipenghui merged pull request #10644: URL: https://github.com/apache/pulsar/pull/10644 -- 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. For queries about this

[GitHub] [pulsar] codelipenghui closed issue #10611: consumer related topic stats only available while consumer or reader are connected

2021-05-21 Thread GitBox
codelipenghui closed issue #10611: URL: https://github.com/apache/pulsar/issues/10611 -- 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. For queries about this service,

[GitHub] [pulsar-client-go] cckellogg commented on pull request #522: Fixed Athenz repository name

2021-05-21 Thread GitBox
cckellogg commented on pull request #522: URL: https://github.com/apache/pulsar-client-go/pull/522#issuecomment-845948611 I think the test file needs updating too https://github.com/apache/pulsar-client-go/blob/master/pulsar/internal/auth/athenz_test.go -- This is an automated

[GitHub] [pulsar] merlimat closed pull request #2602: ManagedLedger should share BK worker pool threads

2021-05-21 Thread GitBox
merlimat closed pull request #2602: URL: https://github.com/apache/pulsar/pull/2602 -- 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. For queries about this service,

[GitHub] [pulsar] cckellogg commented on a change in pull request #10593: [Issue-10109] [admin client] Add --batch-source-config switch to the Pulsar Admin Source API

2021-05-21 Thread GitBox
cckellogg commented on a change in pull request #10593: URL: https://github.com/apache/pulsar/pull/10593#discussion_r636913174 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdSources.java ## @@ -315,6 +316,8 @@ protected void

[GitHub] [pulsar] codelipenghui commented on pull request #10593: [Issue-10109] [admin client] Add --batch-source-config switch to the Pulsar Admin Source API

2021-05-21 Thread GitBox
codelipenghui commented on pull request #10593: URL: https://github.com/apache/pulsar/pull/10593#issuecomment-845943265 @nlu90 Please help review this PR. -- 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 (c9d6b1e -> 85aee95)

2021-05-21 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 c9d6b1e Fix website build CI (#10668) add 85aee95 [Doc] Add descriptions for broker (#10605) No new

[GitHub] [pulsar] codelipenghui closed issue #10311: [Documentation] Add documentation about Pulsar Broker 64-bit JVM requirement

2021-05-21 Thread GitBox
codelipenghui closed issue #10311: URL: https://github.com/apache/pulsar/issues/10311 -- 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. For queries about this service,

[pulsar] branch branch-2.7 updated: Branch 2.7: TopicPoliciesTest.testMaxSubscriptionsFailFast fails (#10640)

2021-05-21 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.7 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.7 by this push: new a19db1d Branch 2.7:

[GitHub] [pulsar] codelipenghui merged pull request #10605: [Doc] Add descriptions for broker

2021-05-21 Thread GitBox
codelipenghui merged pull request #10605: URL: https://github.com/apache/pulsar/pull/10605 -- 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. For queries about this

[GitHub] [pulsar] codelipenghui merged pull request #10640: Branch 2.7: TopicPoliciesTest.testMaxSubscriptionsFailFast fails

2021-05-21 Thread GitBox
codelipenghui merged pull request #10640: URL: https://github.com/apache/pulsar/pull/10640 -- 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. For queries about this

[GitHub] [pulsar] eolivelli commented on pull request #10668: Fix website build failure caused by python doc generate script

2021-05-21 Thread GitBox
eolivelli commented on pull request #10668: URL: https://github.com/apache/pulsar/pull/10668#issuecomment-845938953 great -- 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.

[pulsar] branch master updated (5497107 -> c9d6b1e)

2021-05-21 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 5497107 [api][pulsar-client] make AuthenticationToken serializable (#10574) add c9d6b1e Fix website build

[GitHub] [pulsar] codelipenghui merged pull request #10668: Fix website build failure caused by python doc generate script

2021-05-21 Thread GitBox
codelipenghui merged pull request #10668: URL: https://github.com/apache/pulsar/pull/10668 -- 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. For queries about this

[pulsar] branch master updated (a672fc1 -> 5497107)

2021-05-21 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 a672fc1 Setup initial namespaces with MetadataStore (#10612) add 5497107 [api][pulsar-client] make

[GitHub] [pulsar] codelipenghui merged pull request #10574: [Issue 905][api][pulsar-client] add serializable token class

2021-05-21 Thread GitBox
codelipenghui merged pull request #10574: URL: https://github.com/apache/pulsar/pull/10574 -- 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. For queries about this

[pulsar] branch master updated (047fb6a -> a672fc1)

2021-05-21 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 047fb6a [functions] Add resource granularities settings for functions (#9736) add a672fc1 Setup initial

[GitHub] [pulsar] eolivelli merged pull request #10612: Setup initial namespaces with MetadataStore

2021-05-21 Thread GitBox
eolivelli merged pull request #10612: URL: https://github.com/apache/pulsar/pull/10612 -- 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. For queries about this service,

[GitHub] [pulsar] RobertIndie commented on pull request #10243: [Issue 9204][Python/C++]Expose replicateSubscriptionState setting for python/c++ consumer

2021-05-21 Thread GitBox
RobertIndie commented on pull request #10243: URL: https://github.com/apache/pulsar/pull/10243#issuecomment-845935033 /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 (d700f8d -> 047fb6a)

2021-05-21 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 d700f8d [Issue 10161] Fix missing LoggerFactoryPtr type. (#10164) add 047fb6a [functions] Add resource

[GitHub] [pulsar] codelipenghui merged pull request #9736: [functions] Add resource granularities settings for functions

2021-05-21 Thread GitBox
codelipenghui merged pull request #9736: URL: https://github.com/apache/pulsar/pull/9736 -- 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. For queries about this service,

[GitHub] [pulsar] zymap commented on issue #10636: Within EKS to offload a topic to S3 got AuthorizationException

2021-05-21 Thread GitBox
zymap commented on issue #10636: URL: https://github.com/apache/pulsar/issues/10636#issuecomment-845928642 How did you deploy pulsar to EKS? Do you have any configuration files? Do you deploy the cluster with helm charts? -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] astifter commented on pull request #10164: [Issue 10161] Fix missing LoggerFactoryPtr type.

2021-05-21 Thread GitBox
astifter commented on pull request #10164: URL: https://github.com/apache/pulsar/pull/10164#issuecomment-845927448 > I misunderstood, the name of the branch on your repo is "branch-2.7" but the target branch is "master" Yes, sorry about that. I was working of the 2.7 branch and

[pulsar] branch master updated (2be05fb -> d700f8d)

2021-05-21 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 2be05fb Nack support in WS (#8249) add d700f8d [Issue 10161] Fix missing LoggerFactoryPtr type. (#10164) No

[GitHub] [pulsar] codelipenghui edited a comment on pull request #8611: [Issue 8599] Fix DispatchRateLimiter does not take effect

2021-05-21 Thread GitBox
codelipenghui edited a comment on pull request #8611: URL: https://github.com/apache/pulsar/pull/8611#issuecomment-845923286 @lhotari Good point, Could you please open a new issue for your last comment? This PR fixes the bug of the current implementation, It's better to use a separate

[GitHub] [pulsar] codelipenghui merged pull request #10164: [Issue 10161] Fix missing LoggerFactoryPtr type.

2021-05-21 Thread GitBox
codelipenghui merged pull request #10164: URL: https://github.com/apache/pulsar/pull/10164 -- 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. For queries about this

[GitHub] [pulsar] codelipenghui closed issue #10161: Pulsar 2.7.1 doesn't build with USE_LOG4CXX=ON

2021-05-21 Thread GitBox
codelipenghui closed issue #10161: URL: https://github.com/apache/pulsar/issues/10161 -- 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. For queries about this service,

[GitHub] [pulsar] eolivelli commented on pull request #10164: [Issue 10161] Fix missing LoggerFactoryPtr type.

2021-05-21 Thread GitBox
eolivelli commented on pull request #10164: URL: https://github.com/apache/pulsar/pull/10164#issuecomment-845926069 I misunderstood, the name of the branch on your repo is "branch-2.7" but the target branch is "master" -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] codelipenghui commented on pull request #8611: [Issue 8599] Fix DispatchRateLimiter does not take effect

2021-05-21 Thread GitBox
codelipenghui commented on pull request #8611: URL: https://github.com/apache/pulsar/pull/8611#issuecomment-845923286 @lhotari Could you please open a new issue for your last comment? This PR fixes the bug of the current implementation, It's better to use a separate issue to track the new

[pulsar] branch master updated (87a115e -> 2be05fb)

2021-05-21 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 87a115e [Transaction] Fix transaction end op retry problem. (#10564) add 2be05fb Nack support in WS (#8249)

[GitHub] [pulsar] codelipenghui merged pull request #8249: Nack support in WS

2021-05-21 Thread GitBox
codelipenghui merged pull request #8249: URL: https://github.com/apache/pulsar/pull/8249 -- 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. For queries about this service,

[GitHub] [pulsar] codelipenghui closed issue #8247: Nack support in WS

2021-05-21 Thread GitBox
codelipenghui closed issue #8247: URL: https://github.com/apache/pulsar/issues/8247 -- 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. For queries about this service,

[GitHub] [pulsar] BewareMyPower commented on pull request #10164: [Issue 10161] Fix missing LoggerFactoryPtr type.

2021-05-21 Thread GitBox
BewareMyPower commented on pull request #10164: URL: https://github.com/apache/pulsar/pull/10164#issuecomment-845883677 LGTM. This bug was not exposed because CI only build cpp client with `USE_LOG4CXX=OFF`. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] astifter commented on pull request #10164: [Issue 10161] Fix missing LoggerFactoryPtr type.

2021-05-21 Thread GitBox
astifter commented on pull request #10164: URL: https://github.com/apache/pulsar/pull/10164#issuecomment-845877376 > @astifter thanks for your code work. For this PR, do we need to update docs? No, this just fixes the compile. -- This is an automated message from the

[pulsar] branch master updated (a6bb983 -> 87a115e)

2021-05-21 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 a6bb983 [Performance] Use single instance of parser (#10664) add 87a115e [Transaction] Fix transaction end

  1   2   >