[GitHub] [pulsar] MarvinCai commented on pull request #12720: [Pulsar SQL] Support query chunked messages feature in Pulsar SQL

2021-11-22 Thread GitBox
MarvinCai commented on pull request #12720: URL: https://github.com/apache/pulsar/pull/12720#issuecomment-976234384 /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 to

[GitHub] [pulsar] lhotari commented on issue #12907: Pulsar in AWS ARM processor group unable to start cluster

2021-11-22 Thread GitBox
lhotari commented on issue #12907: URL: https://github.com/apache/pulsar/issues/12907#issuecomment-976230994 > @lhotari This issue seems caused by introduce conscrypt-openjdk in #10541 Is there any problem just removing conscrypt-openjdk uber 2.5.2 jar to run pulsar in ARM env? @wan

[pulsar] branch master updated (c22192f -> 92009de)

2021-11-22 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from c22192f [Doc] add explanations for permissions (#12920) add 92009de [Doc] add explanations for REST (#12918) N

[GitHub] [pulsar] Anonymitaet merged pull request #12918: [Doc] add explanations for REST

2021-11-22 Thread GitBox
Anonymitaet merged pull request #12918: URL: https://github.com/apache/pulsar/pull/12918 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-u

[GitHub] [pulsar] lhotari commented on a change in pull request #12343: fix lastCumulativeAck.messageId npe when PersistentAcknowledgmentsGroupingTracker.flushSync

2021-11-22 Thread GitBox
lhotari commented on a change in pull request #12343: URL: https://github.com/apache/pulsar/pull/12343#discussion_r754857401 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java ## @@ -477,13 +477,16 @@ public vo

[GitHub] [pulsar] ericsyh commented on pull request #12927: feat(cli): support sql-worker in pulsar-daemon

2021-11-22 Thread GitBox
ericsyh commented on pull request #12927: URL: https://github.com/apache/pulsar/pull/12927#issuecomment-976224810 Sure, i will update the doc. But seems i need some help on the CI, my changes won't impact on broker UT. -- This is an automated message from the Apache Git Service. To respo

[pulsar] branch master updated (af1a82d -> c22192f)

2021-11-22 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from af1a82d Apply executable file permissions to py scripts. (#12852) add c22192f [Doc] add explanations for permis

[GitHub] [pulsar] Anonymitaet merged pull request #12920: [Doc] add explanations for permissions

2021-11-22 Thread GitBox
Anonymitaet merged pull request #12920: URL: https://github.com/apache/pulsar/pull/12920 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-u

[GitHub] [pulsar] lhotari commented on a change in pull request #12343: fix lastCumulativeAck.messageId npe when PersistentAcknowledgmentsGroupingTracker.flushSync

2021-11-22 Thread GitBox
lhotari commented on a change in pull request #12343: URL: https://github.com/apache/pulsar/pull/12343#discussion_r754854259 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java ## @@ -477,13 +477,16 @@ public vo

[GitHub] [pulsar] gaozhangmin commented on a change in pull request #12343: fix lastCumulativeAck.messageId npe when PersistentAcknowledgmentsGroupingTracker.flushSync

2021-11-22 Thread GitBox
gaozhangmin commented on a change in pull request #12343: URL: https://github.com/apache/pulsar/pull/12343#discussion_r754853183 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java ## @@ -477,13 +477,16 @@ publi

[GitHub] [pulsar] Jason918 commented on a change in pull request #12930: [Issue 12929] [pulsar-metadata] Fix metadata store deadlock

2021-11-22 Thread GitBox
Jason918 commented on a change in pull request #12930: URL: https://github.com/apache/pulsar/pull/12930#discussion_r754848783 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -1434,7 +1434,7 @@ protected void interna

[GitHub] [pulsar] Jason918 commented on pull request #12896: [Issue 12894] Only refresh metadata if path is already in cache after write.

2021-11-22 Thread GitBox
Jason918 commented on pull request #12896: URL: https://github.com/apache/pulsar/pull/12896#issuecomment-976213983 @315157973 Here is the fix of #12909, PTAL -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [pulsar] Jason918 commented on issue #12909: Metadata Deserialization failed

2021-11-22 Thread GitBox
Jason918 commented on issue #12909: URL: https://github.com/apache/pulsar/issues/12909#issuecomment-976213575 Duplicate of #12894 -- 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 com

[GitHub] [pulsar] lhotari commented on a change in pull request #12343: fix lastCumulativeAck.messageId npe when PersistentAcknowledgmentsGroupingTracker.flushSync

2021-11-22 Thread GitBox
lhotari commented on a change in pull request #12343: URL: https://github.com/apache/pulsar/pull/12343#discussion_r754842709 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java ## @@ -477,13 +477,16 @@ public vo

[GitHub] [pulsar] wangjialing218 commented on issue #12907: Pulsar in AWS ARM processor group unable to start cluster

2021-11-22 Thread GitBox
wangjialing218 commented on issue #12907: URL: https://github.com/apache/pulsar/issues/12907#issuecomment-976210741 @lhotari This issue seems caused by introduce conscrypt-openjdk in #10541 Is there any problem just removing conscrypt-openjdk uber 2.5.2 jar to run pulsar in ARM env? --

[GitHub] [pulsar] Shoothzj commented on pull request #12882: Remove Unused variable init and unused method In AdminResource Since 2018

2021-11-22 Thread GitBox
Shoothzj commented on pull request #12882: URL: https://github.com/apache/pulsar/pull/12882#issuecomment-976210321 /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 to t

[GitHub] [pulsar] Shoothzj removed a comment on pull request #12937: Fix Classloader Leak When register source/sink

2021-11-22 Thread GitBox
Shoothzj removed a comment on pull request #12937: URL: https://github.com/apache/pulsar/pull/12937#issuecomment-976152320 /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] Shoothzj commented on pull request #12937: Fix Classloader Leak When register source/sink

2021-11-22 Thread GitBox
Shoothzj commented on pull request #12937: URL: https://github.com/apache/pulsar/pull/12937#issuecomment-976198307 /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 to t

[GitHub] [pulsar] Shoothzj commented on pull request #12882: Remove Unused variable init and unused method In AdminResource Since 2018

2021-11-22 Thread GitBox
Shoothzj commented on pull request #12882: URL: https://github.com/apache/pulsar/pull/12882#issuecomment-976194738 /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 to t

[GitHub] [pulsar] MarvinCai commented on pull request #12720: [Pulsar SQL] Support query chunked messages feature in Pulsar SQL

2021-11-22 Thread GitBox
MarvinCai commented on pull request #12720: URL: https://github.com/apache/pulsar/pull/12720#issuecomment-976190477 /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 to

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #12902: [pulsar-broker] add uniform load shedder strategy to distribute traffic uniformly across brokers

2021-11-22 Thread GitBox
Anonymitaet commented on a change in pull request #12902: URL: https://github.com/apache/pulsar/pull/12902#discussion_r754793314 ## File path: conf/broker.conf ## @@ -1097,6 +1097,18 @@ loadBalancerLoadSheddingStrategy=org.apache.pulsar.broker.loadbalance.impl.Overl # It only

[GitHub] [pulsar] Shoothzj commented on pull request #12937: Fix Classloader Leak When register source/sink

2021-11-22 Thread GitBox
Shoothzj commented on pull request #12937: URL: https://github.com/apache/pulsar/pull/12937#issuecomment-976152320 /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 to t

[GitHub] [pulsar] casuallc removed a comment on pull request #12396: Fix NPE when create OpEntry

2021-11-22 Thread GitBox
casuallc removed a comment on pull request #12396: URL: https://github.com/apache/pulsar/pull/12396#issuecomment-965951472 -- 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. T

[GitHub] [pulsar] casuallc commented on pull request #12396: Fix NPE when create OpEntry

2021-11-22 Thread GitBox
casuallc commented on pull request #12396: URL: https://github.com/apache/pulsar/pull/12396#issuecomment-976149803 /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 to t

[GitHub] [pulsar] suiyuzeng commented on pull request #12870: replace deprecated function of TopicPoliciesTest

2021-11-22 Thread GitBox
suiyuzeng commented on pull request #12870: URL: https://github.com/apache/pulsar/pull/12870#issuecomment-976142154 /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 to

[GitHub] [pulsar] Anonymitaet commented on pull request #12927: feat(cli): support sql-worker in pulsar-daemon

2021-11-22 Thread GitBox
Anonymitaet commented on pull request #12927: URL: https://github.com/apache/pulsar/pull/12927#issuecomment-976134131 @ericsyh thanks for your explanations! Then can you update docs [here](https://pulsar.apache.org/docs/en/next/reference-cli-tools/#pulsar) and [here](https://pulsar.apache.

[GitHub] [pulsar] 315157973 commented on a change in pull request #12869: [PIP-105] Part-1 Support add subscription properties

2021-11-22 Thread GitBox
315157973 commented on a change in pull request #12869: URL: https://github.com/apache/pulsar/pull/12869#discussion_r754780780 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/CreateSubscriptionTest.java ## @@ -127,4 +139,86 @@ public void createSubscr

[GitHub] [pulsar] casuallc commented on pull request #12396: Fix NPE when create OpEntry

2021-11-22 Thread GitBox
casuallc commented on pull request #12396: URL: https://github.com/apache/pulsar/pull/12396#issuecomment-976126005 /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 to t

[GitHub] [pulsar] suiyuzeng commented on pull request #12870: replace deprecated function of TopicPoliciesTest

2021-11-22 Thread GitBox
suiyuzeng commented on pull request #12870: URL: https://github.com/apache/pulsar/pull/12870#issuecomment-976119137 > @suiyuzeng Could you please help resolve the conflicts? done -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [pulsar] Shoothzj opened a new pull request #12937: Fix Classloader Leak When register source/sink

2021-11-22 Thread GitBox
Shoothzj opened a new pull request #12937: URL: https://github.com/apache/pulsar/pull/12937 ### Motivation During register source/sink, use classloader for validate, but not close when register are done. And the classloader will not use for run. ### Modifications - Close the

[GitHub] [pulsar] 315157973 commented on a change in pull request #12869: [PIP-105] Part-1 Support add subscription properties

2021-11-22 Thread GitBox
315157973 commented on a change in pull request #12869: URL: https://github.com/apache/pulsar/pull/12869#discussion_r754771586 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/Topic.java ## @@ -129,6 +129,7 @@ default boolean isMarkerMessage() {

[GitHub] [pulsar] yuruguo opened a new pull request #12936: [Broker] Correct param of delete method for v1 topic

2021-11-22 Thread GitBox
yuruguo opened a new pull request #12936: URL: https://github.com/apache/pulsar/pull/12936 ### Motivation Currently, the deletion topic method for `v1/topic` has the problem of parameter loss and mismatch as below, 1. `deleteSchema` is missing https://github.com/apache/pulsar/blob

[GitHub] [pulsar] 315157973 commented on pull request #12869: [PIP-105] Part-1 Support add subscription properties

2021-11-22 Thread GitBox
315157973 commented on pull request #12869: URL: https://github.com/apache/pulsar/pull/12869#issuecomment-976111619 > @315157973 In topics stats, Do we need to display subscription properties information? > > When the user sets the sub properties, how do we view the sub properties s

[GitHub] [pulsar] gaozhangmin commented on pull request #11285: broker gc log

2021-11-22 Thread GitBox
gaozhangmin commented on pull request #11285: URL: https://github.com/apache/pulsar/pull/11285#issuecomment-976109768 @codelipenghui PTAL again -- 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] gaozhangmin commented on pull request #11893: Admin cli support terminate a partitioned topic

2021-11-22 Thread GitBox
gaozhangmin commented on pull request #11893: URL: https://github.com/apache/pulsar/pull/11893#issuecomment-976109636 @eolivelli @codelipenghui PTAL again -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [pulsar] gaozhangmin commented on pull request #12025: violation of isolation when using bookie affinity group

2021-11-22 Thread GitBox
gaozhangmin commented on pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#issuecomment-976109418 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To uns

[GitHub] [pulsar] gaozhangmin commented on pull request #12343: fix lastCumulativeAck.messageId npe when PersistentAcknowledgmentsGroupingTracker.flushSync

2021-11-22 Thread GitBox
gaozhangmin commented on pull request #12343: URL: https://github.com/apache/pulsar/pull/12343#issuecomment-976105484 /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 t

[GitHub] [pulsar] gaozhangmin commented on a change in pull request #12343: fix lastCumulativeAck.messageId npe when PersistentAcknowledgmentsGroupingTracker.flushSync

2021-11-22 Thread GitBox
gaozhangmin commented on a change in pull request #12343: URL: https://github.com/apache/pulsar/pull/12343#discussion_r754761842 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java ## @@ -477,13 +477,16 @@ publi

[GitHub] [pulsar] MarvinCai commented on pull request #12720: [Pulsar SQL] Support query chunked messages feature in Pulsar SQL

2021-11-22 Thread GitBox
MarvinCai commented on pull request #12720: URL: https://github.com/apache/pulsar/pull/12720#issuecomment-976100458 /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 to

[GitHub] [pulsar] ericsyh commented on pull request #12927: feat(cli): support sql-worker in pulsar-daemon

2021-11-22 Thread GitBox
ericsyh commented on pull request #12927: URL: https://github.com/apache/pulsar/pull/12927#issuecomment-976100116 @Anonymitaet If this PR merged, the command of start a Pulsar SQL worker should be updated. I changed the start command from `./bin/pulsar sql-worker run` to `./bin/pulsar sql-

[GitHub] [pulsar] ATaurenRideRocket closed issue #12921: Send To Topic Timeout

2021-11-22 Thread GitBox
ATaurenRideRocket closed issue #12921: URL: https://github.com/apache/pulsar/issues/12921 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-

[GitHub] [pulsar] ATaurenRideRocket commented on issue #12921: Send To Topic Timeout

2021-11-22 Thread GitBox
ATaurenRideRocket commented on issue #12921: URL: https://github.com/apache/pulsar/issues/12921#issuecomment-976099000 > @ATaurenRideRocket It's better to check the server-side metrics, maybe reach the disk bottleneck or others, so the messages write latency becomes very high, it's usually

[GitHub] [pulsar] Jason918 commented on pull request #12896: [Issue 12894] Only refresh metadata if path is already in cache after write.

2021-11-22 Thread GitBox
Jason918 commented on pull request #12896: URL: https://github.com/apache/pulsar/pull/12896#issuecomment-976096862 @merlimat please help take a look? -- 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] urfreespace opened a new pull request #12935: [website][upgrade]feat: website upgrade / docs migration - 2.6.0 / development/reference

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

[GitHub] [pulsar] urfreespace opened a new pull request #12934: [website][upgrade]feat: website upgrade / docs migration - 2.6.0 / adaptors/admin/cookbooks

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

[GitHub] [pulsar] Jason918 commented on pull request #12830: [broker]Optimize topicMaxMessageSize with topic local cache.

2021-11-22 Thread GitBox
Jason918 commented on pull request #12830: URL: https://github.com/apache/pulsar/pull/12830#issuecomment-976095726 /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 to t

[GitHub] [pulsar] urfreespace opened a new pull request #12933: [website][upgrade]feat: website upgrade / docs migration - 2.6.0 / helm/deploy/administration

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

[GitHub] [pulsar] urfreespace opened a new pull request #12932: [website][upgrade]feat: website upgrade / docs migration - 2.6.0 / functions/io/sql

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

[GitHub] [pulsar] urfreespace opened a new pull request #12931: [website][upgrade]feat: website upgrade / docs migration - 2.6.0 / get started/concepts/schema

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

[GitHub] [pulsar] Jason918 commented on pull request #12913: [broker] Delete TopicPoliciesService#clean, not used anymore.

2021-11-22 Thread GitBox
Jason918 commented on pull request #12913: URL: https://github.com/apache/pulsar/pull/12913#issuecomment-976092389 /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 to t

[GitHub] [pulsar] mattisonchao commented on issue #12912: org.apache.pulsar.broker.web.AuthenticationFilter Filter Out WebSocket with authentication token in query.

2021-11-22 Thread GitBox
mattisonchao commented on issue #12912: URL: https://github.com/apache/pulsar/issues/12912#issuecomment-976078705 @codelipenghui i want to work on it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] Anonymitaet commented on pull request #12927: feat(cli): support sql-worker in pulsar-daemon

2021-11-22 Thread GitBox
Anonymitaet commented on pull request #12927: URL: https://github.com/apache/pulsar/pull/12927#issuecomment-976071059 @ericsyh need to add docs [here](https://pulsar.apache.org/docs/en/next/reference-cli-tools/#pulsar-daemon)? -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar] AndrewJD79 commented on issue #12844: [pulsar-client-cpp] Excessive locking cause significant performance degradation

2021-11-22 Thread GitBox
AndrewJD79 commented on issue #12844: URL: https://github.com/apache/pulsar/issues/12844#issuecomment-976066157 I'll provide PR in a few days -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] yuruguo commented on pull request #12891: [pulsar-admin] Add get-replicated-subscription-status command for topic

2021-11-22 Thread GitBox
yuruguo commented on pull request #12891: URL: https://github.com/apache/pulsar/pull/12891#issuecomment-976003036 @315157973 @hangc0276 PTAL -- 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 s

[GitHub] [pulsar] bharanic-dev opened a new pull request #12930: [Issue 12929] [pulsar-metadata] Fix metadata store deadlock

2021-11-22 Thread GitBox
bharanic-dev opened a new pull request #12930: URL: https://github.com/apache/pulsar/pull/12930 Fixes #12929 ### Motivation A blocking thenAccept() call in metadata-store callback thread causes deadlock. ### Modifications Change thenAccept() to thenAcceptAsync()

[GitHub] [pulsar] bharanic-dev opened a new issue #12929: Deadlock in internalDeleteSubscription in metadata-store callback thread

2021-11-22 Thread GitBox
bharanic-dev opened a new issue #12929: URL: https://github.com/apache/pulsar/issues/12929 **Describe the bug** When doing certain topic operations (for e.g., delete-subscription), it can result in a deadlock metadata-store callback thread. Please see the stack trace below. "meta

[GitHub] [pulsar] devinbost commented on issue #12928: Rebalancing functions does nothing

2021-11-22 Thread GitBox
devinbost commented on issue #12928: URL: https://github.com/apache/pulsar/issues/12928#issuecomment-975781455 @jerrypeng FYI -- 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

[GitHub] [pulsar] devinbost opened a new issue #12928: Rebalancing functions does nothing

2021-11-22 Thread GitBox
devinbost opened a new issue #12928: URL: https://github.com/apache/pulsar/issues/12928 **Describe the bug** Hitting the endpoint to rebalance functions does not appear to work consistently. **To Reproduce** Steps to reproduce the behavior: First, we look at the functio

[GitHub] [pulsar] ericsyh opened a new pull request #12927: feat(cli): support sql-worker in pulsar-daemon

2021-11-22 Thread GitBox
ericsyh opened a new pull request #12927: URL: https://github.com/apache/pulsar/pull/12927 Signed-off-by: Eric Shen ### Motivation Pulsar sql-worker only supports in pulsar shell cli and not support in pulsar-daemon. So, users have to open two terminals to use pulsar

[pulsar] branch master updated (6200ef1 -> af1a82d)

2021-11-22 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 6200ef1 Fix log level config for pulsar-admin, pulsar-client and pulsar-perf (#12915) add af1a82d Apply execu

[pulsar] branch master updated (2d83a82 -> 6200ef1)

2021-11-22 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 2d83a82 Abstract some common methods to AbstractMetadataStore (#12916) add 6200ef1 Fix log level config for pu

[GitHub] [pulsar] merlimat merged pull request #12852: Source tarball: Apply executable file permissions to py scripts.

2021-11-22 Thread GitBox
merlimat merged pull request #12852: URL: https://github.com/apache/pulsar/pull/12852 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsu

[pulsar] branch master updated: Abstract some common methods to AbstractMetadataStore (#12916)

2021-11-22 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.git The following commit(s) were added to refs/heads/master by this push: new 2d83a82 Abstract some common methods to Abstrac

[GitHub] [pulsar] merlimat merged pull request #12915: Fix log level config for pulsar-admin, pulsar-client and pulsar-perf

2021-11-22 Thread GitBox
merlimat merged pull request #12915: URL: https://github.com/apache/pulsar/pull/12915 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsu

[GitHub] [pulsar] merlimat merged pull request #12916: [metadata] Optimize put and get methods in AbstractMetadataStore

2021-11-22 Thread GitBox
merlimat merged pull request #12916: URL: https://github.com/apache/pulsar/pull/12916 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsu

[GitHub] [pulsar] zeo1995 opened a new pull request #12926: [website][upgrade]feat: website upgrade / docs migration - 2.6.2 /Cookbooks/ Development/Reference

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

[GitHub] [pulsar] zeo1995 opened a new pull request #12925: [website][upgrade]feat: website upgrade / docs migration - 2.6.2 /Performance/ Client Libraries/Admin API/Adaptors

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

[GitHub] [pulsar] zeo1995 opened a new pull request #12924: [website][upgrade]feat: website upgrade / docs migration - 2.6.2 /Kubernetes (Helm)/ Deployment/Administration/Security

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

[GitHub] [pulsar] zeo1995 opened a new pull request #12923: [website][upgrade]feat: website upgrade / docs migration - 2.6.2 Pulsar Functions/Pulsar IO/Pulsar SQL

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

[pulsar] branch master updated: Build website using https://pulsar.apache.org (#12901)

2021-11-22 Thread eolivelli
This is an automated email from the ASF dual-hosted git repository. eolivelli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/master by this push: new a89bdf4 Build website using https://pulsar.a

[GitHub] [pulsar] eolivelli merged pull request #12901: [Website] Builds using https://pulsar.incubator.apache.org as baseurl switch to TLP address

2021-11-22 Thread GitBox
eolivelli merged pull request #12901: URL: https://github.com/apache/pulsar/pull/12901 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-uns

[GitHub] [pulsar] zeo1995 opened a new pull request #12922: [website][upgrade]feat: website upgrade / docs migration - 2.6.2 Get Started/Concepts and Architecture/Pulsar Schema

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

[GitHub] [pulsar] Jason918 commented on a change in pull request #12913: [broker] Delete TopicPoliciesService#clean, not used anymore.

2021-11-22 Thread GitBox
Jason918 commented on a change in pull request #12913: URL: https://github.com/apache/pulsar/pull/12913#discussion_r754378894 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/TopicPoliciesService.java ## @@ -120,14 +120,6 @@ void unregisterList

[GitHub] [pulsar] Jason918 commented on pull request #12830: [broker]Optimize topicMaxMessageSize with topic local cache.

2021-11-22 Thread GitBox
Jason918 commented on pull request #12830: URL: https://github.com/apache/pulsar/pull/12830#issuecomment-975633035 > Yes, but I think we should keep consistent in the local topic policy cache? And for the next step, we can apply the `PolicyHierarchyValue` to get topic policy. Great

[GitHub] [pulsar] Jason918 commented on pull request #12833: [broker] Bug Fix: topic policy is not properly init if namespace is loaded first.

2021-11-22 Thread GitBox
Jason918 commented on pull request #12833: URL: https://github.com/apache/pulsar/pull/12833#issuecomment-975623702 /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 to t

[GitHub] [pulsar] Jason918 commented on pull request #12874: [broker] Optimize blocking backlogQuotaCheck to non-blocking in ServerCnx#handleProducer

2021-11-22 Thread GitBox
Jason918 commented on pull request #12874: URL: https://github.com/apache/pulsar/pull/12874#issuecomment-975617045 > @Jason918 Do you want the resolve the conflicts? Resolved, sorry for missing this. -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [pulsar] hangc0276 commented on issue #12921: Send To Topic Timeout

2021-11-22 Thread GitBox
hangc0276 commented on issue #12921: URL: https://github.com/apache/pulsar/issues/12921#issuecomment-975608807 You'd better give more metrics in the server side, and then we can figure out the problem. -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [pulsar] hangc0276 commented on a change in pull request #12913: [broker] Delete TopicPoliciesService#clean, not used anymore.

2021-11-22 Thread GitBox
hangc0276 commented on a change in pull request #12913: URL: https://github.com/apache/pulsar/pull/12913#discussion_r754348117 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/TopicPoliciesService.java ## @@ -120,14 +120,6 @@ void unregisterLis

[GitHub] [pulsar] hangc0276 commented on a change in pull request #12919: [Broker] Change create topic return error to Status.BAD_REQUEST

2021-11-22 Thread GitBox
hangc0276 commented on a change in pull request #12919: URL: https://github.com/apache/pulsar/pull/12919#discussion_r754339783 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -3694,7 +3694,7 @@ private Topic getTopi

[GitHub] [pulsar] hangc0276 commented on a change in pull request #12869: [PIP-105] Part-1 Support add subscription properties

2021-11-22 Thread GitBox
hangc0276 commented on a change in pull request #12869: URL: https://github.com/apache/pulsar/pull/12869#discussion_r754306400 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/nonpersistent/NonPersistentTopic.java ## @@ -224,6 +225,15 @@ public void r

[GitHub] [pulsar] codelipenghui commented on a change in pull request #12396: Fix NPE when create OpEntry

2021-11-22 Thread GitBox
codelipenghui commented on a change in pull request #12396: URL: https://github.com/apache/pulsar/pull/12396#discussion_r754321070 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java ## @@ -622,9 +622,11 @@ public void asyncRead

[GitHub] [pulsar] codelipenghui commented on issue #12911: Docs token authentication configuration example does not match implementation

2021-11-22 Thread GitBox
codelipenghui commented on issue #12911: URL: https://github.com/apache/pulsar/issues/12911#issuecomment-975568579 @YouJiacheng Do you want to push a fix for the documentation? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] codelipenghui commented on issue #12921: Send To Topic Timeout

2021-11-22 Thread GitBox
codelipenghui commented on issue #12921: URL: https://github.com/apache/pulsar/issues/12921#issuecomment-975566153 @ATaurenRideRocket It's better to check the server-side metrics, maybe reach the disk bottleneck or others, so the messages write latency becomes very high, it's usually the r

[GitHub] [pulsar] codelipenghui commented on pull request #12830: [broker]Optimize topicMaxMessageSize with topic local cache.

2021-11-22 Thread GitBox
codelipenghui commented on pull request #12830: URL: https://github.com/apache/pulsar/pull/12830#issuecomment-975562716 Yes, but I think we should keep consistent in the local topic policy cache? And for the next step, we can apply the `PolicyHierarchyValue` to get topic policy. -- This

[GitHub] [pulsar] codelipenghui commented on pull request #12874: [broker] Optimize blocking backlogQuotaCheck to non-blocking in ServerCnx#handleProducer

2021-11-22 Thread GitBox
codelipenghui commented on pull request #12874: URL: https://github.com/apache/pulsar/pull/12874#issuecomment-975547013 @Jason918 Do you want the resolve the conflicts? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [pulsar] codelipenghui commented on pull request #12870: replace deprecated function of TopicPoliciesTest

2021-11-22 Thread GitBox
codelipenghui commented on pull request #12870: URL: https://github.com/apache/pulsar/pull/12870#issuecomment-975553174 @suiyuzeng Could you please help resolve the conflicts? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [pulsar] codelipenghui commented on a change in pull request #12872: [Broker] Support create subscription by specifying the earliest or latest position.

2021-11-22 Thread GitBox
codelipenghui commented on a change in pull request #12872: URL: https://github.com/apache/pulsar/pull/12872#discussion_r754302614 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/PersistentTopics.java ## @@ -1322,7 +1322,7 @@ public void createSubsc

[pulsar] branch master updated (32b697d -> 093c230)

2021-11-22 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 32b697d [Java Client] Avoid IllegalStateException in ClientCnx debug logs (#12899) add 093c230 [Doc] add exp

[GitHub] [pulsar] codelipenghui merged pull request #12914: [Doc] add explanations for Avro schema in Python client

2021-11-22 Thread GitBox
codelipenghui merged pull request #12914: URL: https://github.com/apache/pulsar/pull/12914 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits

[GitHub] [pulsar] congbobo184 commented on a change in pull request #12919: [Broker] Change create topic return error to Status.BAD_REQUEST

2021-11-22 Thread GitBox
congbobo184 commented on a change in pull request #12919: URL: https://github.com/apache/pulsar/pull/12919#discussion_r754291928 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -3694,7 +3694,7 @@ private Topic getTo

[GitHub] [pulsar] codelipenghui commented on pull request #12918: [Doc] add explanations for REST

2021-11-22 Thread GitBox
codelipenghui commented on pull request #12918: URL: https://github.com/apache/pulsar/pull/12918#issuecomment-975534561 @MarvinCai Could you 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

[GitHub] [pulsar] congbobo184 commented on a change in pull request #12919: [Broker] Change create topic return error to Status.BAD_REQUEST

2021-11-22 Thread GitBox
congbobo184 commented on a change in pull request #12919: URL: https://github.com/apache/pulsar/pull/12919#discussion_r754285406 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -250,7 +250,7 @@ protected void valida

[GitHub] [pulsar] codelipenghui commented on a change in pull request #12919: [Broker] Change create topic return error to Status.BAD_REQUEST

2021-11-22 Thread GitBox
codelipenghui commented on a change in pull request #12919: URL: https://github.com/apache/pulsar/pull/12919#discussion_r754280213 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -3694,7 +3694,7 @@ private Topic get

[GitHub] [pulsar] yuruguo removed a comment on pull request #12891: [pulsar-admin] Add get-replicated-subscription-status command for topic

2021-11-22 Thread GitBox
yuruguo removed a comment on pull request #12891: URL: https://github.com/apache/pulsar/pull/12891#issuecomment-975521553 /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] yuruguo commented on pull request #12891: [pulsar-admin] Add get-replicated-subscription-status command for topic

2021-11-22 Thread GitBox
yuruguo commented on pull request #12891: URL: https://github.com/apache/pulsar/pull/12891#issuecomment-975521553 /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 to th

[GitHub] [pulsar] yuruguo commented on pull request #12351: [pulsar-admin] Add remove-clusters command for namespace

2021-11-22 Thread GitBox
yuruguo commented on pull request #12351: URL: https://github.com/apache/pulsar/pull/12351#issuecomment-975520526 @codelipenghui @hangc0276 @gaoran10 PTAL, thx! -- 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] gaoran10 commented on a change in pull request #12919: [Broker] Change create topic return error to Status.BAD_REQUEST

2021-11-22 Thread GitBox
gaoran10 commented on a change in pull request #12919: URL: https://github.com/apache/pulsar/pull/12919#discussion_r754273716 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -250,7 +250,7 @@ protected void validateA

[pulsar] branch branch-2.8 updated: [Transaction] Fix transaction system topic create in loop (#12889)

2021-11-22 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.8 by this push: new 50727d7 [Transaction] Fix transaction

[GitHub] [pulsar] codelipenghui merged pull request #12889: [Transaction] Fix transaction system topic create in loop

2021-11-22 Thread GitBox
codelipenghui merged pull request #12889: URL: https://github.com/apache/pulsar/pull/12889 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits

[GitHub] [pulsar] wolfstudy edited a comment on pull request #12869: [PIP-105] Part-1 Support add subscription properties

2021-11-22 Thread GitBox
wolfstudy edited a comment on pull request #12869: URL: https://github.com/apache/pulsar/pull/12869#issuecomment-975468918 @315157973 In topics stats, Do we need to display subscription properties information? When the user sets the sub properties, how do we view the sub properties s

  1   2   >