[GitHub] [pulsar] AnonHxy commented on a diff in pull request #15941: [fix][broker]Fix topic policies update not check message expiry

2022-06-10 Thread GitBox
AnonHxy commented on code in PR #15941: URL: https://github.com/apache/pulsar/pull/15941#discussion_r894983469 ## pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java: ## @@ -3026,7 +3026,7 @@ public void onUpdate(TopicPolicies policies) {

[GitHub] [pulsar] yuruguo closed issue #12419: Adjust the judgment logic of topic schema operation permissions

2022-06-10 Thread GitBox
yuruguo closed issue #12419: Adjust the judgment logic of topic schema operation permissions URL: https://github.com/apache/pulsar/issues/12419 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the sp

[GitHub] [pulsar] ericsyh opened a new issue, #16017: [Discuss][Doc]: Remove active-standby replication mode on Geo Replication doc

2022-06-10 Thread GitBox
ericsyh opened a new issue, #16017: URL: https://github.com/apache/pulsar/issues/16017 ### What issue do you find in Pulsar docs? I think this mode on [Geo Replication doc](https://pulsar.apache.org/zh-CN/docs/concepts-replication/#active-standby-replication) is quite easy to misunde

[GitHub] [pulsar] nodece opened a new pull request, #16016: [fix][broker] Fix token expiration

2022-06-10 Thread GitBox
nodece opened a new pull request, #16016: URL: https://github.com/apache/pulsar/pull/16016 Signed-off-by: Zixuan Liu ### Motivation When token expiration, the broker requests the client to refresh the token, then the broker performs `org.apache.pulsar.broker.service.ServerCnx

[GitHub] [pulsar] lgxbslgx commented on a diff in pull request #16008: [fix][env] Support the java version like 'XX-internal' in env file

2022-06-10 Thread GitBox
lgxbslgx commented on code in PR #16008: URL: https://github.com/apache/pulsar/pull/16008#discussion_r894975765 ## conf/pulsar_env.sh: ## @@ -60,6 +60,9 @@ for token in $("$JAVA_BIN" -version 2>&1 | grep 'version "'); do JAVA_MAJOR_VERSION=${BASH_REMATCH[1]}

[GitHub] [pulsar] Jason918 commented on pull request #16005: [Issue 15896] Fix LockTimeout when storePut on the same key concurrently in RocksdbMetadataStore

2022-06-10 Thread GitBox
Jason918 commented on PR #16005: URL: https://github.com/apache/pulsar/pull/16005#issuecomment-1152852900 /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

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

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

[GitHub] [pulsar] ericsyh opened a new pull request, #16015: fix(docs): add missing go format on client sample code

2022-06-10 Thread GitBox
ericsyh opened a new pull request, #16015: URL: https://github.com/apache/pulsar/pull/16015 *(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 #15953: [cleanup][broker]Cleanup DispatchRateLimiter isDispatchRateNeeded()

2022-06-10 Thread GitBox
Jason918 commented on PR #15953: URL: https://github.com/apache/pulsar/pull/15953#issuecomment-1152852370 @Technoboy- 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 specific comment. T

[GitHub] [pulsar] Jason918 commented on a diff in pull request #16008: [fix][env] Support the java version like 'XX-internal' in env file

2022-06-10 Thread GitBox
Jason918 commented on code in PR #16008: URL: https://github.com/apache/pulsar/pull/16008#discussion_r894974404 ## conf/pulsar_env.sh: ## @@ -60,6 +60,9 @@ for token in $("$JAVA_BIN" -version 2>&1 | grep 'version "'); do JAVA_MAJOR_VERSION=${BASH_REMATCH[1]}

[GitHub] [pulsar] Jason918 commented on a diff in pull request #16005: [Issue 15896] Fix LockTimeout when storePut on the same key concurrently in RocksdbMetadataStore

2022-06-10 Thread GitBox
Jason918 commented on code in PR #16005: URL: https://github.com/apache/pulsar/pull/16005#discussion_r894972807 ## pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataStoreTest.java: ## @@ -447,4 +448,33 @@ public void run() { assertEquals(successWrites.get

[GitHub] [pulsar] Jason918 commented on a diff in pull request #16005: [Issue 15896] Fix LockTimeout when storePut on the same key concurrently in RocksdbMetadataStore

