[GitHub] [pulsar] yuruguo removed a comment on pull request #12315: [client-tools] Optimize execution timing of jcommander.parse

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12315: URL: https://github.com/apache/pulsar/pull/12315#issuecomment-939400717 @merlimat @lhotari @eolivelli @codelipenghui @315157973 @hangc0276 please take a look, thanks! -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] yuruguo removed a comment on pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without num-partitions

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939402589 /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 commented on pull request #12315: [client-tools] Optimize execution timing of jcommander.parse

2021-10-09 Thread GitBox
yuruguo commented on pull request #12315: URL: https://github.com/apache/pulsar/pull/12315#issuecomment-939402637 /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 commented on pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without num-partitions

2021-10-09 Thread GitBox
yuruguo commented on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939402589 /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 commented on pull request #12315: [client-tools] Optimize execution timing of jcommander.parse

2021-10-09 Thread GitBox
yuruguo commented on pull request #12315: URL: https://github.com/apache/pulsar/pull/12315#issuecomment-939400717 @merlimat @lhotari @eolivelli @codelipenghui @315157973 @hangc0276 please take a look, thanks! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] yuruguo opened a new pull request #12315: [client-tools] Optimize execution timing of jcommander.parse

2021-10-09 Thread GitBox
yuruguo opened a new pull request #12315: URL: https://github.com/apache/pulsar/pull/12315 ### Motivation Do pre-judgment processing before calling `jcommander.parse` in CLI `pulsar-admin` / `pulsar-client`, including `version` command and `help` command ### Documentation -

[GitHub] [pulsar] yuruguo commented on a change in pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without num-partitions

2021-10-09 Thread GitBox
yuruguo commented on a change in pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#discussion_r725567543 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdNamespaces.java ## @@ -573,8 +573,13 @@ void run() throws

[GitHub] [pulsar] flowchartsman commented on pull request #12013: [Issue 9772][Go Functions]Allow User Metrics

2021-10-09 Thread GitBox
flowchartsman commented on pull request #12013: URL: https://github.com/apache/pulsar/pull/12013#issuecomment-939394508 @merlimat @wolfstudy @freeznet Think we should be good now, if anyone wants to help me put it to bed. -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] Shoothzj commented on pull request #12170: [pulsar-java-client] Auto-recovery after exception like out of direct memory

2021-10-09 Thread GitBox
Shoothzj commented on pull request #12170: URL: https://github.com/apache/pulsar/pull/12170#issuecomment-939391518 /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 #12200: Allow to config pulsar client allocator out of memory policy

2021-10-09 Thread GitBox
Shoothzj commented on pull request #12200: URL: https://github.com/apache/pulsar/pull/12200#issuecomment-939391482 /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 (aa408ff -> 3de54d8)

