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

2021-12-21 Thread GitBox
gaozhangmin commented on a change in pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#discussion_r773670900 ## File path: site2/docs/reference-configuration.md ## @@ -356,6 +356,7 @@ brokerServiceCompactionThresholdInBytes|If the estimated backlog size is

[GitHub] [pulsar] 315157973 commented on a change in pull request #12025: violation of isolation when using bookie affinity group

2021-12-21 Thread GitBox
315157973 commented on a change in pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#discussion_r773670299 ## File path: site2/docs/reference-configuration.md ## @@ -356,6 +356,7 @@ brokerServiceCompactionThresholdInBytes|If the estimated backlog size is

[GitHub] [pulsar] BewareMyPower commented on issue #13342: [BUG] Producer with name xxx is already connect to topic

2021-12-21 Thread GitBox
BewareMyPower commented on issue #13342: URL: https://github.com/apache/pulsar/issues/13342#issuecomment-999359570 Could you confirm whether https://github.com/apache/pulsar/pull/13428 fixed the problem? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] Shoothzj removed a comment on pull request #13354: [security] Allow to config web server's cipher and protocols

2021-12-21 Thread GitBox
Shoothzj removed a comment on pull request #13354: URL: https://github.com/apache/pulsar/pull/13354#issuecomment-997170680 /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

[GitHub] [pulsar] Shoothzj commented on pull request #13354: [security] Allow to config web server's cipher and protocols

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

[GitHub] [pulsar] Shoothzj removed a comment on pull request #13354: [security] Allow to config web server's cipher and protocols

2021-12-21 Thread GitBox
Shoothzj removed a comment on pull request #13354: URL: https://github.com/apache/pulsar/pull/13354#issuecomment-999218249 /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

[GitHub] [pulsar] Shoothzj removed a comment on pull request #13413: Enable CheckStyle Plugin in Pulsar Io

2021-12-21 Thread GitBox
Shoothzj removed a comment on pull request #13413: URL: https://github.com/apache/pulsar/pull/13413#issuecomment-998018995 @codelipenghui @lhotari @michaeljmarshall @merlimat @315157973 @hangc0276 PTAL, thanks -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] Shoothzj commented on pull request #13440: Fix typo in functions-work.yaml

2021-12-21 Thread GitBox
Shoothzj commented on pull request #13440: URL: https://github.com/apache/pulsar/pull/13440#issuecomment-999353396 @codelipenghui @lhotari @michaeljmarshall @merlimat @315157973 @hangc0276 @eolivelli PTAL, thanks -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] nicoloboschi opened a new pull request #13449: Github Workflow: websuite build - increase job timeout

2021-12-21 Thread GitBox
nicoloboschi opened a new pull request #13449: URL: https://github.com/apache/pulsar/pull/13449 ### Motivation The job times out after 2h https://github.com/apache/pulsar/runs/4601374159?check_suite_focus=true#logs ### Modifications Raised the timeout to 3h

[GitHub] [pulsar] eolivelli commented on pull request #13376: Make Docker images non-root, by default, and OpenShift compliant

2021-12-21 Thread GitBox
eolivelli commented on pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#issuecomment-999346356 after committing this patch we should take into consideration this https://github.com/apache/pulsar/pull/10815 about adding tools to manage/inspect/debug the system --

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

2021-12-21 Thread GitBox
gaozhangmin commented on pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#issuecomment-999345194 /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-client-go] wolfstudy closed issue #565: Send gorutine blocked

2021-12-21 Thread GitBox
wolfstudy closed issue #565: URL: https://github.com/apache/pulsar-client-go/issues/565 -- 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] nodece removed a comment on pull request #13420: [Broker] Fix create the dynamic configuration resource if not exist

2021-12-21 Thread GitBox
nodece removed a comment on pull request #13420: URL: https://github.com/apache/pulsar/pull/13420#issuecomment-999223429 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] nodece removed a comment on pull request #13420: [Broker] Fix create the dynamic configuration resource if not exist

2021-12-21 Thread GitBox
nodece removed a comment on pull request #13420: URL: https://github.com/apache/pulsar/pull/13420#issuecomment-999251127 -- 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

