[GitHub] [pulsar] ethqunzhong commented on a change in pull request #14200: Broker: support dynamic update log level at runtime

2022-02-15 Thread GitBox
ethqunzhong commented on a change in pull request #14200: URL: https://github.com/apache/pulsar/pull/14200#discussion_r807635627 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/BrokersBase.java ## @@ -437,5 +440,79 @@ private void

[pulsar] branch branch-2.7 updated: [Branch 2.7] Fix TopicPoliciesCacheNotInitException issue. (#14293)

2022-02-15 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 0abf26b [Branch 2.7] Fix

[GitHub] [pulsar] codelipenghui merged pull request #14293: [Branch 2.7] Fix TopicPoliciesCacheNotInitException issue.

2022-02-15 Thread GitBox
codelipenghui merged pull request #14293: URL: https://github.com/apache/pulsar/pull/14293 -- 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 (422efbb -> 244060a)

2022-02-15 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 422efbb [issue-14304] Check for blank advertised listener name (#14306) add 244060a [Broker] Move schema

[GitHub] [pulsar] codelipenghui merged pull request #14225: [Broker] Move schema compatibility strategy cmd from topics to topicPolicies

2022-02-15 Thread GitBox
codelipenghui merged pull request #14225: URL: https://github.com/apache/pulsar/pull/14225 -- 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 (9b003f9 -> 422efbb)

2022-02-15 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 9b003f9 Fix the wrong parameter in the log. (#14309) add 422efbb [issue-14304] Check for blank advertised

[GitHub] [pulsar] BewareMyPower merged pull request #14306: [issue-14304] Check for blank advertised listener name

2022-02-15 Thread GitBox
BewareMyPower merged pull request #14306: URL: https://github.com/apache/pulsar/pull/14306 -- 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] BewareMyPower closed issue #14304: CommandLookupTopic.advertisedListenerName may be blank

2022-02-15 Thread GitBox
BewareMyPower closed issue #14304: URL: https://github.com/apache/pulsar/issues/14304 -- 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] momo-jun commented on pull request #14313: [Doc] Add two parameters for transaction authentication

2022-02-15 Thread GitBox
momo-jun commented on pull request #14313: URL: https://github.com/apache/pulsar/pull/14313#issuecomment-1041174598 Hi @liangyepianzhou @Anonymitaet @codelipenghui, can you pls review this doc PR? Thank you. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] github-actions[bot] commented on pull request #14313: [Doc] Add two parameters for transaction authentication

2022-02-15 Thread GitBox
github-actions[bot] commented on pull request #14313: URL: https://github.com/apache/pulsar/pull/14313#issuecomment-1041174223 @momo-jun: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

[GitHub] [pulsar] github-actions[bot] commented on pull request #14313: [Doc] Add two parameters for transaction authentication

2022-02-15 Thread GitBox
github-actions[bot] commented on pull request #14313: URL: https://github.com/apache/pulsar/pull/14313#issuecomment-1041174140 @momo-jun:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about

[GitHub] [pulsar] momo-jun opened a new pull request #14313: [Doc] Add two parameters for transaction authentication

2022-02-15 Thread GitBox
momo-jun opened a new pull request #14313: URL: https://github.com/apache/pulsar/pull/14313 ### Modifications To align with #14271, add the following two parameters to the transaction section of pulsar-perf. - `--auth-params` - `--auth-plugin` ### Preview

[GitHub] [pulsar] lhotari commented on issue #14312: Pulsar release 2.8.2 documentation package is missing

2022-02-15 Thread GitBox
lhotari commented on issue #14312: URL: https://github.com/apache/pulsar/issues/14312#issuecomment-1041168749 @315157973 Could you please take a look? I believe that the step [Update the site for minor

[GitHub] [pulsar] lhotari opened a new issue #14312: Pulsar release 2.8.2 documentation package is missing

2022-02-15 Thread GitBox
lhotari opened a new issue #14312: URL: https://github.com/apache/pulsar/issues/14312 **Describe the bug** The 2.8.2 release isn't listed on this page: https://pulsar.apache.org/en/versions/ The documentation is missing for 2.8.2: https://pulsar.apache.org/docs/en/2.8.2/

[pulsar] branch master updated: Fix the wrong parameter in the log. (#14309)

2022-02-15 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj 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 9b003f9 Fix the wrong parameter in the log.

[GitHub] [pulsar] Shoothzj merged pull request #14309: Fix the wrong parameter in the log.

2022-02-15 Thread GitBox
Shoothzj merged pull request #14309: URL: https://github.com/apache/pulsar/pull/14309 -- 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 pull request #14301: [pulsar-admin] Add short name for full name in admin cli

2022-02-15 Thread GitBox
yuruguo commented on pull request #14301: URL: https://github.com/apache/pulsar/pull/14301#issuecomment-1041123818 > @yuruguo - I think it makes sense to add short options. I just wonder if we should stay away from `-v`. What is your perspective? Maybe using `-ve` is also possible

[GitHub] [pulsar] yuruguo commented on a change in pull request #14301: [pulsar-admin] Add short name for full name in admin cli

2022-02-15 Thread GitBox
yuruguo commented on a change in pull request #14301: URL: https://github.com/apache/pulsar/pull/14301#discussion_r807551121 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdBrokers.java ## @@ -64,9 +64,9 @@ void run() throws Exception {

[GitHub] [pulsar] congbobo184 closed issue #14166: [Transaction] Transaction timeout is not canceled in client.

2022-02-15 Thread GitBox
congbobo184 closed issue #14166: URL: https://github.com/apache/pulsar/issues/14166 -- 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] lordcheng10 closed pull request #12384: Variable compressedStr assignment error

2022-02-15 Thread GitBox
lordcheng10 closed pull request #12384: URL: https://github.com/apache/pulsar/pull/12384 -- 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] lordcheng10 closed pull request #13797: update describe about recentlyJoinedConsumers

2022-02-15 Thread GitBox
lordcheng10 closed pull request #13797: URL: https://github.com/apache/pulsar/pull/13797 -- 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] aloyszhang commented on pull request #13796: [Broker]support split bundle by specified topic

2022-02-15 Thread GitBox
aloyszhang commented on pull request #13796: URL: https://github.com/apache/pulsar/pull/13796#issuecomment-1041098947 /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 commented on pull request #13957: [Transaction] Set TC state is Ready after open MLTransactionMetadataStore completely.

2022-02-15 Thread GitBox
congbobo184 commented on pull request #13957: URL: https://github.com/apache/pulsar/pull/13957#issuecomment-1041089240 /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] merlimat commented on issue #14304: CommandLookupTopic.advertisedListenerName may be blank

2022-02-15 Thread GitBox
merlimat commented on issue #14304: URL: https://github.com/apache/pulsar/issues/14304#issuecomment-1041085252 It will return true if the field is set. That is irrespective of the specific value that was set. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] michaeljmarshall commented on issue #14304: CommandLookupTopic.advertisedListenerName may be blank

2022-02-15 Thread GitBox
michaeljmarshall commented on issue #14304: URL: https://github.com/apache/pulsar/issues/14304#issuecomment-1041080489 > I believe that `hasAdvertisedListenerName` would return true for any value, including `null`. I don't believe this is correct. If you look at the protobuf field

[pulsar] branch master updated (3d90165 -> a43fab0)

2022-02-15 Thread bogong
This is an automated email from the ASF dual-hosted git repository. bogong pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 3d90165 [Transaction] Optimize changeToCloseState (#14277) add a43fab0 [Transaction] Optimize

[GitHub] [pulsar] congbobo184 removed a comment on pull request #14276: [Transaction] delete changeMaxReadPositionAndAddAbortTimes when checkIfNoSnapshot

2022-02-15 Thread GitBox
congbobo184 removed a comment on pull request #14276: URL: https://github.com/apache/pulsar/pull/14276#issuecomment-1041074103 @codelipenghui This is to remove useless code, we don't need to add tests -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] congbobo184 merged pull request #14303: [Transaction] Optimize testEndTBRecoveringWhenManagerLedgerDisReadable

2022-02-15 Thread GitBox
congbobo184 merged pull request #14303: URL: https://github.com/apache/pulsar/pull/14303 -- 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] congbobo184 removed a comment on pull request #14303: [Transaction] Optimize testEndTBRecoveringWhenManagerLedgerDisReadable

2022-02-15 Thread GitBox
congbobo184 removed a comment on pull request #14303: URL: https://github.com/apache/pulsar/pull/14303#issuecomment-1041074585 ### Motivation The time used by clean up is too long. This is because the broker service fails to execute the `unloadserviceunit`, and can only rely on the

[GitHub] [pulsar] congbobo184 commented on pull request #14303: [Transaction] Optimize testEndTBRecoveringWhenManagerLedgerDisReadable

2022-02-15 Thread GitBox
congbobo184 commented on pull request #14303: URL: https://github.com/apache/pulsar/pull/14303#issuecomment-1041074585 ### Motivation The time used by clean up is too long. This is because the broker service fails to execute the `unloadserviceunit`, and can only rely on the timeout

[GitHub] [pulsar] congbobo184 commented on pull request #14276: [Transaction] delete changeMaxReadPositionAndAddAbortTimes when checkIfNoSnapshot

2022-02-15 Thread GitBox
congbobo184 commented on pull request #14276: URL: https://github.com/apache/pulsar/pull/14276#issuecomment-1041074103 @codelipenghui This is to remove useless code, we don't need to add tests -- This is an automated message from the Apache Git Service. To respond to the message, please

[pulsar] branch master updated (561f8cf -> 3d90165)

2022-02-15 Thread bogong
This is an automated email from the ASF dual-hosted git repository. bogong pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 561f8cf [Transaction]Add auth for PerformanceTransaction (#14271) add 3d90165 [Transaction] Optimize

[GitHub] [pulsar] congbobo184 merged pull request #14277: [Transaction] Optimize changeToCloseState

2022-02-15 Thread GitBox
congbobo184 merged pull request #14277: URL: https://github.com/apache/pulsar/pull/14277 -- 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] congbobo184 commented on pull request #13969: [Transaction] Adopt Single_thread to handle TcClient connecting

2022-02-15 Thread GitBox
congbobo184 commented on pull request #13969: URL: https://github.com/apache/pulsar/pull/13969#issuecomment-1041067471 @codelipenghui please review again -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [pulsar-client-go] wolfstudy opened a new pull request #728: Update version file to 0.8.0

2022-02-15 Thread GitBox
wolfstudy opened a new pull request #728: URL: https://github.com/apache/pulsar-client-go/pull/728 Signed-off-by: xiaolongran ### Motivation Update version file to 0.8.0 -- This is an automated message from the Apache Git Service. To respond to the message,

[pulsar-client-go] branch master updated: add 0.8.0 changelog for repo (#727)

2022-02-15 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 965045a add 0.8.0 changelog for repo

[GitHub] [pulsar] Anonymitaet commented on issue #14291: [Doc] How does pulsar achieve synchronous replication of data across IDCs for disaster recovery?

2022-02-15 Thread GitBox
Anonymitaet commented on issue #14291: URL: https://github.com/apache/pulsar/issues/14291#issuecomment-1041065031 @iamqq23ue welcome to research and update docs, thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [pulsar-client-go] wolfstudy merged pull request #727: add 0.8.0 changelog for repo

2022-02-15 Thread GitBox
wolfstudy merged pull request #727: URL: https://github.com/apache/pulsar-client-go/pull/727 -- 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] ethqunzhong commented on issue #14298: change pulsar.log.level in log4j2.yaml not work?

2022-02-15 Thread GitBox
ethqunzhong commented on issue #14298: URL: https://github.com/apache/pulsar/issues/14298#issuecomment-1041063002 > Did you try with 2.9 or current master? > > Usually I use a ENV property Also if you are using Pulsar standalone the only way is to use the ENV property you

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #14289: [Doc] [Do-not-Merge] Add a parameter to forcefully close fenced topics

2022-02-15 Thread GitBox
Anonymitaet commented on a change in pull request #14289: URL: https://github.com/apache/pulsar/pull/14289#discussion_r807505218 ## File path: site2/docs/reference-configuration.md ## @@ -257,6 +257,7 @@ brokerServiceCompactionThresholdInBytes|If the estimated backlog size is

[GitHub] [pulsar] michaeljmarshall commented on pull request #14306: [issue-14304] Check for blank advertised listener name

2022-02-15 Thread GitBox
michaeljmarshall commented on pull request #14306: URL: https://github.com/apache/pulsar/pull/14306#issuecomment-1041057960 > @michaeljmarshall I think the bug is just in Go client code that is passing an empty string instead of omitting the field. Yes, that is exactly what it looks

svn commit: r52577 - in /dev/pulsar/pulsar-client-go-0.8.0-candidate-1: ./ apache-pulsar-client-go-0.8.0-src.tar.gz apache-pulsar-client-go-0.8.0-src.tar.gz.asc apache-pulsar-client-go-0.8.0-src.tar.g

2022-02-15 Thread rxl
Author: rxl Date: Wed Feb 16 03:23:50 2022 New Revision: 52577 Log: Staging artifacts and signature for Pulsar Client Go release 0.8.0-candidate-1 Added: dev/pulsar/pulsar-client-go-0.8.0-candidate-1/ dev/pulsar/pulsar-client-go-0.8.0-candidate-1/apache-pulsar-client-go-0.8.0-src.tar.gz

[GitHub] [pulsar-client-go] wolfstudy opened a new pull request #727: add 0.8.0 changelog for repo

2022-02-15 Thread GitBox
wolfstudy opened a new pull request #727: URL: https://github.com/apache/pulsar-client-go/pull/727 Signed-off-by: xiaolongran ### Motivation Add 0.8.0 changelog for repo -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] momo-jun commented on pull request #14271: [Transaction]Add auth for PerformanceTransaction

2022-02-15 Thread GitBox
momo-jun commented on pull request #14271: URL: https://github.com/apache/pulsar/pull/14271#issuecomment-1041050856 > Hi @momo-jun can you follow up on the doc? Thanks Sure. I'm on it. -- This is an automated message from the Apache Git Service. To respond to the message, please

[pulsar] branch master updated (aa9f42c -> 561f8cf)

2022-02-15 Thread bogong
This is an automated email from the ASF dual-hosted git repository. bogong pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from aa9f42c Update .asf.yaml to protect release branches (#14226) add 561f8cf [Transaction]Add auth for

[GitHub] [pulsar] merlimat commented on pull request #14306: [issue-14304] Check for blank advertised listener name

2022-02-15 Thread GitBox
merlimat commented on pull request #14306: URL: https://github.com/apache/pulsar/pull/14306#issuecomment-1041048689 > Given that this bug is in the protocol @michaeljmarshall I think the bug is just in Go client code that is passing an empty string instead of omitting the field.

[GitHub] [pulsar] congbobo184 merged pull request #14271: [Transaction]Add auth for PerformanceTransaction

2022-02-15 Thread GitBox
congbobo184 merged pull request #14271: URL: https://github.com/apache/pulsar/pull/14271 -- 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] liangyepianzhou commented on pull request #14238: [Transaction] Adopt single thread pool in TC

2022-02-15 Thread GitBox
liangyepianzhou commented on pull request #14238: URL: https://github.com/apache/pulsar/pull/14238#issuecomment-1041043466 /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 commented on pull request #14277: [Transaction] Optimize changeToCloseState

2022-02-15 Thread GitBox
liangyepianzhou commented on pull request #14277: URL: https://github.com/apache/pulsar/pull/14277#issuecomment-1041043328 /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] congbobo184 commented on pull request #13969: [Transaction] Adopt Single_thread to handle TcClient connecting

2022-02-15 Thread GitBox
congbobo184 commented on pull request #13969: URL: https://github.com/apache/pulsar/pull/13969#issuecomment-1041041668 @codelipenghui please review again. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [pulsar] fu-turer opened a new issue #14311: creat ledger timeout and thread BookKeeperClientWorker in stuck

2022-02-15 Thread GitBox
fu-turer opened a new issue #14311: URL: https://github.com/apache/pulsar/issues/14311 **Describe the bug** we found many timeout logs when creating producer or sending message.like this,we creat a new topic and then build a producer to send message: ```2022-02-15 12:50:06,265Z

[GitHub] [pulsar] gaozhangmin commented on pull request #13939: Fix potential heap memory leak

2022-02-15 Thread GitBox
gaozhangmin commented on pull request #13939: URL: https://github.com/apache/pulsar/pull/13939#issuecomment-1041038628 /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

[pulsar-client-go] annotated tag v0.8.0-candidate-1 updated (5daa17b -> 58d5097)

2022-02-15 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a change to annotated tag v0.8.0-candidate-1 in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git. *** WARNING: tag v0.8.0-candidate-1 was modified! *** from 5daa17b (commit) to 58d5097

[pulsar-client-go] branch branch-0.8.0 created (now 5daa17b)

2022-02-15 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a change to branch branch-0.8.0 in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git. at 5daa17b Markdown error fix (#722) No new revisions were added by this update.

[GitHub] [pulsar] hangc0276 commented on a change in pull request #13575: [PIP-129] Introduce intermediate state for ledger deletion

2022-02-15 Thread GitBox
hangc0276 commented on a change in pull request #13575: URL: https://github.com/apache/pulsar/pull/13575#discussion_r807486500 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ## @@ -146,6 +151,13 @@ protected static

[GitHub] [pulsar] Technoboy- closed issue #12863: Consumer can't receive message from topic

2022-02-15 Thread GitBox
Technoboy- closed issue #12863: URL: https://github.com/apache/pulsar/issues/12863 -- 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 commented on pull request #14282: [pulsar-broker] fix ack-hole and backlog for persistent-replicator

2022-02-15 Thread GitBox
Anonymitaet commented on pull request #14282: URL: https://github.com/apache/pulsar/pull/14282#issuecomment-1041024710 Confirmed w @codelipenghui, this is a bug fix, no need to update docs. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [pulsar] mattisonchao commented on pull request #14309: Fix the wrong parameter in the log.

2022-02-15 Thread GitBox
mattisonchao commented on pull request #14309: URL: https://github.com/apache/pulsar/pull/14309#issuecomment-1041023220 /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

[GitHub] [pulsar] mattisonchao opened a new issue #14310: Flaky-test: BatchSourceExecutorTest. testPushLifeCycle

2022-02-15 Thread GitBox
mattisonchao opened a new issue #14310: URL: https://github.com/apache/pulsar/issues/14310 BatchSourceExecutorTest.testPushLifeCycle is flaky. It fails sporadically. [example failure](https://github.com/apache/pulsar/runs/5209454617?check_suite_focus=true)

[GitHub] [pulsar] Shoothzj commented on pull request #14308: [CLEANUP] Replace the deprecated method in RelativeTimeUtil

2022-02-15 Thread GitBox
Shoothzj commented on pull request #14308: URL: https://github.com/apache/pulsar/pull/14308#issuecomment-1041008311 /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] nodece commented on pull request #14225: [Broker] Move schema compatibility strategy cmd from topics to topicPolicies

2022-02-15 Thread GitBox
nodece commented on pull request #14225: URL: https://github.com/apache/pulsar/pull/14225#issuecomment-1041008022 /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

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

2022-02-15 Thread GitBox
freeznet commented on pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#issuecomment-104036 @lhotari yes I am working on it, and should be done in this week. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[pulsar] 05/08: Fix validateGlobalNamespaceOwnership wrap exception issue. (#14269)

2022-02-15 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 2554db41ca346d032a20d436600db5e97d1e5b14 Author: Jiwei Guo AuthorDate: Tue Feb 15 22:49:17 2022 +0800 Fix

[pulsar] 04/08: Check ``getTlsTrustStorePath`` NPE when user forget to set it. (#14253)

2022-02-15 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 2bfa60152c3e9889d3f2b904b322603c7d81dd55 Author: Qiang Zhao <74767115+mattisonc...@users.noreply.github.com>

[pulsar] 01/08: [pulsar-client] Add conf backoff values (#12520)

2022-02-15 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 9b37232a3a606b4062798f02680e88cbfc1f6faa Author: Callum Duffy AuthorDate: Fri Feb 11 22:08:05 2022 +

[pulsar] branch branch-2.9 updated (d692f8b -> 4c016b9)

2022-02-15 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 d692f8b If mark-delete operation fails, mark the cursor as "dirty" (#14256) new 9b37232 [pulsar-client]

[pulsar] 08/08: Fix time unit error in errMsg when producer send fails. (#14299)

2022-02-15 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 4c016b987e3f55d40307aa24f190427378545039 Author: JiangHaiting AuthorDate: Wed Feb 16 00:07:23 2022 +0800 Fix

[pulsar] 07/08: Fix batch ack count is negtive issue. (#14288)

2022-02-15 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 48dbf01f3e981856fa3b2129ae8d2963f494dc52 Author: Jiwei Guo AuthorDate: Tue Feb 15 21:39:10 2022 +0800 Fix

[pulsar] 06/08: Fix IllegalArgumentException: Invalid period 0.0 to calculate rate

2022-02-15 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 59fdd431bd09d29743657057032d550521b3b517 Author: chenlin <1572139...@qq.com> AuthorDate: Tue Feb 15 18:34:10 2022

[pulsar] 03/08: Fix PersistentAcknowledgmentsGroupingTracker set bitSet issue. (#14260)

2022-02-15 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 b1bf0022443f110ccbd5a726118e2fa21ad41b93 Author: Jiwei Guo AuthorDate: Mon Feb 14 11:18:11 2022 +0800 Fix

[pulsar] 02/08: Remove log unacked msg. (#14246)

2022-02-15 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 da86d9d062fcb30d60d9026719fd51d86ec8fc75 Author: Jiwei Guo AuthorDate: Sat Feb 12 01:34:53 2022 +0800 Remove

[GitHub] [pulsar] mattisonchao opened a new pull request #14309: Fix the wrong parameter in the log.

2022-02-15 Thread GitBox
mattisonchao opened a new pull request #14309: URL: https://github.com/apache/pulsar/pull/14309 ### Motivation Bad parameters in the log will always print "null", which can confuse users. ### Modifications - Correct exception parameter. ### Verifying this change

[GitHub] [pulsar] yuruguo removed a comment on pull request #14301: [pulsar-admin] Add short name for full name in admin cli

2022-02-15 Thread GitBox
yuruguo removed a comment on pull request #14301: URL: https://github.com/apache/pulsar/pull/14301#issuecomment-1040461846 /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] loganloganlogan commented on issue #13235: PIP 113: Update Pulsar Website Design & Content

2022-02-15 Thread GitBox
loganloganlogan commented on issue #13235: URL: https://github.com/apache/pulsar/issues/13235#issuecomment-1040942273 From our weekly website call, we are close to launch. Next steps: - Submitting to website-next repo - Li Li to resolve changes that were conflicting w/ local

[GitHub] [pulsar] Shoothzj opened a new pull request #14308: [CLEANUP] Replace the deprecated method in RelativeTimeUtil

2022-02-15 Thread GitBox
Shoothzj opened a new pull request #14308: URL: https://github.com/apache/pulsar/pull/14308 ### Modifications - Replace the deprecated method in RelativeTimeUtil - fix a typo -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [pulsar] leizhiyuan opened a new issue #14307: expose git info to endpoint of webservice

2022-02-15 Thread GitBox
leizhiyuan opened a new issue #14307: URL: https://github.com/apache/pulsar/issues/14307 **Is your enhancement request related to a problem? Please describe.** 1. now ,we only print the info to log, but when we want to inspect the git info after a while, the log maybe deleted .

[GitHub] [pulsar] EronWright commented on issue #14304: CommandLookupTopic.advertisedListenerName may be blank

2022-02-15 Thread GitBox
EronWright commented on issue #14304: URL: https://github.com/apache/pulsar/issues/14304#issuecomment-1040914915 @michaeljmarshall thanks for the review. I believe that `hasAdvertisedListenerName` would return true for any value, including `null`. I'm trying to write a test into

[GitHub] [pulsar] michaeljmarshall commented on pull request #14306: [issue-14304] Check for blank advertised listener name

2022-02-15 Thread GitBox
michaeljmarshall commented on pull request #14306: URL: https://github.com/apache/pulsar/pull/14306#issuecomment-1040855033 @EronWright - just so you know, your build failed with the following error: > src/main/java/org/apache/pulsar/broker/service/ServerCnx.java:[423,114]

[GitHub] [pulsar] michaeljmarshall commented on issue #14304: CommandLookupTopic.advertisedListenerName may be blank

2022-02-15 Thread GitBox
michaeljmarshall commented on issue #14304: URL: https://github.com/apache/pulsar/issues/14304#issuecomment-1040853045 https://github.com/apache/pulsar/blob/2d092374e189093cce7c792dd31bf9290a3e7d82/pulsar-common/src/main/java/org/apache/pulsar/common/protocol/Commands.java#L820-L822

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

2022-02-15 Thread mmarshall
Author: mmarshall Date: Tue Feb 15 21:17:19 2022 New Revision: 52574 Log: Staging artifacts and signature for Pulsar release 2.8.3 Added: dev/pulsar/pulsar-2.8.3-candidate-2/ dev/pulsar/pulsar-2.8.3-candidate-2/DEB/ dev/pulsar/pulsar-2.8.3-candidate-2/DEB/Packages.gz (with props)

svn commit: r52574 [2/2] - in /dev/pulsar/pulsar-2.8.3-candidate-2: ./ DEB/ RPMS/ RPMS/repodata/ connectors/

2022-02-15 Thread mmarshall
Propchange: dev/pulsar/pulsar-2.8.3-candidate-2/connectors/pulsar-io-elastic-search-2.8.3.nar -- svn:mime-type = application/octet-stream Added:

[pulsar] annotated tag v2.8.3-candidate-2 updated (83a522f -> f90e81d)

2022-02-15 Thread mmarshall
This is an automated email from the ASF dual-hosted git repository. mmarshall pushed a change to annotated tag v2.8.3-candidate-2 in repository https://gitbox.apache.org/repos/asf/pulsar.git. *** WARNING: tag v2.8.3-candidate-2 was modified! *** from 83a522f (commit) to f90e81d

[GitHub] [pulsar] github-actions[bot] commented on pull request #14306: [issue-14304] Check for blank advertised listener name

2022-02-15 Thread GitBox
github-actions[bot] commented on pull request #14306: URL: https://github.com/apache/pulsar/pull/14306#issuecomment-1040754304 @EronWright: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

[GitHub] [pulsar] github-actions[bot] commented on pull request #14306: [issue-14304] Check for blank advertised listener name

2022-02-15 Thread GitBox
github-actions[bot] commented on pull request #14306: URL: https://github.com/apache/pulsar/pull/14306#issuecomment-1040751036 @EronWright:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about

[GitHub] [pulsar] EronWright opened a new pull request #14306: [issue-14304] Check for blank advertised listener name

2022-02-15 Thread GitBox
EronWright opened a new pull request #14306: URL: https://github.com/apache/pulsar/pull/14306 Fixes #14304 ### Modifications - [ServerCnx] check for blank advertised listener name - [ServerCnx] improved logging ### Verifying this change This change is a

[GitHub] [pulsar] prime-minister-of-fun commented on issue #14305: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x

2022-02-15 Thread GitBox
prime-minister-of-fun commented on issue #14305: URL: https://github.com/apache/pulsar/issues/14305#issuecomment-1040743820 Here's the output from running pulsar standalone. I stopped it towards the end: `2022-02-15T20:07:57,250+ [main] INFO

[GitHub] [pulsar] nicoloboschi commented on pull request #14300: Remove --illegal-access errors resulting from Google Guice - Pulsar IO, Offloaders and Pulsar SQL - Bump Guice to 5.1.0

2022-02-15 Thread GitBox
nicoloboschi commented on pull request #14300: URL: https://github.com/apache/pulsar/pull/14300#issuecomment-1040740422 @eolivelli @lhotari PTAL again -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] prime-minister-of-fun opened a new issue #14305: Debezium Demo Doesn't Work with 2.9.1 and Aurora 5.x

2022-02-15 Thread GitBox
prime-minister-of-fun opened a new issue #14305: URL: https://github.com/apache/pulsar/issues/14305 On either amd64 or arm64 Ubuntu images, https://debezium.io/blog/2019/05/23/tutorial-using-debezium-connectors-with-apache-pulsar/ doesn't work. Tests before running. Pod can

[GitHub] [pulsar] EronWright opened a new issue #14304: CommandLookupTopic.advertisedListenerName may be blank

2022-02-15 Thread GitBox
EronWright opened a new issue #14304: URL: https://github.com/apache/pulsar/issues/14304 **Describe the bug** Some of the language bindings supply an empty string for the `advertisedListenerName` field of `CommandLookupTopic` (e.g.

[GitHub] [pulsar] anvinjain commented on a change in pull request #14102: [#14101] [schema] making schema registry configurable

2022-02-15 Thread GitBox
anvinjain commented on a change in pull request #14102: URL: https://github.com/apache/pulsar/pull/14102#discussion_r807139956 ## File path: site2/website-next/versioned_docs/version-2.7.3/reference-configuration.md ## @@ -235,6 +235,7 @@ Pulsar brokers are responsible for

[GitHub] [pulsar] lhotari commented on a change in pull request #13810: Remove --illegal-access errors resulting from Google Guice (upgrade to 5.0.1 and JClouds to 2.4.0)

2022-02-15 Thread GitBox
lhotari commented on a change in pull request #13810: URL: https://github.com/apache/pulsar/pull/13810#discussion_r807002025 ## File path: pulsar-io/data-generator/pom.xml ## @@ -49,6 +49,18 @@ jfairy 0.5.9 + +

[GitHub] [pulsar] michaeljmarshall commented on pull request #14246: Remove log unacked msg.

2022-02-15 Thread GitBox
michaeljmarshall commented on pull request #14246: URL: https://github.com/apache/pulsar/pull/14246#issuecomment-1040624147 @Technoboy- following up here, now that we merged https://github.com/apache/pulsar/pull/14288, is this still required? Given that the log line helped find a bug, I

[GitHub] [pulsar] michaeljmarshall commented on pull request #14071: Fix unack message count for transaction Ack while disabled batch index ack

2022-02-15 Thread GitBox
michaeljmarshall commented on pull request #14071: URL: https://github.com/apache/pulsar/pull/14071#issuecomment-1040658503 @codelipenghui - is this needed in 2.8? I noticed that the `BatchMessageWithBatchIndexLevelTest` is modified here and that created some conflicts for me when cherry

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

2022-02-15 Thread GitBox
michaeljmarshall commented on pull request #13122: URL: https://github.com/apache/pulsar/pull/13122#issuecomment-1040670409 Closing and re-opening to trigger tests. -- 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] codelipenghui commented on a change in pull request #14225: [Broker] Move schema compatibility strategy cmd from topics to topicPolicies

2022-02-15 Thread GitBox
codelipenghui commented on a change in pull request #14225: URL: https://github.com/apache/pulsar/pull/14225#discussion_r806878163 ## File path: tests/integration/src/test/java/org/apache/pulsar/tests/integration/cli/topicpolicies/SchemaCompatibilityStrategyTest.java ## @@

[GitHub] [pulsar] nicoloboschi commented on pull request #14300: Remove --illegal-access errors resulting from Google Guice - Batch Data Generator connector and jcloud-shaded

2022-02-15 Thread GitBox
nicoloboschi commented on pull request #14300: URL: https://github.com/apache/pulsar/pull/14300#issuecomment-1040310259 /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

[GitHub] [pulsar] lhotari merged pull request #14226: Update .asf.yaml to protect release branches

2022-02-15 Thread GitBox
lhotari merged pull request #14226: URL: https://github.com/apache/pulsar/pull/14226 -- 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] lhotari commented on a change in pull request #14300: Remove --illegal-access errors resulting from Google Guice - Batch Data Generator connector and jcloud-shaded

2022-02-15 Thread GitBox
lhotari commented on a change in pull request #14300: URL: https://github.com/apache/pulsar/pull/14300#discussion_r807003538 ## File path: jclouds-shaded/pom.xml ## @@ -44,6 +44,19 @@ javax.annotation javax.annotation-api +

[GitHub] [pulsar] merlimat merged pull request #14299: [client] Fix time unit mismatch in errMsg when producer send fails.

2022-02-15 Thread GitBox
merlimat merged pull request #14299: URL: https://github.com/apache/pulsar/pull/14299 -- 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] michaeljmarshall commented on pull request #14301: [pulsar-admin] Add short name for full name in admin cli

2022-02-15 Thread GitBox
michaeljmarshall commented on pull request #14301: URL: https://github.com/apache/pulsar/pull/14301#issuecomment-1040669454 @yuruguo - I think it makes sense to add short options. I just wonder if we should stay away from `-v`. What is your perspective? -- This is an automated message

  1   2   3   4   >