2021-10-09 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 aa408ff fix ttl expiry does not take effect (#12266) add 3de54d8 presto sample doc missing the set field

[GitHub] [pulsar] codelipenghui merged pull request #12127: presto sample doc missing the set field method

2021-10-09 Thread GitBox
codelipenghui merged pull request #12127: URL: https://github.com/apache/pulsar/pull/12127 -- 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 #12020: [testclient] Make option -time take effect in PerformanceClient

2021-10-09 Thread GitBox
yuruguo commented on pull request #12020: URL: https://github.com/apache/pulsar/pull/12020#issuecomment-939389122 @merlimat @k2la @massakam @codelipenghui @congbobo184 PTAL, thx! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] yuruguo removed a comment on pull request #12020: [testclient] Make option -time take effect in PerformanceClient

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12020: URL: https://github.com/apache/pulsar/pull/12020#issuecomment-939191520 @merlimat @k2la @massakam PTAL, thx! -- 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] yuruguo removed a comment on pull request #12242: [pulsar-admin] Add option --reset-mode to optimize topic setPublishRate/setDispatchRate policy

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12242: URL: https://github.com/apache/pulsar/pull/12242#issuecomment-939221133 @merlimat @ivankelly @massakam @codelipenghui @315157973 @BewareMyPower @hangc0276 PTAL, thx! -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] yuruguo commented on pull request #12242: [pulsar-admin] Add option --reset-mode to optimize topic setPublishRate/setDispatchRate policy

2021-10-09 Thread GitBox
yuruguo commented on pull request #12242: URL: https://github.com/apache/pulsar/pull/12242#issuecomment-93932 @merlimat @ivankelly @massakam @codelipenghui @315157973 @BewareMyPower @hangc0276 PTAL, thx! -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] yuruguo commented on pull request #12300: [pulsar-perf] Support version command

2021-10-09 Thread GitBox
yuruguo commented on pull request #12300: URL: https://github.com/apache/pulsar/pull/12300#issuecomment-939388693 @merlimat @lhotari @eolivelli @sijie @codelipenghui @315157973 @hangc0276 PTAL, thx! -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] yuruguo removed a comment on pull request #12300: [pulsar-perf] Support version command

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12300: URL: https://github.com/apache/pulsar/pull/12300#issuecomment-939188080 @merlimat @rdhabalia @lhotari @eolivelli @sijie @codelipenghui @murong00 @hangc0276 PTAL, thx! -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] flowchartsman opened a new issue #12314: [Go Functions] ProcessedSuccessfully metric is counted twice.

2021-10-09 Thread GitBox
flowchartsman opened a new issue #12314: URL: https://github.com/apache/pulsar/issues/12314 **Describe the bug** ProcessedSuccessfully metric is counted twice. **To Reproduce** Observe go function: ``` $ pulsarctl functions status --fqfn tenant/namespace/functionName {

[GitHub] [pulsar] yuruguo removed a comment on pull request #12128: [pulsar-admin] Allow setting --forward-source-message-property to false when updating a pulsar function

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12128: URL: https://github.com/apache/pulsar/pull/12128#issuecomment-939188308 @merlimat @lhotari @eolivelli @aahmed-se @k2la @codelipenghui @315157973 @BewareMyPower @hangc0276 @gaoran10 PTAL, thx! -- This is an automated message from the

[GitHub] [pulsar] yuruguo removed a comment on pull request #12187: [testclient] Make --payload-file take effect in PerformanceClient

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12187: URL: https://github.com/apache/pulsar/pull/12187#issuecomment-938219849 @merlimat @lhotari @aahmed-se @sijie @codelipenghui @jiazhai @BewareMyPower @hangc0276 @gaoran10 PTAL, thx! -- This is an automated message from the Apache Git

[GitHub] [pulsar] yuruguo removed a comment on pull request #12251: [pulsar-admin] Add option --reset-mode to optimize namespace setPublishRate/setDispatchRate policy

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12251: URL: https://github.com/apache/pulsar/pull/12251#issuecomment-939227629 @merlimat @ivankelly @eolivelli @massakam @codelipenghui @315157973 @hangc0276 @gaoran10 PTAL, thx! -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] yuruguo removed a comment on pull request #12260: [pulsar-admin] New option takes precedence over deprecated option

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12260: URL: https://github.com/apache/pulsar/pull/12260#issuecomment-938218371 @merlimat @lhotari @aahmed-se @codelipenghui @315157973 @BewareMyPower @hangc0276 @gaoran10 PTAL, thx! -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar] yuruguo removed a comment on pull request #12263: [testclient] Replace '--auth_plugin' with '--auth-plugin'

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12263: URL: https://github.com/apache/pulsar/pull/12263#issuecomment-938217394 @merlimat @lhotari @aahmed-se @codelipenghui @BewareMyPower @hangc0276 @gaoran10 PTAL, thx! -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] yuruguo removed a comment on pull request #12296: [client-tools] Remove redundant initial value

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12296: URL: https://github.com/apache/pulsar/pull/12296#issuecomment-939219920 @merlimat @eolivelli @codelipenghui @freeznet @BewareMyPower PTAL, thx! -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [pulsar] yuruguo removed a comment on pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939271244 @merlimat @eolivelli @codelipenghui @BewareMyPower @hangc0276 @gaoran10 @freeznet PTAL, thx! -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] yuruguo commented on a change in pull request #12242: [pulsar-admin] Add option --reset-mode to optimize topic setPublishRate/setDispatchRate policy

2021-10-09 Thread GitBox
yuruguo commented on a change in pull request #12242: URL: https://github.com/apache/pulsar/pull/12242#discussion_r725537202 ## File path: pulsar-client-admin-api/src/main/java/org/apache/pulsar/client/admin/Topics.java ## @@ -2370,23 +2370,27 @@ void

[GitHub] [pulsar] yuruguo commented on a change in pull request #12242: [pulsar-admin] Add option --reset-mode to optimize topic setPublishRate/setDispatchRate policy

2021-10-09 Thread GitBox
yuruguo commented on a change in pull request #12242: URL: https://github.com/apache/pulsar/pull/12242#discussion_r723084699 ## File path: pulsar-client-admin-api/src/main/java/org/apache/pulsar/client/admin/Topics.java ## @@ -2370,23 +2370,27 @@ void

[GitHub] [pulsar] thomaeschen opened a new issue #12313: Error during handshake for tls auth by apache pulsar client on linux server

2021-10-09 Thread GitBox
thomaeschen opened a new issue #12313: URL: https://github.com/apache/pulsar/issues/12313 **Describe the bug** Follow the instruction from the Apache latest document to setup a TLS connection. The system will generate error related to the handshake and it shows

[GitHub] [pulsar] yuruguo edited a comment on pull request #12300: [pulsar-perf] Support version command

2021-10-09 Thread GitBox
yuruguo edited a comment on pull request #12300: URL: https://github.com/apache/pulsar/pull/12300#issuecomment-939188080 @merlimat @rdhabalia @lhotari @eolivelli @sijie @codelipenghui @murong00 @hangc0276 PTAL, thx! -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] hangc0276 commented on a change in pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
hangc0276 commented on a change in pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#discussion_r725493690 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdNamespaces.java ## @@ -573,8 +573,13 @@ void run() throws

[pulsar] branch master updated: fix ttl expiry does not take effect (#12266)

2021-10-09 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 aa408ff fix ttl expiry does not take effect

[GitHub] [pulsar] codelipenghui merged pull request #12266: fix ttl expiry does not take effect

2021-10-09 Thread GitBox
codelipenghui merged pull request #12266: URL: https://github.com/apache/pulsar/pull/12266 -- 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] codelipenghui closed issue #12265: ttl setting does not take effect when expiry

2021-10-09 Thread GitBox
codelipenghui closed issue #12265: URL: https://github.com/apache/pulsar/issues/12265 -- 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 commented on a change in pull request #12302: reset-cursor support earliest or latest

2021-10-09 Thread GitBox
BewareMyPower commented on a change in pull request #12302: URL: https://github.com/apache/pulsar/pull/12302#discussion_r725488135 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdTopics.java ## @@ -778,6 +778,14 @@ void run() throws

[GitHub] [pulsar] BewareMyPower commented on a change in pull request #12302: reset-cursor support earliest or latest

2021-10-09 Thread GitBox
BewareMyPower commented on a change in pull request #12302: URL: https://github.com/apache/pulsar/pull/12302#discussion_r725488048 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdTopics.java ## @@ -778,6 +778,14 @@ void run() throws

[GitHub] [pulsar] codelipenghui commented on a change in pull request #12240: fix the race of delete subscription and delete topic

2021-10-09 Thread GitBox
codelipenghui commented on a change in pull request #12240: URL: https://github.com/apache/pulsar/pull/12240#discussion_r725487937 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java ## @@ -1006,24 +1006,33 @@ public void

[GitHub] [pulsar] yuruguo removed a comment on pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939277515 /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] liangyepianzhou opened a new pull request #12312: Fix lowWaterMark of TopicTransactionBuffer

2021-10-09 Thread GitBox
liangyepianzhou opened a new pull request #12312: URL: https://github.com/apache/pulsar/pull/12312 ### Motivation Fix a bug written in handleLowWaterMark. When we find that the TxnID of a Transaction in the ongoings which is less than LowWaterMark , we need to abort the

[GitHub] [pulsar] yuruguo edited a comment on pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
yuruguo edited a comment on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939271244 @merlimat @eolivelli @codelipenghui @BewareMyPower @hangc0276 @gaoran10 @freeznet PTAL, thx! -- This is an automated message from the Apache Git Service. To respond

[pulsar] branch master updated (8b50af5 -> f9057c7)

2021-10-09 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 8b50af5 [Transaction] Transaction pending ack lazy init. (#11091) add f9057c7 Improve exceptions thrown when

[GitHub] [pulsar] codelipenghui merged pull request #12155: [Broker] Improve exceptions thrown when handling the schema resource

2021-10-09 Thread GitBox
codelipenghui merged pull request #12155: URL: https://github.com/apache/pulsar/pull/12155 -- 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] codelipenghui closed issue #12010: Pulsar standalone console should print better error message when schema is invalid

2021-10-09 Thread GitBox
codelipenghui closed issue #12010: URL: https://github.com/apache/pulsar/issues/12010 -- 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 #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
yuruguo removed a comment on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939273354 /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 commented on pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
yuruguo commented on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939277515 /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] hangc0276 opened a new pull request #12311: useV2WireProtocol for bookkeeper autorecovery

2021-10-09 Thread GitBox
hangc0276 opened a new pull request #12311: URL: https://github.com/apache/pulsar/pull/12311 ### Motivation Currently we only set `bookkeeperUseV2WireProtocol=true` on Pulsar brokers to enforce brokers to use v2 protocol to avoid GC issues introduced by protobuf. However, we

[GitHub] [pulsar] yuruguo commented on pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
yuruguo commented on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939273354 /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 commented on pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
yuruguo commented on pull request #12310: URL: https://github.com/apache/pulsar/pull/12310#issuecomment-939271244 @merlimat @eolivelli @codelipenghui @BewareMyPower PTAL, thx! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar] yuruguo opened a new pull request #12310: [pulsar-admin] Fix NPE when executing set-auto-topic-creation without setting num-partitions

2021-10-09 Thread GitBox
yuruguo opened a new pull request #12310: URL: https://github.com/apache/pulsar/pull/12310 ### Motivation When I execute the `set-auto-topic-creation` command on namesapce to specify `-e` and specify `-t` as _**partitioned**_ but not specify -n, an NPE will occur. as below,

[GitHub] [pulsar] frankxieke opened a new pull request #12309: [offload] fix FileSystemManagedLedgerOffloader can not cleanup outdated ledger

2021-10-09 Thread GitBox
frankxieke opened a new pull request #12309: URL: https://github.com/apache/pulsar/pull/12309 ### Motivation * When using FileSystem Offloader, and you set the ledger retention policy, after the offloaded ledger is outdated, however the data can not be deleted from file system.

[GitHub] [pulsar-client-go] GPrabhudas commented on pull request #552: Encryption support in Go client

2021-10-09 Thread GitBox
GPrabhudas commented on pull request #552: URL: https://github.com/apache/pulsar-client-go/pull/552#issuecomment-939262293 > ping @cckellogg @GPrabhudas What is the latest status here? @wolfstudy We can close this PR as all of its sub PRs are merged into master. #555 base

[GitHub] [pulsar-client-go] wolfstudy commented on pull request #552: Encryption support in Go client

2021-10-09 Thread GitBox
wolfstudy commented on pull request #552: URL: https://github.com/apache/pulsar-client-go/pull/552#issuecomment-939261353 ping @cckellogg @GPrabhudas What is the latest status here? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar-client-go] wolfstudy closed issue #500: Make Topic Level Prometheus Metrics Optional

2021-10-09 Thread GitBox
wolfstudy closed issue #500: URL: https://github.com/apache/pulsar-client-go/issues/500 -- 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-client-go] wolfstudy merged pull request #604: User-defined metric cardinality

2021-10-09 Thread GitBox
wolfstudy merged pull request #604: URL: https://github.com/apache/pulsar-client-go/pull/604 -- 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: User-defined metric cardinality (#604)

2021-10-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 ce79489 User-defined metric cardinality

[GitHub] [pulsar-client-go] wolfstudy merged pull request #612: Encryption support ext consumer

2021-10-09 Thread GitBox
wolfstudy merged pull request #612: URL: https://github.com/apache/pulsar-client-go/pull/612 -- 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: Encryption support ext consumer (#612)

2021-10-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 ab96ad7 Encryption support ext consumer

[GitHub] [pulsar-client-go] wolfstudy merged pull request #616: [Issue 615] Fix retry on specified-partition topic

2021-10-09 Thread GitBox
wolfstudy merged pull request #616: URL: https://github.com/apache/pulsar-client-go/pull/616 -- 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-client-go] wolfstudy closed issue #615: Consume retry not work on specified-partition topic

2021-10-09 Thread GitBox
wolfstudy closed issue #615: URL: https://github.com/apache/pulsar-client-go/issues/615 -- 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 615] Fix retry on specified-partition topic (#616)

2021-10-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 e9629fb [Issue 615] Fix retry on

[GitHub] [pulsar] liangyepianzhou closed pull request #12268: Code Form

2021-10-09 Thread GitBox
liangyepianzhou closed pull request #12268: URL: https://github.com/apache/pulsar/pull/12268 -- 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-client-go] wuYin closed pull request #579: Ignore partitions shrink in client side

2021-10-09 Thread GitBox
wuYin closed pull request #579: URL: https://github.com/apache/pulsar-client-go/pull/579 -- 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] Jason918 opened a new pull request #12308: [ISSUE-12291][Client] 'StartMessageId' and 'RollbackDuration' not working in MultiTopicsReader for non-partitioned topics

2021-10-09 Thread GitBox
Jason918 opened a new pull request #12308: URL: https://github.com/apache/pulsar/pull/12308 Fixes #12291 ### Motivation Bug fix. 'StartMessageId' and 'RollbackDuration' is not working in MultiTopicsReader for non-partitioned topics. ### Modifications This fix is

[pulsar-client-go] branch master updated (d336ff7 -> 44d6b16)

2021-10-09 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git. from d336ff7 Remove mutitopic- and regexp consumer along with reader from client' handlers map when Close

[GitHub] [pulsar-client-go] wolfstudy merged pull request #619: Fix ConsumersOpened counter not incremented when use multitopic or regexp consumer

2021-10-09 Thread GitBox
wolfstudy merged pull request #619: URL: https://github.com/apache/pulsar-client-go/pull/619 -- 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-client-go] wolfstudy merged pull request #620: Fix multitopic- and regexp consumers not removed from client handlers on Close

2021-10-09 Thread GitBox
wolfstudy merged pull request #620: URL: https://github.com/apache/pulsar-client-go/pull/620 -- 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: Remove mutitopic- and regexp consumer along with reader from client' handlers map when Close called. (#620)

2021-10-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 d336ff7 Remove mutitopic- and regexp

[GitHub] [pulsar-client-go] wolfstudy commented on pull request #579: Ignore partitions shrink in client side

2021-10-09 Thread GitBox
wolfstudy commented on pull request #579: URL: https://github.com/apache/pulsar-client-go/pull/579#issuecomment-939246838 @wuYin Maybe the https://github.com/apache/pulsar-client-go/pull/601 will fix the error, please use the master version and verify the logic. -- This is an automated

[GitHub] [pulsar-client-go] wolfstudy commented on pull request #616: [Issue 615] Fix retry on specified-partition topic

2021-10-09 Thread GitBox
wolfstudy commented on pull request #616: URL: https://github.com/apache/pulsar-client-go/pull/616#issuecomment-939244391 ping @9ft please merge master code and fix the action CI, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar-client-go] wolfstudy commented on pull request #619: Fix ConsumersOpened counter not incremented when use multitopic or regexp consumer

2021-10-09 Thread GitBox
wolfstudy commented on pull request #619: URL: https://github.com/apache/pulsar-client-go/pull/619#issuecomment-939244244 ping @PowerStateFailure please merge master code and fix the action CI, thanks. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar-client-go] wolfstudy commented on pull request #620: Fix multitopic- and regexp consumers not removed from client handlers on Close

2021-10-09 Thread GitBox
wolfstudy commented on pull request #620: URL: https://github.com/apache/pulsar-client-go/pull/620#issuecomment-939244214 ping @PowerStateFailure please merge master code and fix the action CI, thanks. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar-client-go] wolfstudy commented on issue #626: github.com/keybase/go-keychain error with warning on MacOS 11.4 M1

2021-10-09 Thread GitBox
wolfstudy commented on issue #626: URL: https://github.com/apache/pulsar-client-go/issues/626#issuecomment-939243588 > I just try to delete this package and got the error. Seems like oauth2 need this package. > > ``` > go:

[GitHub] [pulsar-client-go] wolfstudy commented on issue #628: please check Failed by consumer Ack Message ID

2021-10-09 Thread GitBox
wolfstudy commented on issue #628: URL: https://github.com/apache/pulsar-client-go/issues/628#issuecomment-939242977 The current Ack command has no return value. If the ack fails, the message will continue to be stored in the backlog on the broker side, and the message will not be lost.

[GitHub] [pulsar] Shoothzj commented on pull request #12200: Allow to config pulsar client allocator out of memory policy

2021-10-09 Thread GitBox
Shoothzj commented on pull request #12200: URL: https://github.com/apache/pulsar/pull/12200#issuecomment-939238362 /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 #12170: [pulsar-java-client] Auto-recovery after exception like out of direct memory

2021-10-09 Thread GitBox
Shoothzj commented on pull request #12170: URL: https://github.com/apache/pulsar/pull/12170#issuecomment-939238331 /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