2022-06-10 Thread GitBox
Jason918 commented on code in PR #16005: URL: https://github.com/apache/pulsar/pull/16005#discussion_r894973097 ## pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataStoreTest.java: ## @@ -447,4 +448,33 @@ public void run() { assertEquals(successWrites.get

[GitHub] [pulsar] codelipenghui commented on pull request #16007: [fix] [admin] [branch-2.9] fix reach max tenants error if the tenant already exists

2022-06-10 Thread GitBox
codelipenghui commented on PR #16007: URL: https://github.com/apache/pulsar/pull/16007#issuecomment-1152847295 @mattisonchao Need this PR to fix the test https://github.com/apache/pulsar/pull/16010 -- This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [pulsar] nodece opened a new pull request, #16014: [fix][broker] Fix create client with TLS config

2022-06-10 Thread GitBox
nodece opened a new pull request, #16014: URL: https://github.com/apache/pulsar/pull/16014 Signed-off-by: Zixuan Liu ### Motivation In PulsarService, create a client with an incorrect config. ### Modifications - Fix check TLS enable - Setup ciphers and protoco

[GitHub] [pulsar] mattisonchao commented on pull request #16007: [fix] [admin] [branch-2.9] fix reach max tenants error if the tenant already exists

2022-06-10 Thread GitBox
mattisonchao commented on PR #16007: URL: https://github.com/apache/pulsar/pull/16007#issuecomment-1152844635 /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 spe

