[GitHub] [pulsar] Technoboy- opened a new pull request, #16155: [Pulsar SQL] Fix `messageQueue` release message issue.

2022-06-20 Thread GitBox
Technoboy- opened a new pull request, #16155: URL: https://github.com/apache/pulsar/pull/16155 ### Motivation When PulsarRecordCursor close, there may occur below error: ``` 2022-06-08T09:33:33.959423259-04:00 2022-06-08T13:33:33.959Z INFO 20220608_133129_00353_ydmer.2.0-21-52

[GitHub] [pulsar] eolivelli commented on a diff in pull request #16152: [fix][client]Fix auto cluster failover can't resolve host bug

2022-06-20 Thread GitBox
eolivelli commented on code in PR #16152: URL: https://github.com/apache/pulsar/pull/16152#discussion_r902218791 ## pulsar-client/src/test/java/org/apache/pulsar/client/impl/AutoClusterFailoverTest.java: ## @@ -283,4 +291,46 @@ public void testAutoClusterFailoverSwitchTlsTrustSt

[GitHub] [pulsar] eolivelli commented on pull request #15957: [fix][tests] TieredStorageConfigurationTests - clear system properties

2022-06-20 Thread GitBox
eolivelli commented on PR #15957: URL: https://github.com/apache/pulsar/pull/15957#issuecomment-1161327169 /pulsarbot rerun-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 the spec

[GitHub] [pulsar] Technoboy- commented on a diff in pull request #16141: [improve][broker][PIP-149]make getInternalStats method async

2022-06-20 Thread GitBox
Technoboy- commented on code in PR #16141: URL: https://github.com/apache/pulsar/pull/16141#discussion_r902213429 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java: ## @@ -1208,24 +1207,24 @@ protected CompletableFuture internalGetStat

[pulsar-site] branch asf-site-next updated: Updated site at revision 19c379d

2022-06-20 Thread urfree
This is an automated email from the ASF dual-hosted git repository. urfree pushed a commit to branch asf-site-next in repository https://gitbox.apache.org/repos/asf/pulsar-site.git The following commit(s) were added to refs/heads/asf-site-next by this push: new 817075ae824 Updated site at r

[GitHub] [pulsar] github-actions[bot] commented on pull request #16154: [fix][client] Fix the startMessageId can't be respected as the ChunkMessageID

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #16154: URL: https://github.com/apache/pulsar/pull/16154#issuecomment-1161306815 @RobertIndie Please provide a correct documentation label for your PR. Instructions see [Pulsar Documentation Label Guide](https://docs.google.com/document/d/1Qw7LHQdXWBW9t

[GitHub] [pulsar] RobertIndie opened a new pull request, #16154: [fix][client] Fix the startMessageId can't be respected as the ChunkMessageID

2022-06-20 Thread GitBox
RobertIndie opened a new pull request, #16154: URL: https://github.com/apache/pulsar/pull/16154 Master Issue: #12402 ### Motivation This is the same problem as when the consumer inclusive seeks the chunked message. See more detail in [PIP-107](https://github.com/

