[GitHub] [pulsar] lhotari edited a comment on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
lhotari edited a comment on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990708597 I can confirm that JVM system property `-Dlog4j2.formatMsgNoLookups=true` prevents the issue. This workaround applies for Log4j >= 2.10.0 . I'm working on a Pulsar Helm

[GitHub] [pulsar] lhotari commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
lhotari commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990708597 I can confirm that JVM system property `-Dlog4j2.formatMsgNoLookups=true` prevents the issue. I'm working on a Pulsar Helm Chart change that passes that automatically for all

[GitHub] [pulsar] Anonymitaet commented on pull request #13077: PIP-45: Allow to configure metadata store URL in broker.conf

2021-12-09 Thread GitBox
Anonymitaet commented on pull request #13077: URL: https://github.com/apache/pulsar/pull/13077#issuecomment-990679671 For the doc side, I've discussed with @codelipenghui, the following changes will be made after 2.9.0 is released: 1. Change `zookeeperServers` to `metadataStoreUrl`

[GitHub] [pulsar] lhotari commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
lhotari commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-99064 The workaround seems to be to set `log4j2.formatMsgNoLookups` to `true`. reference: https://twitter.com/80vul/status/1468968891489857537 -- This is an automated message

[GitHub] [pulsar-client-go] wolfstudy merged pull request #683: Add properties filed for batch container

2021-12-09 Thread GitBox
wolfstudy merged pull request #683: URL: https://github.com/apache/pulsar-client-go/pull/683 -- 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:

