[pulsar] branch master updated: Allow to configure schema compatibility policy for system topics (#12598)

2021-11-03 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 7aea58d Allow to configure schema

[GitHub] [pulsar] codelipenghui merged pull request #12598: Allow to configure schema compatibility policy for system topics

2021-11-03 Thread GitBox
codelipenghui merged pull request #12598: URL: https://github.com/apache/pulsar/pull/12598 -- 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] 609101723 commented on issue #12618: The python pulsar client creates a producer and then closes the producer. The memory will not be released

2021-11-03 Thread GitBox
609101723 commented on issue #12618: URL: https://github.com/apache/pulsar/issues/12618#issuecomment-960462963 > @609101723 The PR #12409 was marked for 2.10.0, 2.9.1 and 2.8.2 releases and none of them were released yet. Which version will be released? When will it be released?

[GitHub] [pulsar] 609101723 commented on issue #12618: The python pulsar client creates a producer and then closes the producer. The memory will not be released

2021-11-03 Thread GitBox
609101723 commented on issue #12618: URL: https://github.com/apache/pulsar/issues/12618#issuecomment-960462796 Which version will be released? When will it be released? Otherwise I can't use this module now -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] lhotari edited a comment on pull request #12606: [ML] Avoid passing OpAddEntry across a thread boundary in asyncAddEntry

2021-11-03 Thread GitBox
lhotari edited a comment on pull request #12606: URL: https://github.com/apache/pulsar/pull/12606#issuecomment-960458045 > This change makes sense to me. Do you have any evidence of a problem that this PR is fixing? Direct evidence in a form of a failing test is hard for a lot of

[GitHub] [pulsar] lhotari commented on pull request #12606: [ML] Avoid passing OpAddEntry across a thread boundary in asyncAddEntry

2021-11-03 Thread GitBox
lhotari commented on pull request #12606: URL: https://github.com/apache/pulsar/pull/12606#issuecomment-960458045 > This change makes sense to me. Do you have any evidence of a problem that this PR is fixing? Direct evidence in a form of a failing test is hard for a lot of thread

[GitHub] [pulsar] github-actions[bot] commented on pull request #12619: Fix String should use equals but not ==.

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

[GitHub] [pulsar] Raven888888 commented on issue #12455: PulsarClientException$BrokerMetadataException error reoccurs in release 2.8.1

2021-11-03 Thread GitBox
Raven88 commented on issue #12455: URL: https://github.com/apache/pulsar/issues/12455#issuecomment-960450869 No I haven't @BewareMyPower , I will try it when I have the capacity, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [pulsar] merlimat commented on issue #12618: The python pulsar client creates a producer and then closes the producer. The memory will not be released

2021-11-03 Thread GitBox
merlimat commented on issue #12618: URL: https://github.com/apache/pulsar/issues/12618#issuecomment-960449679 @609101723 The PR #12409 was marked for 2.10.0, 2.9.1 and 2.8.2 releases and none of them were released yet. -- This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] hangc0276 closed issue #12563: apache-pulsar-2.8.1#bin/pulsar standalone#ERROR org.apache.bookkeeper.clients.impl.internal

2021-11-03 Thread GitBox
hangc0276 closed issue #12563: URL: https://github.com/apache/pulsar/issues/12563 -- 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] hangc0276 commented on issue #12616: [BUG]KOP failed to get partition offset

2021-11-03 Thread GitBox
hangc0276 commented on issue #12616: URL: https://github.com/apache/pulsar/issues/12616#issuecomment-960440058 > > I think we should fix it at KoP side. > > any idea? It's better to check whether the `firstPosition` exists in ManagedLedger in KOP side. -- This is an

[GitHub] [pulsar] hepyu closed issue #12533: bookie entry的p999写入时间达到5秒可能是什么原因?

2021-11-03 Thread GitBox
hepyu closed issue #12533: URL: https://github.com/apache/pulsar/issues/12533 -- 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] hepyu commented on issue #12533: bookie entry的p999写入时间达到5秒可能是什么原因?

2021-11-03 Thread GitBox
hepyu commented on issue #12533: URL: https://github.com/apache/pulsar/issues/12533#issuecomment-960434063 > > But will 50ms ~ 100ms GC cause bookie to write p999 for about 5 seconds? > > No, but a long STW event could. It's better to add "-XX:+PrintGCApplicationStoppedTime" in JVM

[GitHub] [pulsar] xiaotongwang1 commented on issue #12616: [BUG]KOP failed to get partition offset

2021-11-03 Thread GitBox
xiaotongwang1 commented on issue #12616: URL: https://github.com/apache/pulsar/issues/12616#issuecomment-960426783 > I think we should fix it at KoP side. any idea? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] 609101723 opened a new issue #12618: The python pulsar client creates a producer and then closes the producer. The memory will not be released

2021-11-03 Thread GitBox
609101723 opened a new issue #12618: URL: https://github.com/apache/pulsar/issues/12618 Just like issue #12408, I raised a question. I kept creating and closing producers, but the memory would not be released. I saw that the problem had been solved. When I re installed pulsar client with

[GitHub] [pulsar] Richard-Yi opened a new issue #12617: Prometheus collect the metric fail

2021-11-03 Thread GitBox
Richard-Yi opened a new issue #12617: URL: https://github.com/apache/pulsar/issues/12617 **Describe the bug** Prometheus collect the metric of one of brokers fail while the metric interface returns normal metrics and the other broker is successfully collected. I think there may

[GitHub] [pulsar] BewareMyPower commented on issue #12616: [BUG]KOP failed to get partition offset

2021-11-03 Thread GitBox
BewareMyPower commented on issue #12616: URL: https://github.com/apache/pulsar/issues/12616#issuecomment-960424962 I think we should fix it at KoP side. -- 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 closed issue #650: Producer handle ReceivedSendReceipt will close cnx when response sequenceID is not equal pendingQueue msg sequenceID

2021-11-03 Thread GitBox
wolfstudy closed issue #650: URL: https://github.com/apache/pulsar-client-go/issues/650 -- 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 #651: fix issue 650,different handle sequence value

2021-11-03 Thread GitBox
wolfstudy merged pull request #651: URL: https://github.com/apache/pulsar-client-go/pull/651 -- 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: fix issue 650, different handle sequence value (#651)

2021-11-03 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 d80a722 fix issue 650,different handle

[GitHub] [pulsar] wolfstudy commented on pull request #12566: Add negative ack redelivery backoff.

2021-11-03 Thread GitBox
wolfstudy commented on pull request #12566: URL: https://github.com/apache/pulsar/pull/12566#issuecomment-960407268 > Confirmed with @wolfstudy, he will provide tech inputs for doc Ok, will add docs in next pull request. -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar] xiaotongwang1 opened a new issue #12616: [BUG]KOP failed to get partition offset

2021-11-03 Thread GitBox
xiaotongwang1 opened a new issue #12616: URL: https://github.com/apache/pulsar/issues/12616 @hangc0276 @BewareMyPower @codelipenghui 紧急求助 [](https://github.com/apache/pulsar/pull/11912) 这个PR,在asyncReadEntry的时候做了ledger map匹配校验 但是原有的接口: `public PositionImpl

[GitHub] [pulsar] aloyszhang commented on issue #12601: Geo Replica with one producer and two "Shared" Consumers

2021-11-03 Thread GitBox
aloyszhang commented on issue #12601: URL: https://github.com/apache/pulsar/issues/12601#issuecomment-960394834 The subscription stats will snapshot at a specified interval which default is 1 second. Then this snapshot will send to remote clusters, so you may receive duplicated

[GitHub] [pulsar] github-actions[bot] commented on pull request #12615: fix 12614, waitingForPingResponse needs to be modified with volatile for concurrent sence

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

[GitHub] [pulsar] baomingyu opened a new pull request #12615: fix 12614, waitingForPingResponse needs to be modified with volatile for concurrent sence

2021-11-03 Thread GitBox
baomingyu opened a new pull request #12615: URL: https://github.com/apache/pulsar/pull/12615 Fixes #12614 Master Issue: #12614 waitingForPingResponse was used in PulsarHandler for check if or not close connection. and it was set value in different thread , so it

[GitHub] [pulsar] baomingyu opened a new issue #12614: waitingForPingResponse needs to be modified with volatile for concurrent sence

2021-11-03 Thread GitBox
baomingyu opened a new issue #12614: URL: https://github.com/apache/pulsar/issues/12614 waitingForPingResponse was used in PulsarHandler for check if or not close connection. and it was set value in different thread , so it must to be modified with volatile. -- This is an

[GitHub] [pulsar] LeBW commented on a change in pull request #12523: add a metric to get earliest time in the backlog

2021-11-03 Thread GitBox
LeBW commented on a change in pull request #12523: URL: https://github.com/apache/pulsar/pull/12523#discussion_r742484742 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentSubscription.java ## @@ -1065,6 +1066,12 @@ public

[GitHub] [pulsar] hangc0276 commented on a change in pull request #12523: add a metric to get earliest time in the backlog

2021-11-03 Thread GitBox
hangc0276 commented on a change in pull request #12523: URL: https://github.com/apache/pulsar/pull/12523#discussion_r742469504 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentSubscription.java ## @@ -1065,6 +1066,12 @@ public

[GitHub] [pulsar] codelipenghui merged pull request #12607: [Test] Cleanup ProxyPublishConsumeTest

2021-11-03 Thread GitBox
codelipenghui merged pull request #12607: URL: https://github.com/apache/pulsar/pull/12607 -- 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] MarvinCai opened a new pull request #12613: [Doc] Update Pulsar SQL document.

2021-11-03 Thread GitBox
MarvinCai opened a new pull request #12613: URL: https://github.com/apache/pulsar/pull/12613 ### Motivation See couple users ask about why Pulsar SQL can't query latest message from topic, feel we should update the doc. Also seems the config part is out of date, so also updated the

[GitHub] [pulsar] Jason918 commented on a change in pull request #12604: Optimize OverloadShedder: Sort bundle by throughput in ascending order

2021-11-03 Thread GitBox
Jason918 commented on a change in pull request #12604: URL: https://github.com/apache/pulsar/pull/12604#discussion_r742473185 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/OverloadShedder.java ## @@ -121,8 +121,8 @@

[GitHub] [pulsar] BewareMyPower commented on issue #12455: PulsarClientException$BrokerMetadataException error reoccurs in release 2.8.1

2021-11-03 Thread GitBox
BewareMyPower commented on issue #12455: URL: https://github.com/apache/pulsar/issues/12455#issuecomment-960362881 It looks like I only replied via email so it didn't appear in this issue. IMO, it's correct and you can delete this topic manually. Have you tried it? -- This is an

[GitHub] [pulsar] BewareMyPower closed issue #12568: KOP getOffsetOfPosition failed ( Invalid unknonwn tag type: 6 )

2021-11-03 Thread GitBox
BewareMyPower closed issue #12568: URL: https://github.com/apache/pulsar/issues/12568 -- 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] github-actions[bot] commented on pull request #12613: [Doc] Update Pulsar SQL document.

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12613: URL: https://github.com/apache/pulsar/pull/12613#issuecomment-960362583 @MarvinCai: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] BewareMyPower commented on issue #12568: KOP getOffsetOfPosition failed ( Invalid unknonwn tag type: 6 )

2021-11-03 Thread GitBox
BewareMyPower commented on issue #12568: URL: https://github.com/apache/pulsar/issues/12568#issuecomment-960362907 After confirming with @xiaotongwang1 , this issue could be fixed by #12018. So close it. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] github-actions[bot] commented on pull request #12607: [Test] Cleanup ProxyPublishConsumeTest

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

[GitHub] [pulsar] urfreespace opened a new pull request #12612: [website][upgrade]feat: docs migration - 2.7.1 / client

2021-11-03 Thread GitBox
urfreespace opened a new pull request #12612: URL: https://github.com/apache/pulsar/pull/12612 *(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] lordcheng10 commented on pull request #12604: Optimize OverloadShedder: Sort bundle by throughput in ascending order

2021-11-03 Thread GitBox
lordcheng10 commented on pull request #12604: URL: https://github.com/apache/pulsar/pull/12604#issuecomment-959730273 -- 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] yuruguo commented on pull request #12392: [pulsar-admin] Add remove-subscription-types-enabled command for namespace

2021-11-03 Thread GitBox
yuruguo commented on pull request #12392: URL: https://github.com/apache/pulsar/pull/12392#issuecomment-959024668 @eolivelli @codelipenghui @hangc0276 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

[GitHub] [pulsar] RobertIndie commented on pull request #12403: [PIP 107][Client] Introduce chunk message ID

2021-11-03 Thread GitBox
RobertIndie commented on pull request #12403: URL: https://github.com/apache/pulsar/pull/12403#issuecomment-958950788 /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] eolivelli commented on pull request #12606: [ML] Avoid passing OpAddEntry across a thread boundary in asyncAddEntry

2021-11-03 Thread GitBox
eolivelli commented on pull request #12606: URL: https://github.com/apache/pulsar/pull/12606#issuecomment-959976341 This change makes sense to me. Do you have any evidence of a problem that this PR is fixing? -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] lhotari removed a comment on pull request #12606: [ML] Avoid passing OpAddEntry across a thread boundary in asyncAddEntry

2021-11-03 Thread GitBox
lhotari removed a comment on pull request #12606: URL: https://github.com/apache/pulsar/pull/12606#issuecomment-959787897 The orignal PR itself is invalid since buffer would have to be retained before passing to the other thread. -- This is an automated message from the Apache Git

[GitHub] [pulsar] github-actions[bot] commented on pull request #12609: [website][upgrade]feat: docs migration - 2.7.1 / administration

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12609: URL: https://github.com/apache/pulsar/pull/12609#issuecomment-960355823 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] lhotari commented on pull request #12606: [ML] Avoid passing OpAddEntry across a thread boundary in asyncAddEntry

2021-11-03 Thread GitBox
lhotari commented on pull request #12606: URL: https://github.com/apache/pulsar/pull/12606#issuecomment-959787240 -- 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] github-actions[bot] removed a comment on pull request #12606: [ML] Avoid passing OpAddEntry across a thread boundary in asyncAddEntry

2021-11-03 Thread GitBox
github-actions[bot] removed a comment on pull request #12606: URL: https://github.com/apache/pulsar/pull/12606#issuecomment-959770438 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] Anonymitaet merged pull request #12580: [website][upgrade]feat: docs migration - 2.7.1 / helm

2021-11-03 Thread GitBox
Anonymitaet merged pull request #12580: URL: https://github.com/apache/pulsar/pull/12580 -- 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] zbentley commented on issue #12554: Attempting to delete a topic which has recently been created sometimes fails with a 500 error, with an internal broker error of "Maximum redirect

2021-11-03 Thread GitBox
zbentley commented on issue #12554: URL: https://github.com/apache/pulsar/issues/12554#issuecomment-958991410 This behavior also persists when connecting directly to brokers and following redirects to find the authoritative home for a topic. This seems to leave topics in a

[GitHub] [pulsar] anigkus commented on issue #12563: apache-pulsar-2.8.1#bin/pulsar standalone#ERROR org.apache.bookkeeper.clients.impl.internal

2021-11-03 Thread GitBox
anigkus commented on issue #12563: URL: https://github.com/apache/pulsar/issues/12563#issuecomment-958713654 > Delete data dir(data/standalone) and then start pulsar. I adjusted the memory and it was fina. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] madhavan-narayanan commented on pull request #12536: [pulsar-broker] Broker extensions to allow operators of enterprise wide cluster better control and flexibility

2021-11-03 Thread GitBox
madhavan-narayanan commented on pull request #12536: URL: https://github.com/apache/pulsar/pull/12536#issuecomment-959734005 > @madhavan-narayanan It should be good to start with a proposal for this new feature, I noticed there are new API introduced. For starting a proposal for Pulsar,

[GitHub] [pulsar] github-actions[bot] commented on pull request #12608: [website][upgrade]feat: docs migration - 2.7.1 / deploy

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12608: URL: https://github.com/apache/pulsar/pull/12608#issuecomment-960355615 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] casuallc commented on issue #12601: Geo Replica with one producer and two "Shared" Consumers

2021-11-03 Thread GitBox
casuallc commented on issue #12601: URL: https://github.com/apache/pulsar/issues/12601#issuecomment-958764823 You should enable replicated subscription, but it is not in real time. https://pulsar.apache.org/docs/en/administration-geo/#replicated-subscriptions -- This is an

[GitHub] [pulsar] navid1981 commented on issue #12601: Geo Replica with one producer and two "Shared" Consumers

2021-11-03 Thread GitBox
navid1981 commented on issue #12601: URL: https://github.com/apache/pulsar/issues/12601#issuecomment-960136434 @casuallc I did but still receiving the same message in other cluster Consumer consumer = client2.newConsumer()

[GitHub] [pulsar] github-actions[bot] commented on pull request #12611: [website][upgrade]feat: docs migration - 2.7.1 / performance

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12611: URL: https://github.com/apache/pulsar/pull/12611#issuecomment-960356871 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[pulsar] branch master updated (b549620 -> 556ba0e)

2021-11-03 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 b549620 Remove unused zookeeper factory in function worker (#12605) add 556ba0e [Test] Cleanup

[GitHub] [pulsar] congbobo184 commented on a change in pull request #12449: [Transaction] Offload transaction data

2021-11-03 Thread GitBox
congbobo184 commented on a change in pull request #12449: URL: https://github.com/apache/pulsar/pull/12449#discussion_r741641947 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ## @@ -1848,8 +1851,10 @@ public void

[GitHub] [pulsar] urfreespace opened a new pull request #12611: Feature/website next 20211104 4

2021-11-03 Thread GitBox
urfreespace opened a new pull request #12611: URL: https://github.com/apache/pulsar/pull/12611 *(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] github-actions[bot] commented on pull request #12605: Remove unused zookeeper factory in function worker

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12605: URL: https://github.com/apache/pulsar/pull/12605#issuecomment-959459016 @Shoothzj: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] codelipenghui commented on pull request #12602: [Compaction] Do not move the non-durable cursor position when trimming ledgers while topic with compaction

2021-11-03 Thread GitBox
codelipenghui commented on pull request #12602: URL: https://github.com/apache/pulsar/pull/12602#issuecomment-958941238 /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] kuangye098 commented on issue #12269: Support pluggable entry filter in Dispatcher

2021-11-03 Thread GitBox
kuangye098 commented on issue #12269: URL: https://github.com/apache/pulsar/issues/12269#issuecomment-958676323 I recommend: 1. Filtering should not introduce the message body, which can avoid the problem of end-to-end encryption unable to decrypt, and can solve the possible JVM GC

[GitHub] [pulsar] yuruguo commented on pull request #12512: [pulsar-admin] Check backlog quota policy for namespace

2021-11-03 Thread GitBox
yuruguo commented on pull request #12512: URL: https://github.com/apache/pulsar/pull/12512#issuecomment-959017747 @eolivelli Please take a look again, thanks :) -- 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] merlimat merged pull request #12594: Reduced severity of log "refreshing key manager" in KeyManagerProxy

2021-11-03 Thread GitBox
merlimat merged pull request #12594: URL: https://github.com/apache/pulsar/pull/12594 -- 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] hepyu commented on issue #12533: bookie entry的p999写入时间达到5秒可能是什么原因?

2021-11-03 Thread GitBox
hepyu commented on issue #12533: URL: https://github.com/apache/pulsar/issues/12533#issuecomment-958945594 @Jason918 I can't remember clearly. Do I vaguely remember that bookie or broker has GC, 50ms ~ 100ms? But I'm not sure. Now the Prometheus Metris data is gone, and I can't verify

[GitHub] [pulsar] Anonymitaet merged pull request #12578: [website][upgrade]feat: docs migration - 2.7.1 / tiered-storage

2021-11-03 Thread GitBox
Anonymitaet merged pull request #12578: URL: https://github.com/apache/pulsar/pull/12578 -- 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 #12608: [website][upgrade]feat: docs migration - 2.7.1 / deploy

2021-11-03 Thread GitBox
urfreespace opened a new pull request #12608: URL: https://github.com/apache/pulsar/pull/12608 -- 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 commented on a change in pull request #12566: Add negative ack redelivery backoff.

2021-11-03 Thread GitBox
codelipenghui commented on a change in pull request #12566: URL: https://github.com/apache/pulsar/pull/12566#discussion_r741853084 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/NegativeAckRedeliveryExponentialBackoff.java ## @@ -0,0 +1,95 @@ +/** + *

[GitHub] [pulsar] Jason918 commented on issue #12533: bookie entry的p999写入时间达到5秒可能是什么原因?

2021-11-03 Thread GitBox
Jason918 commented on issue #12533: URL: https://github.com/apache/pulsar/issues/12533#issuecomment-959045668 > But will 50ms ~ 100ms GC cause bookie to write p999 for about 5 seconds? No, but a long STW event could. It's better to add "-XX:+PrintGCApplicationStoppedTime" in JVM

[GitHub] [pulsar] x-shadow-man commented on pull request #12567: doc optimization for issue #12488

2021-11-03 Thread GitBox
x-shadow-man commented on pull request #12567: URL: https://github.com/apache/pulsar/pull/12567#issuecomment-958930821 @frankxieke buddy, can u feel free to check the doc? thx. :-) -- 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 #12603: Remove unused code in PulsarAdminTool

2021-11-03 Thread GitBox
yuruguo commented on pull request #12603: URL: https://github.com/apache/pulsar/pull/12603#issuecomment-959009620 @codelipenghui @315157973 Please pay attention to this PR(https://github.com/apache/pulsar/pull/12581), there are differences between the two, remove or assign `lastExitCode`?

[GitHub] [pulsar] liangyepianzhou commented on a change in pull request #12449: [Transaction] Offload transaction data

2021-11-03 Thread GitBox
liangyepianzhou commented on a change in pull request #12449: URL: https://github.com/apache/pulsar/pull/12449#discussion_r741796368 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerImpl.java ## @@ -1848,8 +1851,10 @@ public void

[GitHub] [pulsar] github-actions[bot] commented on pull request #12600: [#12423] allow `GetTopicsOfNamespace` op with `consume` permission

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12600: URL: https://github.com/apache/pulsar/pull/12600#issuecomment-958656833 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] lhotari closed pull request #12606: [ML] Avoid passing OpAddEntry across a thread boundary in asyncAddEntry

2021-11-03 Thread GitBox
lhotari closed pull request #12606: URL: https://github.com/apache/pulsar/pull/12606 -- 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 closed pull request #12499: Fix collection get bug in ResourceGroupService

2021-11-03 Thread GitBox
Shoothzj closed pull request #12499: URL: https://github.com/apache/pulsar/pull/12499 -- 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] github-actions[bot] commented on pull request #12602: [Compaction] Do not move the non-durable cursor position when trimming ledgers while topic with compaction

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12602: URL: https://github.com/apache/pulsar/pull/12602#issuecomment-958862776 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] Anonymitaet commented on pull request #12566: Add negative ack redelivery backoff.

2021-11-03 Thread GitBox
Anonymitaet commented on pull request #12566: URL: https://github.com/apache/pulsar/pull/12566#issuecomment-958704726 Confirmed with @wolfstudy, he will provide tech inputs for doc -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] hangc0276 commented on a change in pull request #12366: expose broker bundles metrics to prometheus

2021-11-03 Thread GitBox
hangc0276 commented on a change in pull request #12366: URL: https://github.com/apache/pulsar/pull/12366#discussion_r742465671 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java ## @@ -916,6 +916,7 @@ public

[GitHub] [pulsar] github-actions[bot] commented on pull request #12606: [ML] Avoid passing OpAddEntry across a thread boundary in asyncAddEntry

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12606: URL: https://github.com/apache/pulsar/pull/12606#issuecomment-959770438 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] codelipenghui commented on a change in pull request #12523: add a metric to get earliest time in the backlog

2021-11-03 Thread GitBox
codelipenghui commented on a change in pull request #12523: URL: https://github.com/apache/pulsar/pull/12523#discussion_r742460181 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentSubscription.java ## @@ -1065,6 +1066,12 @@

[GitHub] [pulsar] github-actions[bot] commented on pull request #12604: OverloadShedder: Sort by throughput in ascending order

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12604: URL: https://github.com/apache/pulsar/pull/12604#issuecomment-959340614 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] merlimat merged pull request #12586: [C++] Fix request timeout for GetLastMessageId doesn't work

2021-11-03 Thread GitBox
merlimat merged pull request #12586: URL: https://github.com/apache/pulsar/pull/12586 -- 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 merged pull request #12386: [Transaction]Fix maxReadPosition with normal publish

2021-11-03 Thread GitBox
congbobo184 merged pull request #12386: URL: https://github.com/apache/pulsar/pull/12386 -- 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 #12605: Remove unused zookeeper factory in function worker

2021-11-03 Thread GitBox
merlimat merged pull request #12605: URL: https://github.com/apache/pulsar/pull/12605 -- 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] chlyzzo edited a comment on issue #12596: python read AVRO topic has error

2021-11-03 Thread GitBox
chlyzzo edited a comment on issue #12596: URL: https://github.com/apache/pulsar/issues/12596#issuecomment-95872 test send the message: in scala: val user = new Person("scala1", 689) producer.send(user) in python: producer.send(Person(name="scala1", age=689) and

[GitHub] [pulsar] merlimat commented on a change in pull request #12536: [pulsar-broker] Broker extensions to allow operators of enterprise wide cluster better control and flexibility

2021-11-03 Thread GitBox
merlimat commented on a change in pull request #12536: URL: https://github.com/apache/pulsar/pull/12536#discussion_r742145102 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedgerConfig.java ## @@ -77,6 +77,7 @@ private int

[GitHub] [pulsar] codelipenghui merged pull request #12597: [pulsar-broker] Cleanup already deleted namespace topics.

2021-11-03 Thread GitBox
codelipenghui merged pull request #12597: URL: https://github.com/apache/pulsar/pull/12597 -- 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] chlyzzo commented on issue #12596: python read AVRO topic has error

2021-11-03 Thread GitBox
chlyzzo commented on issue #12596: URL: https://github.com/apache/pulsar/issues/12596#issuecomment-95872 test send the message: in scala: val user = new Person("werewr", 35) producer.send(user) in python: producer.send(Person(name="wtr", age=23) and python

[GitHub] [pulsar-client-go] cckellogg commented on a change in pull request #651: fix issue 650,different handle sequence value

2021-11-03 Thread GitBox
cckellogg commented on a change in pull request #651: URL: https://github.com/apache/pulsar-client-go/pull/651#discussion_r742266590 ## File path: pulsar/producer_partition.go ## @@ -784,57 +784,57 @@ func (p *partitionProducer) ReceivedSendReceipt(response

[GitHub] [pulsar] github-actions[bot] commented on pull request #12610: [website][upgrade]feat: docs migration - 2.7.1 / security

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12610: URL: https://github.com/apache/pulsar/pull/12610#issuecomment-960356350 @urfreespace: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 #12612: [website][upgrade]feat: docs migration - 2.7.1 / client

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12612: URL: https://github.com/apache/pulsar/pull/12612#issuecomment-960358203 @urfreespace: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] urfreespace opened a new pull request #12610: [website][upgrade]feat: docs migration - 2.7.1 / security

2021-11-03 Thread GitBox
urfreespace opened a new pull request #12610: URL: https://github.com/apache/pulsar/pull/12610 *(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] merlimat merged pull request #12593: [Python] Provide __str__ operator for BytesSchema

2021-11-03 Thread GitBox
merlimat merged pull request #12593: URL: https://github.com/apache/pulsar/pull/12593 -- 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 #12392: [pulsar-admin] Add remove-subscription-types-enabled command for namespace

2021-11-03 Thread GitBox
yuruguo removed a comment on pull request #12392: URL: https://github.com/apache/pulsar/pull/12392#issuecomment-954038741 @eolivelli @codelipenghui 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

[GitHub] [pulsar] Shoothzj closed pull request #12603: Remove unused code in PulsarAdminTool

2021-11-03 Thread GitBox
Shoothzj closed pull request #12603: URL: https://github.com/apache/pulsar/pull/12603 -- 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 #12512: [pulsar-admin] Check backlog quota policy for namespace

2021-11-03 Thread GitBox
yuruguo removed a comment on pull request #12512: URL: https://github.com/apache/pulsar/pull/12512#issuecomment-957102513 Please take a look again, thanks :) @eolivelli -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] yuruguo commented on pull request #12535: [pulsar-admin] Add --all option to get all version schema of topic

2021-11-03 Thread GitBox
yuruguo commented on pull request #12535: URL: https://github.com/apache/pulsar/pull/12535#issuecomment-958948175 -- 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] Shoothzj commented on pull request #12603: Remove unused code in PulsarAdminTool

2021-11-03 Thread GitBox
Shoothzj commented on pull request #12603: URL: https://github.com/apache/pulsar/pull/12603#issuecomment-959414353 > @codelipenghui @315157973 Please pay attention to this PR(#12581), there are differences between the two, remove or assign `lastExitCode`? wow, #12581 is better --

[GitHub] [pulsar] codelipenghui commented on pull request #12536: [pulsar-broker] Broker extensions to allow operators of enterprise wide cluster better control and flexibility

2021-11-03 Thread GitBox
codelipenghui commented on pull request #12536: URL: https://github.com/apache/pulsar/pull/12536#issuecomment-959107261 @madhavan-narayanan It should be good to start with a proposal for this new feature, I noticed there are new API introduced. For starting a proposal for Pulsar, you can

[GitHub] [pulsar] github-actions[bot] commented on pull request #12599: Enable custom Encrypt/Decrypt Methods for ReaderImplementation.

2021-11-03 Thread GitBox
github-actions[bot] commented on pull request #12599: URL: https://github.com/apache/pulsar/pull/12599#issuecomment-958655472 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] urfreespace opened a new pull request #12609: Feature/website next 20211104 2

2021-11-03 Thread GitBox
urfreespace opened a new pull request #12609: URL: https://github.com/apache/pulsar/pull/12609 -- 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] Technoboy- commented on pull request #12597: [pulsar-broker] Cleanup already deleted namespace topics.

2021-11-03 Thread GitBox
Technoboy- commented on pull request #12597: URL: https://github.com/apache/pulsar/pull/12597#issuecomment-958797072 /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] github-actions[bot] commented on pull request #12597: [pulsar-broker] Cleanup already deleted namespace topics.

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

  1   2   3   4   >