[pulsar-site] branch main updated: Docs sync done from apache/pulsar(#31cd857)

2022-06-20 Thread urfree
This is an automated email from the ASF dual-hosted git repository. urfree pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git The following commit(s) were added to refs/heads/main by this push: new 19c379db92f Docs sync done from apache/pulsar(#

[GitHub] [pulsar] tisonkun commented on pull request #16109: [improve][pom] Remove redundant pulsar-zookeeper-utils module

2022-06-20 Thread GitBox
tisonkun commented on PR #16109: URL: https://github.com/apache/pulsar/pull/16109#issuecomment-1161302894 Integrated with #16110 & #16148 . cc @eolivelli @codelipenghui @merlimat would you please give a review to this patch? -- This is an automated message from the Apache Git Servi

[GitHub] [pulsar] tisonkun commented on pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
tisonkun commented on PR #16110: URL: https://github.com/apache/pulsar/pull/16110#issuecomment-1161301802 Thanks for your review! -- 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 comm

[GitHub] [pulsar] tisonkun commented on pull request #16148: [security] Update fastjson version to 1.2.83

2022-06-20 Thread GitBox
tisonkun commented on PR #16148: URL: https://github.com/apache/pulsar/pull/16148#issuecomment-1161301714 Thanks for your review! -- 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 comm

[GitHub] [pulsar] tjiuming commented on pull request #16108: [fix][broker][monitoring] fix message ack rate

2022-06-20 Thread GitBox
tjiuming commented on PR #16108: URL: https://github.com/apache/pulsar/pull/16108#issuecomment-1161296340 @codelipenghui Fixed batch messages ack rate, 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 abov

[GitHub] [pulsar] nicoloboschi merged pull request #16148: [security] Update fastjson version to 1.2.83

2022-06-20 Thread GitBox
nicoloboschi merged PR #16148: URL: https://github.com/apache/pulsar/pull/16148 -- 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-unsubscr...@puls

[pulsar] branch master updated: [security] Update fastjson version to 1.2.83 (#16148)

2022-06-20 Thread nicoloboschi
This is an automated email from the ASF dual-hosted git repository. nicoloboschi 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 31cd8576358 [security] Update fastjson ver

[pulsar] branch master updated: [security] Suppress CVE-2022-23712 warnings (#16110)

2022-06-20 Thread nicoloboschi
This is an automated email from the ASF dual-hosted git repository. nicoloboschi 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 f4a6b864f84 [security] Suppress CVE-2022-2

[GitHub] [pulsar] nicoloboschi merged pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
nicoloboschi merged PR #16110: URL: https://github.com/apache/pulsar/pull/16110 -- 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-unsubscr...@puls

[GitHub] [pulsar] congbobo184 commented on a diff in pull request #16072: [fix][Java Client] Fix thread safety issue of `LastCumulativeAck`

2022-06-20 Thread GitBox
congbobo184 commented on code in PR #16072: URL: https://github.com/apache/pulsar/pull/16072#discussion_r902144130 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java: ## @@ -370,29 +371,9 @@ private void doIndividualBatchAc

[GitHub] [pulsar] Jason918 opened a new issue, #16153: PIP-180: Shadow Topic, an alternative way to support readonly topic ownership.

2022-06-20 Thread GitBox
Jason918 opened a new issue, #16153: URL: https://github.com/apache/pulsar/issues/16153 ## Motivation The motivation is the same as [PIP-63](https://github.com/apache/pulsar/wiki/PIP-63%3A-Readonly-Topic-Ownership-Support), with a new use case support for 100K subscriptions b

[GitHub] [pulsar] poorbarcode commented on issue #16135: [PIP-179] Support the admin API to check unknown request parameters

2022-06-20 Thread GitBox
poorbarcode commented on issue #16135: URL: https://github.com/apache/pulsar/issues/16135#issuecomment-1161239127 > PIP ID is used by #16042 I have corrected it. Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [pulsar] Jason918 commented on issue #16135: [PIP-178] Support the admin API to check unknown request parameters

2022-06-20 Thread GitBox
Jason918 commented on issue #16135: URL: https://github.com/apache/pulsar/issues/16135#issuecomment-1161237208 PIP ID is used by #16042 -- 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 specifi

[pulsar] branch master updated: [cleanup] [broker] when serverCnx.disabledAutoRead(), use same code (#15181)

2022-06-20 Thread jianghaiting
This is an automated email from the ASF dual-hosted git repository. jianghaiting 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 5d563ac47aa [cleanup] [broker] when server

[GitHub] [pulsar] Jason918 merged pull request #15181: [cleanup] [broker] when serverCnx disabled auto read use same implements

2022-06-20 Thread GitBox
Jason918 merged PR #15181: URL: https://github.com/apache/pulsar/pull/15181 -- 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-unsubscr...@pulsar.a

[GitHub] [pulsar] AnonHxy commented on pull request #16144: [fix][broker]Fix removing replica clusters on topic-policies not take effect

2022-06-20 Thread GitBox
AnonHxy commented on PR #16144: URL: https://github.com/apache/pulsar/pull/16144#issuecomment-1161219947 /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 the specific

[GitHub] [pulsar] Demogorgon314 commented on a diff in pull request #16072: [fix][Java Client] Fix thread safety issue of `LastCumulativeAck`

2022-06-20 Thread GitBox
Demogorgon314 commented on code in PR #16072: URL: https://github.com/apache/pulsar/pull/16072#discussion_r902131202 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java: ## @@ -476,12 +457,24 @@ public void flush() { pr

[GitHub] [pulsar] Jason918 commented on a diff in pull request #16152: [fix][client]Fix auto cluster failover can't resolve host bug

2022-06-20 Thread GitBox
Jason918 commented on code in PR #16152: URL: https://github.com/apache/pulsar/pull/16152#discussion_r902130295 ## pulsar-client/src/test/java/org/apache/pulsar/client/impl/AutoClusterFailoverTest.java: ## @@ -283,4 +291,46 @@ public void testAutoClusterFailoverSwitchTlsTrustSto

[GitHub] [pulsar] AnonHxy commented on pull request #16144: [fix][broker]Fix removing replica clusters on topic-policies not take effect

2022-06-20 Thread GitBox
AnonHxy commented on PR #16144: URL: https://github.com/apache/pulsar/pull/16144#issuecomment-1161194780 Also modify the namespace-policy and topic-policy initialization order. PTAL @Jason918 -- This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [pulsar] Nicklee007 commented on pull request #16143: [improve][broker]Specify default nar tmp dir

2022-06-20 Thread GitBox
Nicklee007 commented on PR #16143: URL: https://github.com/apache/pulsar/pull/16143#issuecomment-1161153815 > LGTM > > but this should go into 2.12, so I am keeping "request changes" for now OK, Thx. -- This is an automated message from the Apache Git Service. To respond to t

[GitHub] [pulsar] github-actions[bot] commented on pull request #16152: [fix][client]Fix auto cluster failover can't resolve host bug

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #16152: URL: https://github.com/apache/pulsar/pull/16152#issuecomment-1161126607 @hangc0276 Please provide a correct documentation label for your PR. Instructions see [Pulsar Documentation Label Guide](https://docs.google.com/document/d/1Qw7LHQdXWBW9t2-

[GitHub] [pulsar] github-actions[bot] commented on pull request #16152: [fix][client]Fix auto cluster failover can't resolve host bug

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #16152: URL: https://github.com/apache/pulsar/pull/16152#issuecomment-1161124591 @hangc0276 Please provide a correct documentation label for your PR. Instructions see [Pulsar Documentation Label Guide](https://docs.google.com/document/d/1Qw7LHQdXWBW9t2-

[GitHub] [pulsar] hangc0276 opened a new pull request, #16152: [fix][client]Fix auto cluster failover can't resolve host bug

2022-06-20 Thread GitBox
hangc0276 opened a new pull request, #16152: URL: https://github.com/apache/pulsar/pull/16152 ### Motivation When running a Pulsar client with auto cluster failover, it will throw the following exception when detecting the pulsar service URL. ``` 2022-06-21T09:59:52,646+0800 WARN [

[GitHub] [pulsar] github-actions[bot] commented on issue #15171: delay message deliver lost messages

2022-06-20 Thread GitBox
github-actions[bot] commented on issue #15171: URL: https://github.com/apache/pulsar/issues/15171#issuecomment-1161108811 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [pulsar] github-actions[bot] commented on pull request #15182: [Function] support storageClient backoffPlicy configure

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #15182: URL: https://github.com/apache/pulsar/pull/15182#issuecomment-1161108765 The pr had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [pulsar] github-actions[bot] commented on issue #15258: Pulsar Bookie dies unexpectedly

2022-06-20 Thread GitBox
github-actions[bot] commented on issue #15258: URL: https://github.com/apache/pulsar/issues/15258#issuecomment-1161108670 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [pulsar] leizhiyuan opened a new issue, #16151: support retry strategy when produce message

2022-06-20 Thread GitBox
leizhiyuan opened a new issue, #16151: URL: https://github.com/apache/pulsar/issues/16151 **Is your feature request related to a problem? Please describe.** A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] when we restart broker, client

[GitHub] [pulsar] congbobo184 commented on a diff in pull request #15140: [monitor][txn] Add metrics for transaction

2022-06-20 Thread GitBox
congbobo184 commented on code in PR #15140: URL: https://github.com/apache/pulsar/pull/15140#discussion_r902098175 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/transaction/buffer/TransactionBufferClientStats.java: ## @@ -0,0 +1,70 @@ +/** + * Licensed to the Apache So

[GitHub] [pulsar] congbobo184 commented on a diff in pull request #15140: [monitor][txn] Add metrics for transaction

2022-06-20 Thread GitBox
congbobo184 commented on code in PR #15140: URL: https://github.com/apache/pulsar/pull/15140#discussion_r902097678 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/transaction/buffer/TransactionBufferClientStats.java: ## @@ -0,0 +1,70 @@ +/** + * Licensed to the Apache So

[GitHub] [pulsar] congbobo184 commented on issue #16058: PIP-177: Add the classLoader field for SchemaDefinition

2022-06-20 Thread GitBox
congbobo184 commented on issue #16058: URL: https://github.com/apache/pulsar/issues/16058#issuecomment-1161088628 +1 (none-binding) Thanks, Bo > 2022年6月14日 下午9:54,Cong Zhao ***@***.***> 写道: > > > Motivation > > Now, don‘t register logical type conversions when us

[GitHub] [pulsar] tisonkun commented on pull request #16148: [security] Update fastjson version to 1.2.83

2022-06-20 Thread GitBox
tisonkun commented on PR #16148: URL: https://github.com/apache/pulsar/pull/16148#issuecomment-1161071831 @nicoloboschi @codelipenghui could you merge this PR or #16110 first? They're logically as a whole and block my initial PR #16109. IIUC we should merge this PR and #16110 first and revi

[GitHub] [pulsar] momo-jun commented on pull request #16041: [improve][function] Support Record as Function output type

2022-06-20 Thread GitBox
momo-jun commented on PR #16041: URL: https://github.com/apache/pulsar/pull/16041#issuecomment-1161068313 Hi @cbornet , will you update relevant docs in a follow-up PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [pulsar] codelipenghui commented on pull request #16138: [improve][broker] Use LinkedHashSet for config items of type Set to preserve elements order

2022-06-20 Thread GitBox
codelipenghui commented on PR #16138: URL: https://github.com/apache/pulsar/pull/16138#issuecomment-1161050118 @tuteng I have removed the cherry-picked/ label, the cherry-picked label should be added after we complete the cherry-picking. Otherwise, we might miss the cherry-pick after the PR

[GitHub] [pulsar] github-actions[bot] commented on pull request #16138: [improve][broker] Use LinkedHashSet for config items of type Set to preserve elements order

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #16138: URL: https://github.com/apache/pulsar/pull/16138#issuecomment-1161048884 @fantapsody Please provide a correct documentation label for your PR. Instructions see [Pulsar Documentation Label Guide](https://docs.google.com/document/d/1Qw7LHQdXWBW9t2

[pulsar] branch master updated: remove duplicated release (#16118)

2022-06-20 Thread aloyszhang
This is an automated email from the ASF dual-hosted git repository. aloyszhang 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 603a5cdd8e4 remove duplicated release (#1611

[GitHub] [pulsar] aloyszhang merged pull request #16118: [hotfix] remove duplicated code

2022-06-20 Thread GitBox
aloyszhang merged PR #16118: URL: https://github.com/apache/pulsar/pull/16118 -- 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-unsubscr...@pulsar

[GitHub] [pulsar] github-actions[bot] commented on pull request #16147: [fix][txn]Fix race in TransactionMetaStoreHandler

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #16147: URL: https://github.com/apache/pulsar/pull/16147#issuecomment-1161045402 @eolivelli Please provide a correct documentation label for your PR. Instructions see [Pulsar Documentation Label Guide](https://docs.google.com/document/d/1Qw7LHQdXWBW9t2-

[GitHub] [pulsar] github-actions[bot] commented on pull request #16147: [fix][txn]Fix race in TransactionMetaStoreHandler

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #16147: URL: https://github.com/apache/pulsar/pull/16147#issuecomment-1161044849 @eolivelli Please provide a correct documentation label for your PR. Instructions see [Pulsar Documentation Label Guide](https://docs.google.com/document/d/1Qw7LHQdXWBW9t2-

[pulsar] branch master updated (159ead34d3f -> ec276da5323)

2022-06-20 Thread bogong
This is an automated email from the ASF dual-hosted git repository. bogong pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git from 159ead34d3f Use non-reentrant lock for GrowableArrayBlockingQueue tail (#16125) add ec276da5323 Transactions: fi

[GitHub] [pulsar] congbobo184 merged pull request #16147: Transactions: fix race in TransactionMetaStoreHandler

2022-06-20 Thread GitBox
congbobo184 merged PR #16147: URL: https://github.com/apache/pulsar/pull/16147 -- 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-unsubscr...@pulsa

[GitHub] [pulsar] congbobo184 closed issue #15375: [Transactions] NullPointerException during Java Client bootstrap

2022-06-20 Thread GitBox
congbobo184 closed issue #15375: [Transactions] NullPointerException during Java Client bootstrap URL: https://github.com/apache/pulsar/issues/15375 -- 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

[pulsar-site] branch main updated: Docs sync done from apache/pulsar(#159ead3)

2022-06-20 Thread urfree
This is an automated email from the ASF dual-hosted git repository. urfree pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git The following commit(s) were added to refs/heads/main by this push: new 93ef11cd395 Docs sync done from apache/pulsar(#

[GitHub] [pulsar-client-go] thinker0 commented on issue #676: [BUG] When reconnecting to the server, the old connection is not closed

2022-06-20 Thread GitBox
thinker0 commented on issue #676: URL: https://github.com/apache/pulsar-client-go/issues/676#issuecomment-1160960275 This problem is occurring irregularly in pulsar-client-go. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [pulsar] dependabot[bot] opened a new pull request, #16150: Bump kotlin-stdlib from 1.4.32 to 1.6.0

2022-06-20 Thread GitBox
dependabot[bot] opened a new pull request, #16150: URL: https://github.com/apache/pulsar/pull/16150 Bumps [kotlin-stdlib](https://github.com/JetBrains/kotlin) from 1.4.32 to 1.6.0. Release notes Sourced from https://github.com/JetBrains/kotlin/releases";>kotlin-stdlib's releases.

[pulsar] branch dependabot/maven/org.jetbrains.kotlin-kotlin-stdlib-1.6.0 created (now 239e408f260)

2022-06-20 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/org.jetbrains.kotlin-kotlin-stdlib-1.6.0 in repository https://gitbox.apache.org/repos/asf/pulsar.git at 239e408f260 Bump kotlin-stdlib from 1.4.32 to 1.6.0 No new re

[GitHub] [pulsar] github-actions[bot] commented on pull request #16150: Bump kotlin-stdlib from 1.4.32 to 1.6.0

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #16150: URL: https://github.com/apache/pulsar/pull/16150#issuecomment-1160919463 @dependabot[bot] Please provide a correct documentation label for your PR. Instructions see [Pulsar Documentation Label Guide](https://docs.google.com/document/d/1Qw7LHQdXW

[GitHub] [pulsar] frankjkelly commented on issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms

2022-06-20 Thread GitBox
frankjkelly commented on issue #15904: URL: https://github.com/apache/pulsar/issues/15904#issuecomment-1160890925 Thanks @Anonymitaet and @momo-jun -- 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] MMirelli opened a new pull request, #16149: Make permissions of python docker image scripts effective

2022-06-20 Thread GitBox
MMirelli opened a new pull request, #16149: URL: https://github.com/apache/pulsar/pull/16149 ### Motivation Makes the incomplete fix introduced by https://github.com/apache/pulsar/pull/11951 effective. `mvn clean install -DskipTests` generates `target/apache-pulsar-2.X.0-src.t

[pulsar-site] branch main updated: Docs sync done from apache/pulsar(#159ead3)

2022-06-20 Thread urfree
This is an automated email from the ASF dual-hosted git repository. urfree pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git The following commit(s) were added to refs/heads/main by this push: new f36cd85b23a Docs sync done from apache/pulsar(#

[pulsar] branch master updated: Use non-reentrant lock for GrowableArrayBlockingQueue tail (#16125)

2022-06-20 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 159ead34d3f Use non-reentrant lock for GrowableA

[GitHub] [pulsar] merlimat merged pull request #16125: Use non-reentrant lock for GrowableArrayBlockingQueue tail

2022-06-20 Thread GitBox
merlimat merged PR #16125: URL: https://github.com/apache/pulsar/pull/16125 -- 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-unsubscr...@pulsar.a

[pulsar] branch master updated: Avoid some promise object allocations when writing on channel (#16113)

2022-06-20 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 e604348c94a Avoid some promise object allocation

[GitHub] [pulsar] merlimat merged pull request #16113: Avoid some promise object allocations when writing on channel

2022-06-20 Thread GitBox
merlimat merged PR #16113: URL: https://github.com/apache/pulsar/pull/16113 -- 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-unsubscr...@pulsar.a

[GitHub] [pulsar] codelipenghui commented on pull request #16147: Transactions: fix race in TransactionMetaStoreHandler

2022-06-20 Thread GitBox
codelipenghui commented on PR #16147: URL: https://github.com/apache/pulsar/pull/16147#issuecomment-1160607928 @congbobo184 @liangyepianzhou Please help review the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #16131: [fix][python client] Fixed schema validate_type method throw a wrong exception

2022-06-20 Thread GitBox
BewareMyPower commented on code in PR #16131: URL: https://github.com/apache/pulsar/pull/16131#discussion_r901774301 ## pulsar-client-cpp/python/pulsar/schema/definition.py: ## @@ -444,8 +444,8 @@ def validate_type(self, name, val): for x in val: if type(

[GitHub] [pulsar] Demogorgon314 commented on a diff in pull request #16072: [fix][Java Client] Fix thread safety issue of `LastCumulativeAck`

2022-06-20 Thread GitBox
Demogorgon314 commented on code in PR #16072: URL: https://github.com/apache/pulsar/pull/16072#discussion_r901774237 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java: ## @@ -116,8 +118,8 @@ public PersistentAcknowledgmen

[pulsar-helm-chart] branch master updated: Support defining extra env for broker and proxy statefulsset. (#273)

2022-06-20 Thread xxc
This is an automated email from the ASF dual-hosted git repository. xxc pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git The following commit(s) were added to refs/heads/master by this push: new c6ab1d1 Support defining extra env for

[GitHub] [pulsar] tisonkun commented on pull request #16148: [security] Update fastjson version to 1.2.83

2022-06-20 Thread GitBox
tisonkun commented on PR #16148: URL: https://github.com/apache/pulsar/pull/16148#issuecomment-1160526237 cc @lhotari @nicoloboschi -- 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 c

[GitHub] [pulsar] tisonkun opened a new pull request, #16148: [security] Update fastjson version to 1.2.83

2022-06-20 Thread GitBox
tisonkun opened a new pull request, #16148: URL: https://github.com/apache/pulsar/pull/16148 This PR is related to #16110. ### Motivation Fixes CVE-2022-25845. ### Modifications Update fastjson version to 1.2.83 ### Verifying this change - [ ] Make su

[GitHub] [pulsar] Technoboy- merged pull request #15736: [cleanup][ledger] Remove unnecessary code in ManagedLedgerImpl

2022-06-20 Thread GitBox
Technoboy- merged PR #15736: URL: https://github.com/apache/pulsar/pull/15736 -- 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-unsubscr...@pulsar

[pulsar] branch master updated: [cleanup][ledger] Remove unnecessary code in ManagedLedgerImpl (#15736)

2022-06-20 Thread technoboy
This is an automated email from the ASF dual-hosted git repository. technoboy 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 88a89f815ee [cleanup][ledger] Remove unnecess

[GitHub] [pulsar] eolivelli commented on a diff in pull request #15957: [fix][tests] TieredStorageConfigurationTests - clear system properties

2022-06-20 Thread GitBox
eolivelli commented on code in PR #15957: URL: https://github.com/apache/pulsar/pull/15957#discussion_r901677944 ## tiered-storage/jcloud/src/test/java/org/apache/bookkeeper/mledger/offload/jcloud/provider/TieredStorageConfigurationTests.java: ## @@ -217,9 +217,10 @@ public void

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #16072: [fix][Java Client] Fix thread safety issue of `LastCumulativeAck`

2022-06-20 Thread GitBox
BewareMyPower commented on code in PR #16072: URL: https://github.com/apache/pulsar/pull/16072#discussion_r901656825 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java: ## @@ -116,8 +118,8 @@ public PersistentAcknowledgmen

[GitHub] [pulsar] BewareMyPower commented on a diff in pull request #16072: [fix][Java Client] Fix thread safety issue of `LastCumulativeAck`

2022-06-20 Thread GitBox
BewareMyPower commented on code in PR #16072: URL: https://github.com/apache/pulsar/pull/16072#discussion_r901656825 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java: ## @@ -116,8 +118,8 @@ public PersistentAcknowledgmen

[GitHub] [pulsar] HQebupt commented on pull request #16141: [improve][broker][PIP-149]make getInternalStats method async

2022-06-20 Thread GitBox
HQebupt commented on PR #16141: URL: https://github.com/apache/pulsar/pull/16141#issuecomment-1160420755 /pulsarbot rerun-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 the specif

[GitHub] [pulsar] Demogorgon314 commented on a diff in pull request #16072: [fix][Java Client] Fix thread safety issue of `LastCumulativeAck`

2022-06-20 Thread GitBox
Demogorgon314 commented on code in PR #16072: URL: https://github.com/apache/pulsar/pull/16072#discussion_r901633127 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/PersistentAcknowledgmentsGroupingTracker.java: ## @@ -68,18 +74,14 @@ public class PersistentAcknowle

[GitHub] [pulsar] github-actions[bot] commented on pull request #16147: Transactions: fix race in TransactionMetaStoreHandler

2022-06-20 Thread GitBox
github-actions[bot] commented on PR #16147: URL: https://github.com/apache/pulsar/pull/16147#issuecomment-1160412440 @eolivelli Please provide a correct documentation label for your PR. Instructions see [Pulsar Documentation Label Guide](https://docs.google.com/document/d/1Qw7LHQdXWBW9t2-

[GitHub] [pulsar] eolivelli commented on issue #15375: [Transactions] NullPointerException during Java Client bootstrap

2022-06-20 Thread GitBox
eolivelli commented on issue #15375: URL: https://github.com/apache/pulsar/issues/15375#issuecomment-1160410017 @congbobo184 this is my fix https://github.com/apache/pulsar/pull/16147 -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [pulsar] eolivelli opened a new pull request, #16147: Transactions: fix race in TransactionMetaStoreHandler

2022-06-20 Thread GitBox
eolivelli opened a new pull request, #16147: URL: https://github.com/apache/pulsar/pull/16147 ### Motivation Fixes 15375 the NPE happens because internalPinnedExecutor has not been assigned yet. ### Modifications Refactor the constructor, in a way that "this" is not ac

[GitHub] [pulsar] tisonkun commented on a diff in pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
tisonkun commented on code in PR #16110: URL: https://github.com/apache/pulsar/pull/16110#discussion_r901625812 ## src/owasp-dependency-check-suppressions.xml: ## @@ -37,7 +37,19 @@ .* - + +CVE-2022-23712 is only related to Elastic server +

[GitHub] [pulsar] tisonkun commented on pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
tisonkun commented on PR #16110: URL: https://github.com/apache/pulsar/pull/16110#issuecomment-1160406665 Updated. -- 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 unsubs

[GitHub] [pulsar] tisonkun commented on pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
tisonkun commented on PR #16110: URL: https://github.com/apache/pulsar/pull/16110#issuecomment-1160405909 @nicoloboschi thanks for your explanation. Will push a commit to revert fastjson changes and do it in another PR. -- This is an automated message from the Apache Git Service. To respo

[GitHub] [pulsar] poorbarcode commented on pull request #15736: [cleanup][ledger] Remove unnecessary code in ManagedLedgerImpl

2022-06-20 Thread GitBox
poorbarcode commented on PR #15736: URL: https://github.com/apache/pulsar/pull/15736#issuecomment-1160396058 @Technoboy- Can this PR merge ^_^? -- 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] poorbarcode commented on pull request #15181: [cleanup] [broker] when serverCnx disabled auto read use same implements

2022-06-20 Thread GitBox
poorbarcode commented on PR #15181: URL: https://github.com/apache/pulsar/pull/15181#issuecomment-1160395190 @gaoran10 Can this PR merge ^_^? -- 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] Nicklee007 commented on pull request #15474: [fix][broker] Fix cannot cleanup expired ledger by trim ledgers

2022-06-20 Thread GitBox
Nicklee007 commented on PR #15474: URL: https://github.com/apache/pulsar/pull/15474#issuecomment-1160391220 ping -- 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 unsubscr

[GitHub] [pulsar] nicoloboschi commented on pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
nicoloboschi commented on PR #16110: URL: https://github.com/apache/pulsar/pull/16110#issuecomment-1160385673 @tisonkun we have to do the upgrade of fastjson in another pull. WIll merge this one even if the OWASP will fail but it's better to have two different pulls (easier to revert, to ch

[GitHub] [pulsar] nicoloboschi commented on a diff in pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
nicoloboschi commented on code in PR #16110: URL: https://github.com/apache/pulsar/pull/16110#discussion_r901605904 ## src/owasp-dependency-check-suppressions.xml: ## @@ -37,7 +37,19 @@ .* - + +CVE-2022-23712 is only related to Elastic server +

[GitHub] [pulsar] nicoloboschi commented on a diff in pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
nicoloboschi commented on code in PR #16110: URL: https://github.com/apache/pulsar/pull/16110#discussion_r901605295 ## src/owasp-dependency-check-suppressions.xml: ## @@ -37,7 +37,19 @@ .* - + +CVE-2022-23712 is only related to Elastic server +

[GitHub] [pulsar] tisonkun commented on pull request #16110: [security] Suppress CVE-2022-23712 warnings

2022-06-20 Thread GitBox
tisonkun commented on PR #16110: URL: https://github.com/apache/pulsar/pull/16110#issuecomment-1160370894 cc @nicoloboschi @lhotari -- 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 c

[pulsar] branch master updated: [feature][doc] Add docs for failure domain + anti-affinity namespace (#16069)

2022-06-20 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu 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 d5d8923265b [feature][doc] Add docs for failure d

[GitHub] [pulsar] Anonymitaet merged pull request #16069: [feature][doc] Add docs for failure domain + anti-affinity namespace

2022-06-20 Thread GitBox
Anonymitaet merged PR #16069: URL: https://github.com/apache/pulsar/pull/16069 -- 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-unsubscr...@pulsa

[GitHub] [pulsar] Anonymitaet closed issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms

2022-06-20 Thread GitBox
Anonymitaet closed issue #15904: [Doc] Load Balancer - Broker overload thresholds does not apply to some Shedding algorithms URL: https://github.com/apache/pulsar/issues/15904 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [pulsar] Anonymitaet closed issue #15629: [Doc] Add concepts to explain failure domain and anti-affinity group

2022-06-20 Thread GitBox
Anonymitaet closed issue #15629: [Doc] Add concepts to explain failure domain and anti-affinity group URL: https://github.com/apache/pulsar/issues/15629 -- 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] Jason918 commented on pull request #16144: [fix][broker]Fix removing replica clusters on topic-policies not take effect

2022-06-20 Thread GitBox
Jason918 commented on PR #16144: URL: https://github.com/apache/pulsar/pull/16144#issuecomment-1160369873 /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 the specifi

[pulsar-site] branch main updated: Docs sync done from apache/pulsar(#8730c22)

2022-06-20 Thread urfree
This is an automated email from the ASF dual-hosted git repository. urfree pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/pulsar-site.git The following commit(s) were added to refs/heads/main by this push: new 920cd0815da Docs sync done from apache/pulsar(#

svn commit: r55218 [1/2] - in /dev/pulsar/pulsar-2.9.3-candidate-1: ./ DEB/ RPMS/ RPMS/repodata/ connectors/

2022-06-20 Thread mattisonchao
Author: mattisonchao Date: Mon Jun 20 11:45:12 2022 New Revision: 55218 Log: Staging artifacts and signature for Pulsar release 2.9.3 Added: dev/pulsar/pulsar-2.9.3-candidate-1/ dev/pulsar/pulsar-2.9.3-candidate-1/DEB/ dev/pulsar/pulsar-2.9.3-candidate-1/DEB/Packages.gz (with props)

svn commit: r55218 [2/2] - in /dev/pulsar/pulsar-2.9.3-candidate-1: ./ DEB/ RPMS/ RPMS/repodata/ connectors/

2022-06-20 Thread mattisonchao
Added: dev/pulsar/pulsar-2.9.3-candidate-1/connectors/pulsar-io-hdfs3-2.9.3.nar.sha512 == --- dev/pulsar/pulsar-2.9.3-candidate-1/connectors/pulsar-io-hdfs3-2.9.3.nar.sha512 (added) +++ dev/pulsar/pulsar-2.9.3-candidate

[GitHub] [pulsar] codelipenghui commented on pull request #16108: [fix][broker][monitoring] fix message ack rate

2022-06-20 Thread GitBox
codelipenghui commented on PR #16108: URL: https://github.com/apache/pulsar/pull/16108#issuecomment-1160342368 @tjiuming Please check the failed test ``` Error: Tests run: 15, Failures: 1, Errors: 0, Skipped: 11, Time elapsed: 35.275 s <<< FAILURE! - in org.apache.pulsar.broker.s

[GitHub] [pulsar] xiaotongwang1 closed issue #16145: [BUG] ThresholdShedder Bundles Inaccurate traffic statistics

2022-06-20 Thread GitBox
xiaotongwang1 closed issue #16145: [BUG] ThresholdShedder Bundles Inaccurate traffic statistics URL: https://github.com/apache/pulsar/issues/16145 -- 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] tjiuming opened a new pull request, #16146: [branch-2.9]add message ack rate

2022-06-20 Thread GitBox
tjiuming opened a new pull request, #16146: URL: https://github.com/apache/pulsar/pull/16146 ### Motivation Merge https://github.com/apache/pulsar/pull/15674 and https://github.com/apache/pulsar/pull/16108 into branch-2.9 ### Modifications add metrics for consumer messag

[GitHub] [pulsar] xiaotongwang1 opened a new issue, #16145: [BUG] ThresholdShedder Bundles Inaccurate traffic statistics

2022-06-20 Thread GitBox
xiaotongwang1 opened a new issue, #16145: URL: https://github.com/apache/pulsar/issues/16145 broker DEBUG Log in leader : 2022-06-20 18:50:20.324 [pulsar-load-manager-1-1] INFO o.a.p.broker.loadbalance.impl.ThresholdShedder - [x.x.x.x:8080] broker is planning to shed throughput 0.0

[GitHub] [pulsar] gaoran10 commented on a diff in pull request #16142: [fix][txn] Fix NPE when ack message with transaction at cnx = null

2022-06-20 Thread GitBox
gaoran10 commented on code in PR #16142: URL: https://github.com/apache/pulsar/pull/16142#discussion_r901528330 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java: ## @@ -2679,7 +2679,14 @@ private CompletableFuture doTransactionAcknowledgeForRespons

[GitHub] [pulsar] bpereto commented on pull request #14584: [Issue 6790][pulsar-function-go] FIX: use topic name without partition in pulsar-fuction-go

2022-06-20 Thread GitBox
bpereto commented on PR #14584: URL: https://github.com/apache/pulsar/pull/14584#issuecomment-1160292489 still relevant. -- 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

  1   2   >