[GitHub] [pulsar] nodece commented on pull request #13420: [Broker] Fix create the dynamic configuration resource if not exist

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

[GitHub] [pulsar] dave2wave commented on a change in pull request #13446: Improve clarity of the Log4Shell CVE blog post

2021-12-21 Thread GitBox
dave2wave commented on a change in pull request #13446: URL: https://github.com/apache/pulsar/pull/13446#discussion_r773638618 ## File path: site2/website/blog/2021-12-11-Log4j-CVE.md ## @@ -8,9 +8,10 @@ allow remote execution for attackers. The vulnerability issue is

[GitHub] [pulsar] nodece commented on pull request #13420: [Broker] Fix create the dynamic configuration resource if not exist

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

[GitHub] [pulsar] Anonymitaet commented on pull request #13448: [website] fix: pulsar-admin cli docs of transactions

2021-12-21 Thread GitBox
Anonymitaet commented on pull request #13448: URL: https://github.com/apache/pulsar/pull/13448#issuecomment-999326861 @tuteng can you help review this PR? Thanks -- 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] rdhabalia commented on pull request #13073: [pulsar-broker] Support roll-over ledgers for inactive topics

2021-12-21 Thread GitBox
rdhabalia commented on pull request #13073: URL: https://github.com/apache/pulsar/pull/13073#issuecomment-999325144 @michaeljmarshall 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

[GitHub] [pulsar] urfreespace opened a new pull request #13448: fix: pulsar-admin cli docs of transactions

2021-12-21 Thread GitBox
urfreespace opened a new pull request #13448: URL: https://github.com/apache/pulsar/pull/13448 Signed-off-by: LiLi *(If this PR fixes a github issue, please add `Fixes #`.)* Fix: pulsar-admin cli docs of transactions is missing Master Issue: # ###

[GitHub] [pulsar] Shoothzj closed pull request #13440: Fix typo in functions-work.yaml

2021-12-21 Thread GitBox
Shoothzj closed pull request #13440: URL: https://github.com/apache/pulsar/pull/13440 -- 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] Shoothzj removed a comment on pull request #13440: Fix typo in functions-work.yaml

2021-12-21 Thread GitBox
Shoothzj removed a comment on pull request #13440: URL: https://github.com/apache/pulsar/pull/13440#issuecomment-999218118 /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

[GitHub] [pulsar] liangyepianzhou opened a new pull request #13447: [Transaction] Add transaction admin

2021-12-21 Thread GitBox
liangyepianzhou opened a new pull request #13447: URL: https://github.com/apache/pulsar/pull/13447 ## Motivation Add the CmdTransactions to PulsarAdminTool. ## Modification Add the CmdTransactions to PulsarAdminTool. ### Verifying this change - [ ] Make sure that the

[GitHub] [pulsar-client-go] wolfstudy merged pull request #678: [Issue 513] Correct apparent logic error in batchContainer's hasSpace() method

2021-12-21 Thread GitBox
wolfstudy merged pull request #678: URL: https://github.com/apache/pulsar-client-go/pull/678 -- 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: [Issue 513] Correct apparent logic error in batchContainer's hasSpace() method (#678)

2021-12-21 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 2bbfb8e [Issue 513] Correct apparent

[GitHub] [pulsar] mattisonchao removed a comment on issue #13444: Topic policy across multiple clusters

2021-12-21 Thread GitBox
mattisonchao removed a comment on issue #13444: URL: https://github.com/apache/pulsar/issues/13444#issuecomment-999311917 Can i try to work on it ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [pulsar] mattisonchao commented on issue #13444: Topic policy across multiple clusters

2021-12-21 Thread GitBox
mattisonchao commented on issue #13444: URL: https://github.com/apache/pulsar/issues/13444#issuecomment-999312257 Can I try to help with this? On Wed, 22 Dec 2021 at 11:58, feynmanlin ***@***.***> wrote: > *Is your feature request related to a problem? Please describe.* >

[GitHub] [pulsar] mattisonchao commented on issue #13444: Topic policy across multiple clusters