[pulsar] branch branch-2.9 updated: Upgrade Netty Reactive Streams to 2.0.6 (#15990)

2022-06-10 Thread mattisonchao
This is an automated email from the ASF dual-hosted git repository. mattisonchao pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.9 by this push: new 88c5681b219 Upgrade Netty Reactive

[GitHub] [pulsar] github-actions[bot] commented on issue #13453: [Website] generate HTML for Pulsar configuration pages

2022-06-10 Thread GitBox
github-actions[bot] commented on issue #13453: URL: https://github.com/apache/pulsar/issues/13453#issuecomment-1152835890 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 issue #14861: Broker OOM when the bookie is slowly to respond and AQ is smaller than WQ

2022-06-10 Thread GitBox
github-actions[bot] commented on issue #14861: URL: https://github.com/apache/pulsar/issues/14861#issuecomment-1152835608 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 issue #15037: PIP-151: Use the system topic to store the bundle load data

2022-06-10 Thread GitBox
github-actions[bot] commented on issue #15037: URL: https://github.com/apache/pulsar/issues/15037#issuecomment-1152835564 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 issue #15101: When notify wait opReadEntry, the readPosition calculated policy is different from opReadEntry.create()

2022-06-10 Thread GitBox
github-actions[bot] commented on issue #15101: URL: https://github.com/apache/pulsar/issues/15101#issuecomment-1152835557 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 issue #15113: ResourceGroupConfigListener and ResourceGroupNamespaceConfigListener has an cocurrent bug at startrup

2022-06-10 Thread GitBox
github-actions[bot] commented on issue #15113: URL: https://github.com/apache/pulsar/issues/15113#issuecomment-115283 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 issue #15477: Python Pulsar-Client: Import error with Python 3.10

2022-06-10 Thread GitBox
github-actions[bot] commented on issue #15477: URL: https://github.com/apache/pulsar/issues/15477#issuecomment-1152835542 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 issue #15528: PIP-161 Exclusive Producer: new mode ExclusiveWithFencing

2022-06-10 Thread GitBox
github-actions[bot] commented on issue #15528: URL: https://github.com/apache/pulsar/issues/15528#issuecomment-1152835533 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

[pulsar] branch master updated: [fix][admin] Fix get-publish-rete Admin API handle exception behavior (#16001)

2022-06-10 Thread mattisonchao
This is an automated email from the ASF dual-hosted git repository. mattisonchao 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 addfbd351c9 [fix][admin] Fix get-publish-r

[GitHub] [pulsar] mattisonchao merged pull request #16001: [fix][admin] Fix get-publish-rete Admin API handle exception behavior

2022-06-10 Thread GitBox
mattisonchao merged PR #16001: URL: https://github.com/apache/pulsar/pull/16001 -- 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: Add comment for using `configurationMetadataStoreUrl` instead of `configurationStoreServers` (#16012)

2022-06-10 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 c4ee1097087 Add comment for using `configuration

[GitHub] [pulsar] merlimat merged pull request #16012: Add comment for using `configurationMetadataStoreUrl` instead of `configurationStoreServers`

2022-06-10 Thread GitBox
merlimat merged PR #16012: URL: https://github.com/apache/pulsar/pull/16012 -- 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] lgxbslgx commented on issue #15696: Java pulsar function can not subscribe to a non-defined schema topic

2022-06-10 Thread GitBox
lgxbslgx commented on issue #15696: URL: https://github.com/apache/pulsar/issues/15696#issuecomment-1152628937 > Steps to reproduce the behavior: > > Create new topic with no schema defined > Create a new Java function to subscribe from the topic > The function can not subscribe

[GitHub] [pulsar] david-streamlio opened a new issue, #16013: Pulsar Standalone Docker image does not work.

2022-06-10 Thread GitBox
david-streamlio opened a new issue, #16013: URL: https://github.com/apache/pulsar/issues/16013 **Describe the bug** Following the instructions on the Apache Pulsar documentation site found [here](https://pulsar.apache.org/docs/getting-started-docker), the pulsar cluster fails to start.

[GitHub] [pulsar] devinbost commented on issue #8666: java client Metadata is not available for client consumption

2022-06-10 Thread GitBox
devinbost commented on issue #8666: URL: https://github.com/apache/pulsar/issues/8666#issuecomment-1152620162 Does this issue occur with other mismatched Pulsar versions? We see it a lot. -- This is an automated message from the Apache Git Service. To respond to the message, please log

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

2022-06-10 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 22604033a87 Docs sync done from apache/pulsar(#

[GitHub] [pulsar] coderzc commented on pull request #15678: [improve][proxy] Support zero-copy of NIC to NIC on Proxy

2022-06-10 Thread GitBox
coderzc commented on PR #15678: URL: https://github.com/apache/pulsar/pull/15678#issuecomment-1152604387 /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] github-actions[bot] commented on pull request #16012: Add comment for using `configurationMetadataStoreUrl` instead of `configurationStoreServers`

2022-06-10 Thread GitBox
github-actions[bot] commented on PR #16012: URL: https://github.com/apache/pulsar/pull/16012#issuecomment-1152599107 @ZiyaoWei:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/master/.gi

[GitHub] [pulsar] github-actions[bot] commented on pull request #16012: Add comment for using `configurationMetadataStoreUrl` instead of `configurationStoreServers`

2022-06-10 Thread GitBox
github-actions[bot] commented on PR #16012: URL: https://github.com/apache/pulsar/pull/16012#issuecomment-1152599147 @ZiyaoWei:Thanks for providing doc info! -- 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] ZiyaoWei opened a new pull request, #16012: Add comment for using `configurationMetadataStoreUrl` instead of `configurationStoreServers`

2022-06-10 Thread GitBox
ZiyaoWei opened a new pull request, #16012: URL: https://github.com/apache/pulsar/pull/16012 ### Motivation #13777 deprecated `zookeeperServers` and `configurationStoreServers`. However, while the former has comments pointing to `metadataStoreUrl` which should be used instead, the la

[GitHub] [pulsar] coderzc commented on pull request #15915: [fix][client] Add classLoader field for `SchemaDefinition`

2022-06-10 Thread GitBox
coderzc commented on PR #15915: URL: https://github.com/apache/pulsar/pull/15915#issuecomment-1152575320 /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

[pulsar] branch master updated: [fix][clients]Check pendingIndividualBatchIndexAcks size in doIndividualBatchAckAsync (#15877)

2022-06-10 Thread zhangmingao
This is an automated email from the ASF dual-hosted git repository. zhangmingao 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 de9928a5bb4 [fix][clients]Check pendingIndi

[GitHub] [pulsar] gaozhangmin merged pull request #15877: [fix][clients]Check pendingIndividualBatchIndexAcks size in doIndividualBatchAckAsync

2022-06-10 Thread GitBox
gaozhangmin merged PR #15877: URL: https://github.com/apache/pulsar/pull/15877 -- 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] heesung-sn commented on pull request #15762: [improve][deploy] changed the default GC options to ZGC from G1GC

2022-06-10 Thread GitBox
heesung-sn commented on PR #15762: URL: https://github.com/apache/pulsar/pull/15762#issuecomment-1152554059 Raised PR to fix the test issues: https://github.com/apache/pulsar/pull/16011 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] heesung-sn opened a new pull request, #16011: [improve][tests] improved flaky test runs

2022-06-10 Thread GitBox
heesung-sn opened a new pull request, #16011: URL: https://github.com/apache/pulsar/pull/16011 *(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] codelipenghui opened a new pull request, #16010: [branch-2.9] Disable testReuseFork for broker_group_2

2022-06-10 Thread GitBox
codelipenghui opened a new pull request, #16010: URL: https://github.com/apache/pulsar/pull/16010 Fix https://github.com/apache/pulsar/runs/6833250379?check_suite_focus=true ``` Error: Tests run: 4, Failures: 1, Errors: 0, Skipped: 3, Time elapsed: 9.736 s <<< FAILURE! - in org.a

[GitHub] [pulsar] coderzc commented on pull request #15915: [fix][client] Add classLoader field for `SchemaDefinition`

2022-06-10 Thread GitBox
coderzc commented on PR #15915: URL: https://github.com/apache/pulsar/pull/15915#issuecomment-1152526327 /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] tisonkun commented on pull request #16004: [cleanup][pom] Remove unused TestNG conf files and clean up repeated notes

2022-06-10 Thread GitBox
tisonkun commented on PR #16004: URL: https://github.com/apache/pulsar/pull/16004#issuecomment-1152513679 cc @eolivelli could you take a look at 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 to

[GitHub] [pulsar] zbentley opened a new issue, #16009: When using the Python pulsar client, crashes occasionally happen on connect/disconnect/close due to pthread_mutex_lock assertion failures

2022-06-10 Thread GitBox
zbentley opened a new issue, #16009: URL: https://github.com/apache/pulsar/issues/16009 **Describe the bug** Occasionally, in flakey CI tests that run Python code in a heavily threaded environment, we have segfaults when calling `connect`, `create_producer`, or `producer.close()` on Puls

[GitHub] [pulsar] liangyuanpeng commented on pull request #15907: [feature][functions]Support auth for function go

2022-06-10 Thread GitBox
liangyuanpeng commented on PR #15907: URL: https://github.com/apache/pulsar/pull/15907#issuecomment-1152493537 /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] nodece commented on a diff in pull request #15761: [fix][client] Remove consumer when close consumer command is received

2022-06-10 Thread GitBox
nodece commented on code in PR #15761: URL: https://github.com/apache/pulsar/pull/15761#discussion_r894640933 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java: ## @@ -115,6 +115,7 @@ public class ClientCnx extends PulsarHandler { .

[GitHub] [pulsar] e-marchand-exensa commented on issue #13712: [2.8.2] error while create opSendMsg by batch message container - zstd overflow detected

2022-06-10 Thread GitBox
e-marchand-exensa commented on issue #13712: URL: https://github.com/apache/pulsar/issues/13712#issuecomment-1152470705 The issue still exists with version 2.10.0. Still no problem with 2.6.4. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

[pulsar] 07/12: [Revert] [#15483] Remove sensitive msg from consumer/producer stats log (#15817)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 4a471d06ce173a99850866c095c54c88810e2ee4 Author: ZhangJian He AuthorDate: Sun Jun 5 09:44:47 2022 +0800 [Rever

[pulsar] 09/12: [fix][txn]Fix transasction ack batch message (#15875)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit faac51937d2f9c95811e85446b3319f877dc9fd4 Author: Xiangying Meng <55571188+liangyepianz...@users.noreply.github.com>

[pulsar] 11/12: [Fix][broker] Fix NPE when ledger id not found in `OpReadEntry` (#15837)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 53615e05cff4010ef0834f15f057b78c1eb6fc77 Author: Qiang Zhao AuthorDate: Tue Jun 7 21:52:05 2022 +0800 [Fix][br

[pulsar] 06/12: Fix avro conversion order of registration (#15863)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit d6565a5d3dac9910a8c361a4a08a3a7faf101331 Author: Cong Zhao AuthorDate: Wed Jun 1 16:58:48 2022 +0800 Fix avro

[pulsar] 04/12: fix bug in getNumberOfEntriesInStorage (#15627)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit c1e6606682053c2ad38d141faaab7314a910c692 Author: Qiang Huang AuthorDate: Sat May 28 10:58:35 2022 +0800 fix bu

[pulsar] 03/12: [feat] [tiered-storage] Add pure S3 provider for the offloader (#15710)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit b2de04362b302d8d4294e6e4073c9d046f9bb84d Author: Yong Zhang AuthorDate: Wed May 25 15:37:33 2022 +0800 [feat]

[pulsar] 05/12: Fix NPE in MessageDeduplication. (#15820)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 89e005f939fa4400bc1be5a9058264d9e071622a Author: Jiwei Guo AuthorDate: Wed Jun 1 11:09:01 2022 +0800 Fix NPE i

[pulsar] branch branch-2.9 updated (43234c63bab -> 3411bf21176)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git from 43234c63bab Allow PULSAR_MEM & PULSAR_GC to be Overridden in pulsar_tool_env.sh (#15868) new 55e4a1d66a7 [c

[pulsar] 12/12: [fix][txn] fix NPE of TransactionMetaStoreHandler (#15840)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 3411bf211761dbbdf9f78a5b768ca30a52363148 Author: congbo <39078850+congbobo...@users.noreply.github.com> AuthorDate:

[pulsar] 10/12: [fix][auth] Generate correct well-known OpenID configuration URL (#15928)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 43ab20b735f3a7cd989369725206c79d82b4d4f0 Author: ran AuthorDate: Tue Jun 7 15:46:57 2022 +0800 [fix][auth] Gen

[pulsar] 08/12: [fix][broker]Fast return if ack cumulative illegal (#15695)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit dac1c6f39d17b4f64c5dcc8ccd342fc88ecbc2af Author: Xiaoyu Hou AuthorDate: Mon Jun 6 10:59:55 2022 +0800 [fix][br

[pulsar] 01/12: [cleanup] [broker] Remove useless code to avoid confusion in OpReadEntry#checkReadCompletion. (#15104)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 55e4a1d66a7cd7853d37c76a47fcb965bf3e937c Author: 赵延 AuthorDate: Mon Apr 11 12:34:36 2022 +0800 [cleanup] [brok

[pulsar] 02/12: [fix][broker] Fix MultiRolesTokenAuthorizationProvider `authorize` issue. (#15454)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 960b80f59e8b8700e6a5d30aca7ceb784d267ced Author: Jiwei Guo AuthorDate: Mon May 9 09:11:19 2022 +0800 [fix][bro

[GitHub] [pulsar] lgxbslgx opened a new pull request, #16008: [fix][env] Support the java version like 'XX-internal' in env file

2022-06-10 Thread GitBox
lgxbslgx opened a new pull request, #16008: URL: https://github.com/apache/pulsar/pull/16008 Related to #15670, cc'ing its author @lhotari . ### Motivation My local JDKs are almost built from the source code, such as [JDK Mainline](https://github.com/openjdk/jdk), so when I use

[GitHub] [pulsar] poorbarcode opened a new pull request, #16007: [fix] [admin] [branch-2.9] fix reach max tenants error if the tenant already exists

2022-06-10 Thread GitBox
poorbarcode opened a new pull request, #16007: URL: https://github.com/apache/pulsar/pull/16007 ### Motivation see #15932 At branch `master` REST API async change, there are many conflicts when cherry-picking #15932 PR, so I create separate PR to fix branch-2.9 and branch-2.1

[GitHub] [pulsar] wuxuanqicn commented on a diff in pull request #16005: [Issue 15896] Fix LockTimeout when storePut on the same key concurrently in RocksdbMetadataStore

2022-06-10 Thread GitBox
wuxuanqicn commented on code in PR #16005: URL: https://github.com/apache/pulsar/pull/16005#discussion_r894598868 ## pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataStoreTest.java: ## @@ -447,4 +448,33 @@ public void run() { assertEquals(successWrites.g

[GitHub] [pulsar] wuxuanqicn commented on a diff in pull request #16005: [Issue 15896] Fix LockTimeout when storePut on the same key concurrently in RocksdbMetadataStore

2022-06-10 Thread GitBox
wuxuanqicn commented on code in PR #16005: URL: https://github.com/apache/pulsar/pull/16005#discussion_r894598868 ## pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataStoreTest.java: ## @@ -447,4 +448,33 @@ public void run() { assertEquals(successWrites.g

[pulsar] branch branch-2.10 updated: Fix failed tests for branch-2.10 (#15995)

2022-06-10 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.10 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.10 by this push: new 0b7bacc448a Fix failed tests for bran

[GitHub] [pulsar] codelipenghui merged pull request #15995: Fix failed tests for branch-2.10

2022-06-10 Thread GitBox
codelipenghui merged PR #15995: URL: https://github.com/apache/pulsar/pull/15995 -- 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...@pul

[GitHub] [pulsar] nicoloboschi merged pull request #15428: [feat][elasticsearch] Add hashed id support

2022-06-10 Thread GitBox
nicoloboschi merged PR #15428: URL: https://github.com/apache/pulsar/pull/15428 -- 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: [feat][elasticsearch] Add hashed id support (#15428)

2022-06-10 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 67224175bfb [feat][elasticsearch] Add hash

[GitHub] [pulsar] tisonkun commented on pull request #16004: [cleanup][pom] Remove unused TestNG conf files and clean up repeated notes

2022-06-10 Thread GitBox
tisonkun commented on PR #16004: URL: https://github.com/apache/pulsar/pull/16004#issuecomment-1152342671 @codelipenghui could you take a look at this PR? I'm unsure who is supposed to request review. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] codelipenghui commented on pull request #15995: Fix failed tests for branch-2.10

2022-06-10 Thread GitBox
codelipenghui commented on PR #15995: URL: https://github.com/apache/pulsar/pull/15995#issuecomment-1152336144 > @codelipenghui can you please add in the description the reason why we need to set -DtestReuseFork=false ? Not 100% sure what's going on for now, but looks like It's relate

[GitHub] [pulsar] codelipenghui commented on issue #16000: [test] NPE of StringSchema for branch-2.10

2022-06-10 Thread GitBox
codelipenghui commented on issue #16000: URL: https://github.com/apache/pulsar/issues/16000#issuecomment-1152329048 Sorry @eolivelli I'm still testing. It's not a JDK issue but related to `reuseFork`. I have disabled the `reuseFork in PR #15995`, the test gets passed. Looks like the

[GitHub] [pulsar] codelipenghui commented on a diff in pull request #16005: [Issue 15896] Fix LockTimeout when storePut on the same key concurrently in RocksdbMetadataStore

2022-06-10 Thread GitBox
codelipenghui commented on code in PR #16005: URL: https://github.com/apache/pulsar/pull/16005#discussion_r894488726 ## pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataStoreTest.java: ## @@ -447,4 +448,33 @@ public void run() { assertEquals(successWrite

[GitHub] [pulsar] eolivelli commented on issue #16000: [test] NPE of StringSchema for branch-2.10

2022-06-10 Thread GitBox
eolivelli commented on issue #16000: URL: https://github.com/apache/pulsar/issues/16000#issuecomment-1152325095 @codelipenghui @nicoloboschi is there any link to a OpenJDK issue ? I can create tickets on OpenJDK JIRA, if it is a new issue we can report it but we need more context -- Th

[GitHub] [pulsar] Jaudouard commented on issue #14714: cpp-tests job is flaky: python pulsar_test.py fails with Segmentation fault

2022-06-10 Thread GitBox
Jaudouard commented on issue #14714: URL: https://github.com/apache/pulsar/issues/14714#issuecomment-1152320473 Thanks for your help, the workaround might well be helpful if we manage to apply it only to our product topics. I've created a simple bug report: https://github.com/apache/

[GitHub] [pulsar] Jaudouard opened a new issue, #16006: [python client] cannot close consumer after an unsubscribe

2022-06-10 Thread GitBox
Jaudouard opened a new issue, #16006: URL: https://github.com/apache/pulsar/issues/16006 **Describe the bug** Currently, calling close() after unsubscribe() with the latest python client crashed and the connection with pulsar is not fully released if close() is not called, causing a leak

[GitHub] [pulsar] github-actions[bot] commented on pull request #16005: [Issue 15896] Fix LockTimeout when storePut on the same key concurrently in RocksdbMetadataStore

2022-06-10 Thread GitBox
github-actions[bot] commented on PR #16005: URL: https://github.com/apache/pulsar/pull/16005#issuecomment-1152302296 @Jason918:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/master/.gi

[GitHub] [pulsar] michaeljmarshall commented on a diff in pull request #14795: [feature][doc] Add docs for system topic and update for topic policy

2022-06-10 Thread GitBox
michaeljmarshall commented on code in PR #14795: URL: https://github.com/apache/pulsar/pull/14795#discussion_r894453371 ## site2/docs/concepts-messaging.md: ## @@ -857,6 +857,38 @@ Producer producer = client.newProducer() ``` +## System topic + +System topic is a predefined

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

2022-06-10 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 fd813fb2379 Docs sync done from apache/pulsar(#

[GitHub] [pulsar] Jason918 commented on issue #15896: Flaky-test: PulsarLedgerIdGeneratorTest.testGenerateLedgerId

2022-06-10 Thread GitBox
Jason918 commented on issue #15896: URL: https://github.com/apache/pulsar/issues/15896#issuecomment-1152288335 @wuxuanqicn I have pushed a PR #16005 to fix this. PTAL. -- 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] Jason918 opened a new pull request, #16005: [Issue 15896] Fix LockTimeout when storePut on the same key concurrently in RocksdbMetadataStore

2022-06-10 Thread GitBox
Jason918 opened a new pull request, #16005: URL: https://github.com/apache/pulsar/pull/16005 Fixes #15896 ### Motivation See #15896 When the same key is modified concurrently, there will be LockTimeout exception in `RocksdbMetadataStore`. The r

[GitHub] [pulsar] nicoloboschi commented on pull request #15428: [feat][elasticsearch] Add hashed id support

2022-06-10 Thread GitBox
nicoloboschi commented on PR #15428: URL: https://github.com/apache/pulsar/pull/15428#issuecomment-1152280439 /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 s

[GitHub] [pulsar] AnonHxy commented on pull request #15825: [fix][broker]Fix topic-level replicator rate limiter not init

2022-06-10 Thread GitBox
AnonHxy commented on PR #15825: URL: https://github.com/apache/pulsar/pull/15825#issuecomment-1152273690 /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] tisonkun opened a new pull request, #16004: [cleanup][pom] Remove unused TestNG conf files and clean up repeated notes

2022-06-10 Thread GitBox
tisonkun opened a new pull request, #16004: URL: https://github.com/apache/pulsar/pull/16004 This is a chore job to cleanup files. ### Motivation Although https://github.com/cbeust/testng/issues/508 is open, we should take care of TestNG configuration when config `maven-surefir

[GitHub] [pulsar] Demogorgon314 commented on pull request #16001: [fix][admin] Fix get-publish-rete Admin API handle exception behavior

2022-06-10 Thread GitBox
Demogorgon314 commented on PR #16001: URL: https://github.com/apache/pulsar/pull/16001#issuecomment-1152247332 /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 sp

[GitHub] [pulsar] Demogorgon314 commented on pull request #15815: [fix][test] Fix schema not found check on unit test

2022-06-10 Thread GitBox
Demogorgon314 commented on PR #15815: URL: https://github.com/apache/pulsar/pull/15815#issuecomment-1152247184 /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 sp

[GitHub] [pulsar] lgxbslgx commented on a diff in pull request #15998: [doc] Add a step about JDK17 configuration to the Pulsar SQL document

2022-06-10 Thread GitBox
lgxbslgx commented on code in PR #15998: URL: https://github.com/apache/pulsar/pull/15998#discussion_r894398589 ## site2/docs/sql-getting-started.md: ## @@ -21,23 +21,34 @@ To query data in Pulsar with Pulsar SQL, complete the following steps. ``` -2. Start a Pulsar SQL wo

[GitHub] [pulsar] AnonHxy commented on pull request #15825: [fix][broker]Fix topic-level replicator rate limiter not init

2022-06-10 Thread GitBox
AnonHxy commented on PR #15825: URL: https://github.com/apache/pulsar/pull/15825#issuecomment-1152230333 /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] shink commented on pull request #16003: [feature][pulsar-io-mongo] Add support for full message synchronization

2022-06-10 Thread GitBox
shink commented on PR #16003: URL: https://github.com/apache/pulsar/pull/16003#issuecomment-1152212638 I will finish UT later. Convert this PR to a draft temporarily. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [pulsar] shink commented on a diff in pull request #16003: [feature][pulsar-io-mongo] Add support for full message synchronization

2022-06-10 Thread GitBox
shink commented on code in PR #16003: URL: https://github.com/apache/pulsar/pull/16003#discussion_r894378935 ## pulsar-io/mongo/src/main/java/org/apache/pulsar/io/mongodb/MongoSourceConfig.java: ## @@ -0,0 +1,102 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] [pulsar] shink commented on a diff in pull request #16003: [feature][pulsar-io-mongo] Add support for full message synchronization

2022-06-10 Thread GitBox
shink commented on code in PR #16003: URL: https://github.com/apache/pulsar/pull/16003#discussion_r894377745 ## pulsar-io/mongo/src/main/java/org/apache/pulsar/io/mongodb/MongoSource.java: ## @@ -79,38 +78,46 @@ public MongoSource(Supplier clientProvider) { public void open

[GitHub] [pulsar] shink commented on a diff in pull request #16003: [feature][pulsar-io-mongo] Add support for full message synchronization

2022-06-10 Thread GitBox
shink commented on code in PR #16003: URL: https://github.com/apache/pulsar/pull/16003#discussion_r894377258 ## pulsar-io/mongo/src/main/java/org/apache/pulsar/io/mongodb/MongoSource.java: ## @@ -51,19 +52,17 @@ name = "mongo", type = IOType.SOURCE, he

[GitHub] [pulsar] codelipenghui commented on a diff in pull request #15761: [fix][client] Remove consumer when close consumer command is received

2022-06-10 Thread GitBox
codelipenghui commented on code in PR #15761: URL: https://github.com/apache/pulsar/pull/15761#discussion_r894376775 ## pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java: ## @@ -115,6 +115,7 @@ public class ClientCnx extends PulsarHandler {

[GitHub] [pulsar] shink opened a new pull request, #16003: [feature][pulsar-io-mongo] Add support for full message synchronization

2022-06-10 Thread GitBox
shink opened a new pull request, #16003: URL: https://github.com/apache/pulsar/pull/16003 ### Motivation Now, the MongoDB source connector only supports the incremental message synchronization. This PR adds support for full message synchronization. Since MongDB 4.0, we can s

[GitHub] [pulsar] momo-jun commented on a diff in pull request #15809: Add new draft of architecture-overview.md and accompanying images, add under-construction.md and accompanying images

2022-06-10 Thread GitBox
momo-jun commented on code in PR #15809: URL: https://github.com/apache/pulsar/pull/15809#discussion_r894365788 ## site2/docs/architecture-overview.md: ## @@ -0,0 +1,143 @@ +--- + +id: concepts-architecture-overview + +title: Architecture overview + +sidebar_label: Concepts + +-

[GitHub] [pulsar] BewareMyPower commented on issue #14714: cpp-tests job is flaky: python pulsar_test.py fails with Segmentation fault

2022-06-10 Thread GitBox
BewareMyPower commented on issue #14714: URL: https://github.com/apache/pulsar/issues/14714#issuecomment-1152177219 Got it. I think one workaround is to modify the `brokerDeleteInactiveTopicsMode` config of the broker and acknowledge messages. BTW, I just tested `close` after `unsubsc

[GitHub] [pulsar-client-go] hriday-ns closed pull request #787: Test msg stuck

2022-06-10 Thread GitBox
hriday-ns closed pull request #787: Test msg stuck URL: https://github.com/apache/pulsar-client-go/pull/787 -- 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-ma

[GitHub] [pulsar-client-go] hriday-ns opened a new pull request, #787: Test msg stuck

2022-06-10 Thread GitBox
hriday-ns opened a new pull request, #787: URL: https://github.com/apache/pulsar-client-go/pull/787 <-- ### 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] poorbarcode commented on issue #15370: [PIP-160] Batch writing ledger for transaction operation

2022-06-10 Thread GitBox
poorbarcode commented on issue #15370: URL: https://github.com/apache/pulsar/issues/15370#issuecomment-1152174215 @eolivelli @gaoran10 @gaozhangmin @315157973 @lhotari @michaeljmarshall @congbobo184 @Technoboy- I have start a vote and already send email, could you take a look. Disc

[GitHub] [pulsar] Jaudouard commented on issue #14714: cpp-tests job is flaky: python pulsar_test.py fails with Segmentation fault

2022-06-10 Thread GitBox
Jaudouard commented on issue #14714: URL: https://github.com/apache/pulsar/issues/14714#issuecomment-1152164222 > Could you explain it in detail? It's possible to delete the topic even if it has a subscription unless the subscription is active. It is indeed possible but it requires a

[GitHub] [pulsar] eolivelli commented on a diff in pull request #15979: [fix][broker] fix DEFAULT_NAR_EXTRACTION_DIR cause NoClassDefFoundError

2022-06-10 Thread GitBox
eolivelli commented on code in PR #15979: URL: https://github.com/apache/pulsar/pull/15979#discussion_r894331214 ## pulsar-common/src/main/java/org/apache/pulsar/common/nar/NarClassLoader.java: ## @@ -139,7 +139,7 @@ public class NarClassLoader extends URLClassLoader { pr

  1   2   >