[pulsar-client-go] branch master updated: Add properties filed for batch (#683)

2021-12-09 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new d0d5d0a Add properties filed for batch

[GitHub] [pulsar] lhotari commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
lhotari commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990669717 I also cherry-picked to branch-2.6 . I think it would make sense to do a 2.6.5 release since CVE-2021-44228 is such a severe issue. @merlimat @eolivelli @massakam WDYT? --

[pulsar] branch branch-2.6 updated: Bump log4j to 2.15.0 (#13226)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch branch-2.6 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.6 by this push: new f7c6bdb Bump log4j to 2.15.0 (#13226)

[pulsar] branch branch-2.7 updated: [logging] Use Log4j2's BOM to ensure that Log4j2 version is consistent. Replace legacy log4j with log4j-1.2-api

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch branch-2.7 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.7 by this push: new 6fee20f [logging] Use Log4j2's BOM to

[pulsar] branch branch-2.9 updated: Bump log4j to 2.15.0 (#13226)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 6cdebb7 Bump log4j to 2.15.0 (#13226)

[GitHub] [pulsar-client-go] wolfstudy opened a new pull request #683: Add properties filed for batch container

2021-12-09 Thread GitBox
wolfstudy opened a new pull request #683: URL: https://github.com/apache/pulsar-client-go/pull/683 Signed-off-by: xiaolongran ### Motivation Currently, when we disable batch in Producer, in `handleSend()` of `serverCnx.java`, the `msgMetadata.hasNumMessagesInBatch()` is

[pulsar] branch branch-2.7 updated: Bump log4j to 2.15.0 (#13226)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch branch-2.7 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.7 by this push: new dea0f43 Bump log4j to 2.15.0 (#13226)

[GitHub] [pulsar-client-go] imryao opened a new issue #682: [Doc] add docs for producer metrics

2021-12-09 Thread GitBox
imryao opened a new issue #682: URL: https://github.com/apache/pulsar-client-go/issues/682 **Is your feature request related to a problem? Please describe.** Hi there! I found there's not much documentation about producer metrics of pulsar go client. **Describe the solution you'd

[pulsar] branch branch-2.8 updated: Bump log4j to 2.15.0 (#13226)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 b136a73 Bump log4j to 2.15.0 (#13226)

[GitHub] [pulsar] imryao commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
imryao commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990655002 > > > which vulnerability is it? severity level > > > > > > Very serious > > @315157973 I don't see any vulnerability listed here:

[GitHub] [pulsar] lhotari commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
lhotari commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990653309 @315157973 I'll cherry pick to branch-2.8 and branch-2.7 . Since the issue is extremely severe, I think we need to expedite 2.8.2 and 2.7.4 releases. /cc @michaeljmarshall

[GitHub] [pulsar] massakam commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
massakam commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990647233 FYI https://www.lunasec.io/docs/blog/log4j-zero-day/ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] lhotari commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
lhotari commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990643946 > > which vulnerability is it? severity level > > Very serious @315157973 I don't see any vulnerability listed here:

[pulsar] branch master updated (02b8de0 -> 0015daba)

2021-12-09 Thread massakam
This is an automated email from the ASF dual-hosted git repository. massakam pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 02b8de0 [Broker] Optimize ManagedLedger Ledger Ownership Check (#13222) add 0015daba Bump log4j to 2.15.0

[GitHub] [pulsar] massakam merged pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
massakam merged pull request #13226: URL: https://github.com/apache/pulsar/pull/13226 -- 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:

[GitHub] [pulsar] ericsyh opened a new pull request #13230: feat(docs): add more limitations for replicated subscriptions

2021-12-09 Thread GitBox
ericsyh opened a new pull request #13230: URL: https://github.com/apache/pulsar/pull/13230 Signed-off-by: ericsyh ### Motivation The current doc about [Replicated subscriptions Limitations](https://pulsar.apache.org/docs/en/administration-geo/#limitations) only

[GitHub] [pulsar] 315157973 commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
315157973 commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990636783 > which vulnerability is it? severity level Very serious -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[pulsar] 01/02: [c++] Define and expose PULSAR_VERSION macro (#12769)

2021-12-09 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 7e4a93a0f012144ff275deb68a67d2cc09e6ec6c Author: Masahiro Sakamoto AuthorDate: Fri Nov 26 11:43:13 2021 +0900

[pulsar] 02/02: [C++]Fix libcurl miss auth header when broker return 307 (#13112)

2021-12-09 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 07c6fb2891367257be763c2c9ff74da7e6f6ec70 Author: billowqiu AuthorDate: Thu Dec 9 10:44:10 2021 +0800 [C++]Fix

[pulsar] branch branch-2.8 updated (b72d452 -> 07c6fb2)

2021-12-09 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz pushed a change to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git. from b72d452 [Java Client] Send CloseProducer on timeout (#13161) new 7e4a93a [c++] Define and expose

[GitHub] [pulsar] lhotari commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
lhotari commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990628863 > Should we add this PR to 2.8.2 since there is a security vulnerability for log4j < 2.15? @315157973 @BewareMyPower which vulnerability is it? severity level? --

[GitHub] [pulsar] nlu90 edited a comment on pull request #13205: [PIP 86][Function] add funtion api to support preload and release external resources

2021-12-09 Thread GitBox
nlu90 edited a comment on pull request #13205: URL: https://github.com/apache/pulsar/pull/13205#issuecomment-990627687 @wangjialing218 Did you call a vote for the PIP-86 in the dev@pulsar mailing list? Also you may want to update the PIP-86 content to reflect the newest idea. It's

[GitHub] [pulsar] nlu90 commented on pull request #13205: [PIP 86][Function] add funtion api to support preload and release external resources

2021-12-09 Thread GitBox
nlu90 commented on pull request #13205: URL: https://github.com/apache/pulsar/pull/13205#issuecomment-990627687 @wangjialing218 Did you call a vote for the PIP-86 in the dev@pulsar mailing list? It's good if we can get feedback from the community. And then start the review work.

[pulsar] 02/02: [Java Client] Send CloseProducer on timeout (#13161)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 168bbab4ca5d13afda5eaa173dbcf9f923f22427 Author: Michael Marshall AuthorDate: Thu Dec 9 13:17:13 2021 -0600

[pulsar] 01/02: Fix namespace policy override ignored when creating subscription (#12699)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit a749d564ff361a4cb7cf7d49fb014e5bfcca4dd0 Author: Christophe Bornet AuthorDate: Thu Dec 9 18:47:50 2021 +0100

[pulsar] branch branch-2.9 updated (de2ad45 -> 168bbab)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git. from de2ad45 [Issue 12796][broker] Fix replicator error with user defined ReplicatorPrefix (#12797) new

[GitHub] [pulsar] lhotari closed pull request #13122: Trigger a build for branch-2.8 to see if the build passes

2021-12-09 Thread GitBox
lhotari closed pull request #13122: URL: https://github.com/apache/pulsar/pull/13122 -- 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:

[pulsar] 01/02: Fix namespace policy override ignored when creating subscription (#12699)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit a034d14ebaa8320be0cb17dd145742ef70d0aaa0 Author: Christophe Bornet AuthorDate: Thu Dec 9 18:47:50 2021 +0100

[pulsar] 02/02: [Java Client] Send CloseProducer on timeout (#13161)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit b72d452a91b70b6949762700ee18c26bf3e832c2 Author: Michael Marshall AuthorDate: Thu Dec 9 13:17:13 2021 -0600

[pulsar] branch branch-2.8 updated (9b8e7d2 -> b72d452)

2021-12-09 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 9b8e7d2 [ServerCnx] Close connection after receiving unexpected SendCommand (#12780) new a034d14 Fix

[GitHub] [pulsar] tuteng commented on issue #13174: Pulsar admin api golang client

2021-12-09 Thread GitBox
tuteng commented on issue #13174: URL: https://github.com/apache/pulsar/issues/13174#issuecomment-990619063 https://github.com/streamnative/pulsarctl/blob/master/docs/en/overview_of_pulsarctl.md#admin-api This sdk for admin api https://github.com/apache/pulsar-client-go This sdk for

[GitHub] [pulsar] Anonymitaet opened a new issue #13229: PIP 45: Update zookeeperServers to metadataStoreUrl

2021-12-09 Thread GitBox
Anonymitaet opened a new issue #13229: URL: https://github.com/apache/pulsar/issues/13229 As discussed with @codelipenghui, https://github.com/apache/pulsar/pull/13077 changes `zookeeperServers` to `metadataStoreUrl`. Consequently, the flag `zookeeperServers` of `pulsar-perf

[GitHub] [pulsar] sl1316 commented on issue #13174: Pulsar admin api golang client

2021-12-09 Thread GitBox
sl1316 commented on issue #13174: URL: https://github.com/apache/pulsar/issues/13174#issuecomment-990610183 @tuteng this package only contains the producer/consumer apis but not the admin apis -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] merlimat commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
merlimat commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990608986 > Should we add this PR to 2.8.2 since there is a security vulnerability for log4j < 2.15? @315157973 If there’s a security issue, we should also backport to 2.7 --

[GitHub] [pulsar] Dulingxiao opened a new issue #13228: Only 0 bookies are live since 1629 seconds elapsed, wait for another 1 bookies for another 1 second

2021-12-09 Thread GitBox
Dulingxiao opened a new issue #13228: URL: https://github.com/apache/pulsar/issues/13228 log keep hitting 03:48:26.020 [LedgerDirsMonitorThread] WARN org.apache.bookkeeper.bookie.LedgerDirsMonitor - LedgerDirsMonitor check process: All ledger directories are non writable

[GitHub] [pulsar] sl1316 opened a new issue #13227: Deploy function worker in a separate kubernetes

2021-12-09 Thread GitBox
sl1316 opened a new issue #13227: URL: https://github.com/apache/pulsar/issues/13227 Please answer the following questions before submitting your doc issue. Thanks! # What issue do you find in Pulsar docs? we are looking into deploy the connector in a separate kubernetes cluster

[GitHub] [pulsar] Jason918 commented on a change in pull request #13225: Added Etcd MetadataStore implementation

2021-12-09 Thread GitBox
Jason918 commented on a change in pull request #13225: URL: https://github.com/apache/pulsar/pull/13225#discussion_r766316292 ## File path: pulsar-metadata/src/main/java/org/apache/pulsar/metadata/impl/EtcdMetadataStore.java ## @@ -0,0 +1,423 @@ +/** + * Licensed to the

[GitHub] [pulsar] congbobo184 commented on pull request #13209: [Transaction] Fix transaction sequenceId generate error.

2021-12-09 Thread GitBox
congbobo184 commented on pull request #13209: URL: https://github.com/apache/pulsar/pull/13209#issuecomment-990565584 > is this change compatible with existing Pulsar clusters ? It will be compatible -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] BewareMyPower commented on pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
BewareMyPower commented on pull request #13226: URL: https://github.com/apache/pulsar/pull/13226#issuecomment-990581339 Should we add this PR to 2.8.2 since there is a security vulnerability for log4j < 2.15? @315157973 -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] xuesongxs commented on issue #13224: Broker Java heap space OOM

2021-12-09 Thread GitBox
xuesongxs commented on issue #13224: URL: https://github.com/apache/pulsar/issues/13224#issuecomment-990580726 broker.conf ``` bookkeeper_waitTimeoutOnBackpressureMillis=5000 # # Licensed to the Apache Software Foundation (ASF) under one # or more contributor license

[GitHub] [pulsar] Jason918 commented on pull request #13208: fix shedding heartbeat ns

2021-12-09 Thread GitBox
Jason918 commented on pull request #13208: URL: https://github.com/apache/pulsar/pull/13208#issuecomment-990580326 And there is another special namespace "SLA_NAMESPACE", do we need to avoid this LoadShedding too? @codelipenghui -- This is an automated message from the Apache Git

[GitHub] [pulsar] Jason918 commented on a change in pull request #13208: fix shedding heartbeat ns

2021-12-09 Thread GitBox
Jason918 commented on a change in pull request #13208: URL: https://github.com/apache/pulsar/pull/13208#discussion_r766314855 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/OverloadShedder.java ## @@ -103,8 +104,10 @@ //

[GitHub] [pulsar] Shoothzj opened a new pull request #13226: Bump log4j to 2.15.0

2021-12-09 Thread GitBox
Shoothzj opened a new pull request #13226: URL: https://github.com/apache/pulsar/pull/13226 ### Modifications Bump log4j to 2.15.0 ### Documentation Check the box below and label this PR (if you have committer privilege). Need to update docs? - [x]

[GitHub] [pulsar] BewareMyPower commented on pull request #12769: [c++] Define and expose PULSAR_VERSION macro

2021-12-09 Thread GitBox
BewareMyPower commented on pull request #12769: URL: https://github.com/apache/pulsar/pull/12769#issuecomment-990579592 Add `release/2.8.2` label because this PR is depended by https://github.com/apache/pulsar/pull/13112, which is labeled as `release/2.8.2`. Ping @315157973 since

[GitHub] [pulsar] Technoboy- commented on pull request #13210: Close old compacted ledger when open new.

2021-12-09 Thread GitBox
Technoboy- commented on pull request #13210: URL: https://github.com/apache/pulsar/pull/13210#issuecomment-990573866 /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

[GitHub] [pulsar] congbobo184 removed a comment on pull request #13209: [Transaction] Fix transaction sequenceId generate error.

2021-12-09 Thread GitBox
congbobo184 removed a comment on pull request #13209: URL: https://github.com/apache/pulsar/pull/13209#issuecomment-989872114 > is this change compatible with existing Pulsar clusters ? It will not. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] github-actions[bot] removed a comment on pull request #13225: Added Etcd MetadataStore implementation

2021-12-09 Thread GitBox
github-actions[bot] removed a comment on pull request #13225: URL: https://github.com/apache/pulsar/pull/13225#issuecomment-990559285 @merlimat:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about

[GitHub] [pulsar] github-actions[bot] commented on pull request #13225: Added Etcd MetadataStore implementation

2021-12-09 Thread GitBox
github-actions[bot] commented on pull request #13225: URL: https://github.com/apache/pulsar/pull/13225#issuecomment-990559285 @merlimat:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about

[GitHub] [pulsar] merlimat opened a new pull request #13225: Added Etcd MetadataStore implementation

2021-12-09 Thread GitBox
merlimat opened a new pull request #13225: URL: https://github.com/apache/pulsar/pull/13225 ### Motivation Added a new MetadataStore implementation based on Etcd client. The new implementation has all the feature as the ZK based backend: * Batching of read/write requests

[GitHub] [pulsar] xuesongxs opened a new issue #13224: Broker Java heap space OOM

2021-12-09 Thread GitBox
xuesongxs opened a new issue #13224: URL: https://github.com/apache/pulsar/issues/13224 **Describe the bug** Pulsar v2.8.1 ``` 21:24:55.363 [BookKeeperClientWorker-OrderedExecutor-15-0] ERROR org.apache.bookkeeper.common.util.SafeRunnable - Unexpected throwable caught

[GitHub] [pulsar] murong00 commented on pull request #13218: [Issue 13217][pulsar-package-management] Fix the filesystem storage failure

2021-12-09 Thread GitBox
murong00 commented on pull request #13218: URL: https://github.com/apache/pulsar/pull/13218#issuecomment-990550652 > I think we need to add a test to cover this case Sure, I'm working on this. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] congbobo184 commented on a change in pull request #13094: [ Java Client ] Support Protobuf ``AutoSchema``

2021-12-09 Thread GitBox
congbobo184 commented on a change in pull request #13094: URL: https://github.com/apache/pulsar/pull/13094#discussion_r766290057 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/schema/generic/AbstractGenericProtobufSchema.java ## @@ -0,0 +1,61 @@ +/**

[GitHub] [pulsar] Jason918 commented on pull request #13196: [pulsar-broker] clean up active consumer on already closed connection

2021-12-09 Thread GitBox
Jason918 commented on pull request #13196: URL: https://github.com/apache/pulsar/pull/13196#issuecomment-990542217 > @rdhabalia is this a bug fix? (do not need to update docs?) Yes, a bug fix. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] Jason918 commented on a change in pull request #13196: [pulsar-broker] clean up active consumer on already closed connection

2021-12-09 Thread GitBox
Jason918 commented on a change in pull request #13196: URL: https://github.com/apache/pulsar/pull/13196#discussion_r766293227 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java ## @@ -804,6 +804,16 @@ protected void

[GitHub] [pulsar] Jason918 commented on a change in pull request #13195: [pulsar-broker] Support of TopicLifecyclePolicies REST handler.

2021-12-09 Thread GitBox
Jason918 commented on a change in pull request #13195: URL: https://github.com/apache/pulsar/pull/13195#discussion_r766290790 ## File path: pulsar-client-admin-api/src/main/java/org/apache/pulsar/common/policies/data/TopicLifecyclePolicies.java ## @@ -0,0 +1,44 @@ +/** + *

[GitHub] [pulsar] liangyepianzhou commented on a change in pull request #13135: [Transaction]No TransactionCoordinatorNotFound, but automatic reconnect

2021-12-09 Thread GitBox
liangyepianzhou commented on a change in pull request #13135: URL: https://github.com/apache/pulsar/pull/13135#discussion_r766276661 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/TransactionMetaStoreHandler.java ## @@ -206,8 +235,27 @@ void

[GitHub] [pulsar] wangjialing218 opened a new pull request #13223: Change some WebApplicationException log level to debug

2021-12-09 Thread GitBox
wangjialing218 opened a new pull request #13223: URL: https://github.com/apache/pulsar/pull/13223 ### Motivation #7725 change some WebApplicationException log level to debug, This PR add some parts of WebApplicationException miss to handle. ### Modifications Change

[GitHub] [pulsar] wuzhanpeng commented on pull request #13208: fix shedding heartbeat ns

2021-12-09 Thread GitBox
wuzhanpeng commented on pull request #13208: URL: https://github.com/apache/pulsar/pull/13208#issuecomment-990520916 /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

[GitHub] [pulsar] liangyepianzhou commented on a change in pull request #13135: [Transaction]No TransactionCoordinatorNotFound, but automatic reconnect

2021-12-09 Thread GitBox
liangyepianzhou commented on a change in pull request #13135: URL: https://github.com/apache/pulsar/pull/13135#discussion_r766276661 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/TransactionMetaStoreHandler.java ## @@ -206,8 +235,27 @@ void

[GitHub] [pulsar] wangjialing218 commented on pull request #13204: support limit message size produced by protocol handlers by maxMessag…

2021-12-09 Thread GitBox
wangjialing218 commented on pull request #13204: URL: https://github.com/apache/pulsar/pull/13204#issuecomment-990491001 We do not have to do `LongAdder.sum()` each time when add or decrease pending bytes, since it's a little heavy CPU work. We could do sum() in another scheduler task

[GitHub] [pulsar] tuteng commented on issue #13174: Pulsar admin api golang client

2021-12-09 Thread GitBox
tuteng commented on issue #13174: URL: https://github.com/apache/pulsar/issues/13174#issuecomment-990481634 > @codelipenghui it's a cli tool which operator runs manually. we would like to use the client like the consumer/producer client package which you can run your application on top of

[GitHub] [pulsar] wangjialing218 commented on pull request #13205: [PIP 86][Function] add funtion api to support preload and release external resources

2021-12-09 Thread GitBox
wangjialing218 commented on pull request #13205: URL: https://github.com/apache/pulsar/pull/13205#issuecomment-990480438 @nlu90 @eolivelli could you please have a check -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [pulsar] kukuzidian closed issue #11318: NonRecoverableLedgerException No such ledger exists on Metadata Server

2021-12-09 Thread GitBox
kukuzidian closed issue #11318: URL: https://github.com/apache/pulsar/issues/11318 -- 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:

[GitHub] [pulsar] zymap commented on a change in pull request #13218: [Issue 13217][pulsar-package-management] Fix the filesystem storage failure

2021-12-09 Thread GitBox
zymap commented on a change in pull request #13218: URL: https://github.com/apache/pulsar/pull/13218#discussion_r766269175 ## File path: pulsar-package-management/core/src/main/java/org/apache/pulsar/packages/management/core/impl/PackagesManagementImpl.java ## @@ -248,7

[GitHub] [pulsar] zymap commented on a change in pull request #13218: [Issue 13217][pulsar-package-management] Fix the filesystem storage failure

2021-12-09 Thread GitBox
zymap commented on a change in pull request #13218: URL: https://github.com/apache/pulsar/pull/13218#discussion_r766268846 ## File path: pulsar-package-management/core/src/main/java/org/apache/pulsar/packages/management/core/impl/PackagesManagementImpl.java ## @@ -248,7

[pulsar] branch master updated (47e44c8 -> 02b8de0)

2021-12-09 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 47e44c8 [DOCS] Update pip install command (#13221) add 02b8de0 [Broker] Optimize ManagedLedger Ledger

[GitHub] [pulsar] merlimat merged pull request #13222: [Broker] Optimize ManagedLedger Ledger Ownership Check

2021-12-09 Thread GitBox
merlimat merged pull request #13222: URL: https://github.com/apache/pulsar/pull/13222 -- 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:

[GitHub] [pulsar] yuruguo commented on a change in pull request #13192: [Broker] Add removeMaxConsumersPerSubscription method for v1 namespace

2021-12-09 Thread GitBox
yuruguo commented on a change in pull request #13192: URL: https://github.com/apache/pulsar/pull/13192#discussion_r764666813 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/Namespaces.java ## @@ -1094,7 +1094,7 @@ public void

[pulsar-client-node] branch master updated: Update dependencies (#186)

2021-12-09 Thread hrsakai
This is an automated email from the ASF dual-hosted git repository. hrsakai pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-node.git The following commit(s) were added to refs/heads/master by this push: new 03440da Update dependencies

[pulsar-client-node] branch master updated (661e965 -> 3d7714c)

2021-12-09 Thread hrsakai
This is an automated email from the ASF dual-hosted git repository. hrsakai pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-node.git. from 661e965 fix: use pause and resume api to ensure orderly init listener->consumer (#182) (#183)

[GitHub] [pulsar] michaeljmarshall commented on a change in pull request #11974: [Issue 11846][Java client] Create a consumer in the paused state

2021-12-09 Thread GitBox
michaeljmarshall commented on a change in pull request #11974: URL: https://github.com/apache/pulsar/pull/11974#discussion_r766226285 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/ConsumerImplTest.java ## @@ -200,4 +201,21 @@ public void testClose()

[GitHub] [pulsar] zwalsh-toast commented on a change in pull request #11974: [Issue 11846][Java client] Create a consumer in the paused state

2021-12-09 Thread GitBox
zwalsh-toast commented on a change in pull request #11974: URL: https://github.com/apache/pulsar/pull/11974#discussion_r766222152 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/ConsumerImplTest.java ## @@ -200,4 +201,21 @@ public void testClose() {

[GitHub] [pulsar] merlimat commented on a change in pull request #12838: Initial commit for TableView

2021-12-09 Thread GitBox
merlimat commented on a change in pull request #12838: URL: https://github.com/apache/pulsar/pull/12838#discussion_r766194312 ## File path: pulsar-client-api/src/main/java/org/apache/pulsar/client/api/TableViewBuilder.java ## @@ -0,0 +1,43 @@ +/** + * Licensed to the Apache

[GitHub] [pulsar] michaeljmarshall opened a new pull request #13222: [Broker] Optimize ManagedLedger Ledger Ownership Check

2021-12-09 Thread GitBox
michaeljmarshall opened a new pull request #13222: URL: https://github.com/apache/pulsar/pull/13222 ### Motivation The logic added in https://github.com/apache/pulsar/pull/11912 forces all reads to check for containment in the managed ledger's collection of ledgers. Since the

[pulsar] branch master updated (50d2e30 -> 47e44c8)

2021-12-09 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 50d2e30 clean unused method isTopicOwned,we can use isTopicOwnedAsync (#13189) add 47e44c8 [DOCS] Update pip

[GitHub] [pulsar] merlimat merged pull request #13221: [DOCS] Update pip install command

2021-12-09 Thread GitBox
merlimat merged pull request #13221: URL: https://github.com/apache/pulsar/pull/13221 -- 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:

[GitHub] [pulsar] sijia-w opened a new pull request #13221: [DOCS] Update pip install command

2021-12-09 Thread GitBox
sijia-w opened a new pull request #13221: URL: https://github.com/apache/pulsar/pull/13221 ### Motivation Update the pip install commands. ### Documentation Check the box below and label this PR (if you have committer privilege). Need to update docs? - [

[pulsar] branch master updated (7948f1f -> 50d2e30)

2021-12-09 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 7948f1f [Broker] fix typo in v2 PersistentTopics annotation (#13186) add 50d2e30 clean unused method

[GitHub] [pulsar] sijia-w commented on a change in pull request #13118: [DOCS] Update the Package Management page

2021-12-09 Thread GitBox
sijia-w commented on a change in pull request #13118: URL: https://github.com/apache/pulsar/pull/13118#discussion_r766086439 ## File path: site2/docs/admin-api-packages.md ## @@ -4,77 +4,89 @@ title: Manage packages sidebar_label: Packages --- -> **Important** -> -> This

[pulsar] branch master updated (27d5429 -> 7948f1f)

2021-12-09 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 27d5429 [Java Client] Send CloseProducer on timeout (#13161) add 7948f1f [Broker] fix typo in v2

[pulsar] branch master updated (804e2de -> 27d5429)

2021-12-09 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 804e2de Add package management filesystem storage into the distribution lib (#13202) add 27d5429 [Java

[GitHub] [pulsar] merlimat merged pull request #13189: clean unused method isTopicOwned,we can use isTopicOwnedAsync

2021-12-09 Thread GitBox
merlimat merged pull request #13189: URL: https://github.com/apache/pulsar/pull/13189 -- 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:

[GitHub] [pulsar] sijia-w commented on a change in pull request #13118: [DOCS] Update the Package Management page

2021-12-09 Thread GitBox
sijia-w commented on a change in pull request #13118: URL: https://github.com/apache/pulsar/pull/13118#discussion_r766083167 ## File path: site2/docs/admin-api-packages.md ## @@ -109,142 +124,168 @@ bin/pulsar-admin packages download functions://public/default/example@v0.1

[GitHub] [pulsar] merlimat merged pull request #13186: [Broker] fix typo in v2 PersistentTopics annotation

2021-12-09 Thread GitBox
merlimat merged pull request #13186: URL: https://github.com/apache/pulsar/pull/13186 -- 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:

[GitHub] [pulsar] merlimat merged pull request #13161: [Java Client] Send CloseProducer on timeout

2021-12-09 Thread GitBox
merlimat merged pull request #13161: URL: https://github.com/apache/pulsar/pull/13161 -- 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:

[pulsar] branch master updated (440b0d9 -> 804e2de)

2021-12-09 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 440b0d9 fix:error num for 1MB in metrics (#13213) add 804e2de Add package management filesystem storage into

[GitHub] [pulsar] merlimat merged pull request #13202: Add package management filesystem storage into the distribution lib

2021-12-09 Thread GitBox
merlimat merged pull request #13202: URL: https://github.com/apache/pulsar/pull/13202 -- 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:

[pulsar] branch master updated: fix:error num for 1MB in metrics (#13213)

2021-12-09 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 440b0d9 fix:error num for 1MB in metrics

[GitHub] [pulsar] merlimat closed issue #13212: ENTRY_SIZE_BUCKETS_BYTES is wrong for 1MB

2021-12-09 Thread GitBox
merlimat closed issue #13212: URL: https://github.com/apache/pulsar/issues/13212 -- 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:

[GitHub] [pulsar] merlimat merged pull request #13213: fix:error num for 1MB in metrics

2021-12-09 Thread GitBox
merlimat merged pull request #13213: URL: https://github.com/apache/pulsar/pull/13213 -- 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:

[pulsar] branch master updated (965a80f -> af7bd95)

2021-12-09 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 965a80f Fix namespace policy override ignored when creating subscription (#12699) add af7bd95 fix(docs):

[GitHub] [pulsar] merlimat merged pull request #13216: fix(docs): update zk metrics info in docs

2021-12-09 Thread GitBox
merlimat merged pull request #13216: URL: https://github.com/apache/pulsar/pull/13216 -- 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:

[pulsar] branch master updated (fdfea8a -> 965a80f)

2021-12-09 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 fdfea8a Fix in macOS cmake might find error boost-python libs path (#13193) add 965a80f Fix namespace policy

[GitHub] [pulsar] merlimat merged pull request #12699: Fix namespace policy override ignored when creating subscription

2021-12-09 Thread GitBox
merlimat merged pull request #12699: URL: https://github.com/apache/pulsar/pull/12699 -- 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:

[pulsar] branch master updated (65046ca -> fdfea8a)

2021-12-09 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 65046ca [admin-client] 'rebalance' command for functions worker (#13169) add fdfea8a Fix in macOS cmake

  1   2   >