2021-12-21 Thread GitBox
mattisonchao commented on issue #13444: URL: https://github.com/apache/pulsar/issues/13444#issuecomment-999311917 Can i try to work on it ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar] lhotari opened a new pull request #13446: Improve clarity of the Log4Shell CVE blog post

2021-12-21 Thread GitBox
lhotari opened a new pull request #13446: URL: https://github.com/apache/pulsar/pull/13446 ### Motivation - There was feedback from Senior Security Engineer [Gabriel Marquet](https://twitter.com/gbysec) that the message about Pulsar being impacted isn't clearly communicated [in the

[pulsar] branch master updated (b98f6f1 -> 3316db5)

2021-12-21 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from b98f6f1 Fix License check failure (#13437) add 3316db5 [Broker] Fix race conditions in closing producers and

[GitHub] [pulsar] lhotari merged pull request #13428: [Broker] Fix race conditions in closing producers and consumers

2021-12-21 Thread GitBox
lhotari merged pull request #13428: URL: https://github.com/apache/pulsar/pull/13428 -- 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] timmyyuan commented on issue #13152: [Security] Upgrade vulnerable packages in the pulsar-all image

2021-12-21 Thread GitBox
timmyyuan commented on issue #13152: URL: https://github.com/apache/pulsar/issues/13152#issuecomment-999273739 @TIBCOeddie FYI: https://github.com/apache/pulsar/pull/13392 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] yuruguo removed a comment on pull request #12242: [pulsar-admin] Add --update-mode to optimize topic rate policy

2021-12-21 Thread GitBox
yuruguo removed a comment on pull request #12242: URL: https://github.com/apache/pulsar/pull/12242#issuecomment-999266816 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] 315157973 opened a new pull request #13445: Backlog quota support cross multiple clusters

2021-12-21 Thread GitBox
315157973 opened a new pull request #13445: URL: https://github.com/apache/pulsar/pull/13445 ### Motivation Master Issue: #13444 ### Modifications Backlog quota support cross multiple clusters ### Documentation - [x] `no-need-doc` -- This is

[GitHub] [pulsar] 315157973 opened a new issue #13444: Topic policy across multiple clusters

2021-12-21 Thread GitBox
315157973 opened a new issue #13444: URL: https://github.com/apache/pulsar/issues/13444 **Is your feature request related to a problem? Please describe.** In PIP 92, we have supported remote replication of Topic Policies However, many existing APIs do not support IsGlobal

[GitHub] [pulsar] yuruguo commented on pull request #12242: [pulsar-admin] Add --update-mode to optimize topic rate policy

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

[GitHub] [pulsar] yuruguo removed a comment on pull request #12242: [pulsar-admin] Add --update-mode to optimize topic rate policy

2021-12-21 Thread GitBox
yuruguo removed a comment on pull request #12242: URL: https://github.com/apache/pulsar/pull/12242#issuecomment-999246908 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] michaeljmarshall commented on a change in pull request #13376: Make Docker images non-root, by default, and OpenShift compliant

2021-12-21 Thread GitBox
michaeljmarshall commented on a change in pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#discussion_r773581354 ## File path: docker/pulsar/Dockerfile ## @@ -69,4 +83,8 @@ ENV PULSAR_ROOT_LOGGER=INFO,CONSOLE COPY --from=pulsar /pulsar /pulsar WORKDIR

[GitHub] [pulsar] michaeljmarshall commented on a change in pull request #13376: Make Docker images non-root, by default, and OpenShift compliant

2021-12-21 Thread GitBox
michaeljmarshall commented on a change in pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#discussion_r773580954 ## File path: docker/pulsar/Dockerfile ## @@ -69,4 +83,8 @@ ENV PULSAR_ROOT_LOGGER=INFO,CONSOLE COPY --from=pulsar /pulsar /pulsar WORKDIR

[GitHub] [pulsar] michaeljmarshall commented on a change in pull request #13376: Make Docker images non-root, by default, and OpenShift compliant

2021-12-21 Thread GitBox
michaeljmarshall commented on a change in pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#discussion_r773579601 ## File path: tests/docker-images/latest-version-image/Dockerfile ## @@ -55,6 +56,16 @@ FROM apachepulsar/pulsar-all:latest as pulsar-all

[pulsar] branch master updated (055b258 -> b98f6f1)

2021-12-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 055b258 Pulsar 2.8.2 release notes (#13400) add b98f6f1 Fix License check failure (#13437) No new revisions

[GitHub] [pulsar] merlimat merged pull request #13437: Fix License check failure

2021-12-21 Thread GitBox
merlimat merged pull request #13437: URL: https://github.com/apache/pulsar/pull/13437 -- 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 edited a comment on pull request #13437: Fix License check failure

2021-12-21 Thread GitBox
yuruguo edited a comment on pull request #13437: URL: https://github.com/apache/pulsar/pull/13437#issuecomment-999254880 > Seems #13436 has done it. Thanks for your reply, I see. But I found other problem(from this [PR](https://github.com/apache/pulsar/pull/13401)) with `License

[GitHub] [pulsar] yuruguo commented on pull request #13437: Fix License check failure

2021-12-21 Thread GitBox
yuruguo commented on pull request #13437: URL: https://github.com/apache/pulsar/pull/13437#issuecomment-999254880 > Seems #13436 has done it. thanks for your reply, I see. But I found a problem(from this [PR](https://github.com/apache/pulsar/pull/13401)) with License check and it

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

2021-12-21 Thread GitBox
gaozhangmin commented on pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#issuecomment-999252118 /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] nodece commented on pull request #13420: [Broker] Fix create the dynamic configuration resource if not exist

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

[GitHub] [pulsar] urfreespace opened a new pull request #13443: [website][upgrade]feat: website upgrade / docs migration - 2.8.2

2021-12-21 Thread GitBox
urfreespace opened a new pull request #13443: URL: https://github.com/apache/pulsar/pull/13443 *(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] yuruguo commented on pull request #12242: [pulsar-admin] Add --update-mode to optimize topic rate policy

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

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

2021-12-21 Thread GitBox
gaozhangmin commented on a change in pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#discussion_r773562466 ## File path: site2/docs/reference-configuration.md ## @@ -356,6 +356,7 @@ brokerServiceCompactionThresholdInBytes|If the estimated backlog size is

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

2021-12-21 Thread GitBox
gaozhangmin commented on a change in pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#discussion_r773562466 ## File path: site2/docs/reference-configuration.md ## @@ -356,6 +356,7 @@ brokerServiceCompactionThresholdInBytes|If the estimated backlog size is

[GitHub] [pulsar] Jason918 commented on a change in pull request #13297: Add schema compatibility strategy on topic level

2021-12-21 Thread GitBox
Jason918 commented on a change in pull request #13297: URL: https://github.com/apache/pulsar/pull/13297#discussion_r773555946 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -4668,4 +4669,38 @@ private void

[pulsar] branch master updated (305863e -> 055b258)

2021-12-21 Thread linlin
This is an automated email from the ASF dual-hosted git repository. linlin pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 305863e [Doc] fix invalid wiki URL (#13430) add 055b258 Pulsar 2.8.2 release notes (#13400) No new revisions

[GitHub] [pulsar] 315157973 merged pull request #13400: Pulsar 2.8.2 release notes

2021-12-21 Thread GitBox
315157973 merged pull request #13400: URL: https://github.com/apache/pulsar/pull/13400 -- 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] urfreespace opened a new pull request #13442: [website][upgrade]feat: website upgrade / docs migration - 2.8.1

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

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

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

[GitHub] [pulsar] ericsyh commented on issue #13441: feat(offloader): support kerberos config set for hdfs offloader

2021-12-21 Thread GitBox
ericsyh commented on issue #13441: URL: https://github.com/apache/pulsar/issues/13441#issuecomment-999224759 @zymap 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.

[GitHub] [pulsar] ericsyh opened a new issue #13441: feat(offloader): support kerberos config set for hdfs offloader

2021-12-21 Thread GitBox
ericsyh opened a new issue #13441: URL: https://github.com/apache/pulsar/issues/13441 **Is your feature request related to a problem? Please describe.** Usually hdfs is kerberos is enabled in production environment. **Describe the solution you'd like** Support kerberos

[GitHub] [pulsar] nodece commented on pull request #13420: [Broker] Fix create the dynamic configuration resource if not exist

2021-12-21 Thread GitBox
nodece commented on pull request #13420: URL: https://github.com/apache/pulsar/pull/13420#issuecomment-999223429 /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

[pulsar] branch master updated (510fae3 -> 305863e)

2021-12-21 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 510fae3 [broker] Fix compatibility issue with other metadata store implementations in resources (#13394) add

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #13433: fix(docs): broker metrics url

2021-12-21 Thread GitBox
Anonymitaet commented on a change in pull request #13433: URL: https://github.com/apache/pulsar/pull/13433#discussion_r773549093 ## File path: site2/docs/deploy-monitoring.md ## @@ -31,7 +31,7 @@ All the message rates are updated every minute. The aggregated broker metrics

[GitHub] [pulsar] Anonymitaet merged pull request #13430: [Doc] fix invalid wiki URL

2021-12-21 Thread GitBox
Anonymitaet merged pull request #13430: URL: https://github.com/apache/pulsar/pull/13430 -- 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] Anonymitaet closed issue #13414: [Doc] Pulsar Client Libraries Page has a Dead Link

2021-12-21 Thread GitBox
Anonymitaet closed issue #13414: URL: https://github.com/apache/pulsar/issues/13414 -- 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] Shoothzj commented on pull request #13390: [security] Allow config client dns bind addr and port

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

[GitHub] [pulsar] Shoothzj commented on pull request #13354: [security] Allow to config web server's cipher and protocols

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

[GitHub] [pulsar] Shoothzj commented on pull request #13440: Fix typo in functions-work.yaml

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

[GitHub] [pulsar] freeznet commented on a change in pull request #13376: Make Docker images non-root, by default, and OpenShift compliant

2021-12-21 Thread GitBox
freeznet commented on a change in pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#discussion_r773544439 ## File path: tests/docker-images/latest-version-image/Dockerfile ## @@ -55,6 +56,16 @@ FROM apachepulsar/pulsar-all:latest as pulsar-all

[GitHub] [pulsar] Technoboy- commented on pull request #13337: Fix delete namespace with 'Cannot delete non empty bundle' issue.

2021-12-21 Thread GitBox
Technoboy- commented on pull request #13337: URL: https://github.com/apache/pulsar/pull/13337#issuecomment-999214840 /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] Technoboy- commented on pull request #13437: [pulsar-admin] Reduce code duplication in TopicsImpl

2021-12-21 Thread GitBox
Technoboy- commented on pull request #13437: URL: https://github.com/apache/pulsar/pull/13437#issuecomment-999213318 Seems #13436 has done it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [pulsar] rdhabalia commented on a change in pull request #12025: violation of isolation when using bookie affinity group

2021-12-21 Thread GitBox
rdhabalia commented on a change in pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#discussion_r773533656 ## File path: site2/docs/reference-configuration.md ## @@ -356,6 +356,7 @@ brokerServiceCompactionThresholdInBytes|If the estimated backlog size is

[GitHub] [pulsar] Shoothzj opened a new pull request #13440: Fix typo in functions-work.yaml

2021-12-21 Thread GitBox
Shoothzj opened a new pull request #13440: URL: https://github.com/apache/pulsar/pull/13440 ### Modifications Fix typo in functions-work.yaml ### Documentation Check the box below and label this PR (if you have committer privilege). Need to update docs?

[GitHub] [pulsar] Shoothzj closed pull request #13345: Fix typo in functions-work.yaml, add functionWebServiceUrl in yaml

2021-12-21 Thread GitBox
Shoothzj closed pull request #13345: URL: https://github.com/apache/pulsar/pull/13345 -- 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] Shoothzj commented on pull request #13345: Fix typo in functions-work.yaml, add functionWebServiceUrl in yaml

2021-12-21 Thread GitBox
Shoothzj commented on pull request #13345: URL: https://github.com/apache/pulsar/pull/13345#issuecomment-999183636 > @lhotari Ok, I will start a thread on mail list, for now I will close this PR :) -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] Shoothzj commented on pull request #13390: [security] Allow config client dns bind addr and port

2021-12-21 Thread GitBox
Shoothzj commented on pull request #13390: URL: https://github.com/apache/pulsar/pull/13390#issuecomment-999182960 @michaeljmarshall Currently, I test it locally, and have some configuration read, write test. It's hard to write unit test for this pr, need a mock dns server and ensure the

[GitHub] [pulsar] Shoothzj commented on a change in pull request #13390: [security] Allow config client dns bind addr and port

2021-12-21 Thread GitBox
Shoothzj commented on a change in pull request #13390: URL: https://github.com/apache/pulsar/pull/13390#discussion_r773520002 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java ## @@ -295,6 +295,19 @@ @JsonIgnore

[GitHub] [pulsar] Shoothzj commented on a change in pull request #13390: [security] Allow config client dns bind addr and port

2021-12-21 Thread GitBox
Shoothzj commented on a change in pull request #13390: URL: https://github.com/apache/pulsar/pull/13390#discussion_r773519774 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java ## @@ -295,6 +295,19 @@ @JsonIgnore

[GitHub] [pulsar] Shoothzj commented on a change in pull request #13390: [security] Allow config client dns bind addr and port

2021-12-21 Thread GitBox
Shoothzj commented on a change in pull request #13390: URL: https://github.com/apache/pulsar/pull/13390#discussion_r773519463 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java ## @@ -295,6 +295,19 @@ @JsonIgnore

[GitHub] [pulsar] merlimat commented on pull request #13424: Allow to config no-password truststore

2021-12-21 Thread GitBox
merlimat commented on pull request #13424: URL: https://github.com/apache/pulsar/pull/13424#issuecomment-999171615 > I don't believe we're allowed to add binary files to our repo. @dave2wave and @eolivelli - can you confirm this for me? We do have binaries when it make sense. eg:

[GitHub] [pulsar] dave2wave commented on pull request #13424: Allow to config no-password truststore

2021-12-21 Thread GitBox
dave2wave commented on pull request #13424: URL: https://github.com/apache/pulsar/pull/13424#issuecomment-999170323 > I don't believe we're allowed to add binary files to our repo. Assuming it is not possible to create this *test* file from source then to me this test case is

[GitHub] [pulsar] Shoothzj commented on a change in pull request #13424: Allow to config no-password truststore

2021-12-21 Thread GitBox
Shoothzj commented on a change in pull request #13424: URL: https://github.com/apache/pulsar/pull/13424#discussion_r773500013 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/keystoretls/KeyStoreSSLContext.java ## @@ -105,7 +105,11 @@ public

[GitHub] [pulsar] rdhabalia commented on pull request #12258: [pulsar-broker] Support caching to drain backlog consumers

2021-12-21 Thread GitBox
rdhabalia commented on pull request #12258: URL: https://github.com/apache/pulsar/pull/12258#issuecomment-999140159 ping. can we merge this PR as it keep needs rebasing the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] rdhabalia commented on pull request #12235: [pulsar-broker] PIP-100 Support pluggable topic factory

2021-12-21 Thread GitBox
rdhabalia commented on pull request #12235: URL: https://github.com/apache/pulsar/pull/12235#issuecomment-999140570 @eolivelli addressed the comment. can you PTAL and unblock the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

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

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

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

2021-12-21 Thread GitBox
rdhabalia commented on pull request #13196: URL: https://github.com/apache/pulsar/pull/13196#issuecomment-999139688 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.

[GitHub] [pulsar] rdhabalia commented on pull request #13074: [pulsar-broker] Make load-balancer config dynamic for the runtime tuning

2021-12-21 Thread GitBox
rdhabalia commented on pull request #13074: URL: https://github.com/apache/pulsar/pull/13074#issuecomment-999139229 I think we kept most of them dynamic which are required. However, can we merge this PR as it's been open for long and time require to tune load balancer. -- This is an

[pulsar] branch master updated (522249f -> 510fae3)

2021-12-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 522249f [broker-test] make test independent of serviceConf change (#13416) add 510fae3 [broker] Fix

[pulsar] branch master updated (43ca11b -> 522249f)

2021-12-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 43ca11b [Doc] Add docs on how to install the Pulsar Wireshark dissector (#13401) add 522249f [broker-test]

[GitHub] [pulsar] merlimat merged pull request #13394: [broker] Fix compatibility issue with other metadata store implementations in resources

2021-12-21 Thread GitBox
merlimat merged pull request #13394: URL: https://github.com/apache/pulsar/pull/13394 -- 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 #13416: [broker-test] make test independent of serviceConf change

2021-12-21 Thread GitBox
merlimat merged pull request #13416: URL: https://github.com/apache/pulsar/pull/13416 -- 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 removed a comment on pull request #13437: [pulsar-admin] Reduce code duplication in TopicsImpl

2021-12-21 Thread GitBox
yuruguo removed a comment on pull request #13437: URL: https://github.com/apache/pulsar/pull/13437#issuecomment-999110575 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] yuruguo removed a comment on pull request #13437: [pulsar-admin] Reduce code duplication in TopicsImpl

2021-12-21 Thread GitBox
yuruguo removed a comment on pull request #13437: URL: https://github.com/apache/pulsar/pull/13437#issuecomment-998919730 @lhotari @Technoboy- A better improvement and solution, ptal -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] yuruguo commented on pull request #13437: [pulsar-admin] Reduce code duplication in TopicsImpl

2021-12-21 Thread GitBox
yuruguo commented on pull request #13437: URL: https://github.com/apache/pulsar/pull/13437#issuecomment-999110575 /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

[pulsar] branch master updated (601e141 -> 43ca11b)

2021-12-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 601e141 Fix invalid setting of enabled ciphers to fix warning from BoringSSL (#13435) add 43ca11b [Doc] Add

[GitHub] [pulsar] merlimat commented on issue #13432: [pulsar-cpp-client] Consumer should be able to read messages in batch

2021-12-21 Thread GitBox
merlimat commented on issue #13432: URL: https://github.com/apache/pulsar/issues/13432#issuecomment-999103952 @lalit-g-deepr The consumer is already internally receiving the messages in batches and doing prefetching. There is no performance difference in calling `receive()` or

[pulsar] branch master updated (9f46c4a -> 601e141)

2021-12-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 9f46c4a [Java Client] Improve consumer listener logic (#13273) add 601e141 Fix invalid setting of enabled

[GitHub] [pulsar] merlimat merged pull request #13401: [Doc] Add docs on how to install the Pulsar Wireshark dissector

2021-12-21 Thread GitBox
merlimat merged pull request #13401: URL: https://github.com/apache/pulsar/pull/13401 -- 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 #13435: Fix invalid setting of enabled ciphers to fix warning from BoringSSL

2021-12-21 Thread GitBox
merlimat merged pull request #13435: URL: https://github.com/apache/pulsar/pull/13435 -- 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 (2690a2b -> 9f46c4a)

2021-12-21 Thread mmarshall
This is an automated email from the ASF dual-hosted git repository. mmarshall pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 2690a2b Optimize TopicsImpl fix made in #13434 (#13436) add 9f46c4a [Java Client] Improve consumer

[GitHub] [pulsar] michaeljmarshall merged pull request #13273: [Java Client] Improve consumer listener logic

2021-12-21 Thread GitBox
michaeljmarshall merged pull request #13273: URL: https://github.com/apache/pulsar/pull/13273 -- 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] eolivelli commented on a change in pull request #13438: [Issue 13220][blog] Publishing a community blog for Pulsar's 10k+ GitHub Stars milestone

2021-12-21 Thread GitBox
eolivelli commented on a change in pull request #13438: URL: https://github.com/apache/pulsar/pull/13438#discussion_r773442561 ## File path: site2/website/blog/2021-12-21-apache-pulsar-hits-1-github-stars.md ## @@ -0,0 +1,58 @@ +--- +author: Alice Bi +title: Apache Pulsar

[GitHub] [pulsar] michaeljmarshall commented on pull request #13376: Make Docker images non-root, by default, and OpenShift compliant

2021-12-21 Thread GitBox
michaeljmarshall commented on pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#issuecomment-999078443 New docker image built from the most recent commit: `michaelmarshall/pulsar:2.10.0-SNAPSHOT-1dec8b9`. Note that I ran the dive tool

  1   2   >