[GitHub] [pulsar] BewareMyPower edited a comment on issue #509: C++ Client - Producer is never destructed

2022-03-21 Thread GitBox
BewareMyPower edited a comment on issue #509: URL: https://github.com/apache/pulsar/issues/509#issuecomment-1074741277 Here is the reproduce code. ```c++ #include #include #include using namespace pulsar; int main() { ClientConfiguration conf;

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #14766: Mark CGo client as deprecated in the documentation

2022-03-21 Thread GitBox
Anonymitaet commented on a change in pull request #14766: URL: https://github.com/apache/pulsar/pull/14766#discussion_r831791621 ## File path: site2/docs/client-libraries-cgo.md ## @@ -4,6 +4,8 @@ title: Pulsar CGo client sidebar_label: CGo(deprecated) --- +> The CGo

[GitHub] [pulsar] Anonymitaet commented on pull request #14754: Version docs for the 2.8.3 release

2022-03-21 Thread GitBox
Anonymitaet commented on pull request #14754: URL: https://github.com/apache/pulsar/pull/14754#issuecomment-1074757344 > It appears that the build is failing due to a timeout. @Anonymitaet - do you know why we're building the python client during the website build? @urfreespace do

[GitHub] [pulsar] BewareMyPower edited a comment on issue #509: C++ Client - Producer is never destructed

2022-03-21 Thread GitBox
BewareMyPower edited a comment on issue #509: URL: https://github.com/apache/pulsar/issues/509#issuecomment-1074741277 It's because even if `Producer` gone out of the scope,`Client` holds a reference to `ProducerImpl`. Here is the reproduce code. ```c++ #include

[GitHub] [pulsar] Technoboy- commented on pull request #14747: [cleanup][broker]: Refactor PulsarAuthorizationProvider.

2022-03-21 Thread GitBox
Technoboy- commented on pull request #14747: URL: https://github.com/apache/pulsar/pull/14747#issuecomment-1074744048 > @Technoboy- I notice that no tests to cover the PulsarAuthorizationProvider, could you add some tests for this? Integration test has coverage this. -- This is

[GitHub] [pulsar] BewareMyPower commented on issue #509: C++ Client - Producer is never destructed

2022-03-21 Thread GitBox
BewareMyPower commented on issue #509: URL: https://github.com/apache/pulsar/issues/509#issuecomment-1074741277 It's because even if `Producer` gone out of the scope,`Client` holds a reference to `ProducerImpl`. Here is the reproduce code. ```c++ #include #include

[GitHub] [pulsar] Technoboy- commented on a change in pull request #14730: [fix][broker]: Fix cannot delete namespace with system topic

2022-03-21 Thread GitBox
Technoboy- commented on a change in pull request #14730: URL: https://github.com/apache/pulsar/pull/14730#discussion_r831781274 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java ## @@ -791,6 +791,11 @@ protected void

[GitHub] [pulsar] Technoboy- commented on a change in pull request #14730: [fix][broker]: Fix cannot delete namespace with system topic

2022-03-21 Thread GitBox
Technoboy- commented on a change in pull request #14730: URL: https://github.com/apache/pulsar/pull/14730#discussion_r831780114 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java ## @@ -791,6 +791,11 @@ protected void

[GitHub] [pulsar] nodece commented on pull request #14747: [cleanup][broker]: Refactor PulsarAuthorizationProvider.

2022-03-21 Thread GitBox
nodece commented on pull request #14747: URL: https://github.com/apache/pulsar/pull/14747#issuecomment-1074732511 @Technoboy- I notice that no tests to cover the PulsarAuthorizationProvider, could you add some tests for this? -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar] codelipenghui commented on pull request #14542: [PIP-146] ManagedCursorInfo compression

2022-03-21 Thread GitBox
codelipenghui commented on pull request #14542: URL: https://github.com/apache/pulsar/pull/14542#issuecomment-1074708319 @eolivelli Please help review this PR again. -- 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 #745: Exposing broker metadata

2022-03-21 Thread GitBox
wolfstudy merged pull request #745: URL: https://github.com/apache/pulsar-client-go/pull/745 -- 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: [PIP 90] go client retrieve broker metadata (#745)

2022-03-21 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new b58f115 [PIP 90] go client retrieve

[GitHub] [pulsar-client-go] wolfstudy merged pull request #751: [build] make go version consistent

2022-03-21 Thread GitBox
wolfstudy merged pull request #751: URL: https://github.com/apache/pulsar-client-go/pull/751 -- 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: [build] make go version consistent (#751)

2022-03-21 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 95fdb3f [build] make go version

[pulsar] branch master updated (a1037c7 -> 46886b0)

2022-03-21 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 a1037c7 [refactor][proxy] Refactor Proxy code and fix connection stalling by switching to auto read mode (#14713)

[GitHub] [pulsar] codelipenghui closed issue #14771: Flaky-test: ProducerTest. multipleMembers

2022-03-21 Thread GitBox
codelipenghui closed issue #14771: URL: https://github.com/apache/pulsar/issues/14771 -- 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 merged pull request #14776: [fix][test] Fix flaky test LeaderElectionTest

2022-03-21 Thread GitBox
codelipenghui merged pull request #14776: URL: https://github.com/apache/pulsar/pull/14776 -- 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 #14730: [fix][broker]: Fix cannot delete namespace with system topic

2022-03-21 Thread GitBox
codelipenghui commented on a change in pull request #14730: URL: https://github.com/apache/pulsar/pull/14730#discussion_r831750436 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java ## @@ -791,6 +791,11 @@ protected void

[GitHub] [pulsar] michaeljmarshall commented on pull request #14754: Version docs for the 2.8.3 release

2022-03-21 Thread GitBox
michaeljmarshall commented on pull request #14754: URL: https://github.com/apache/pulsar/pull/14754#issuecomment-1074700096 It appears that the build is failing due to a timeout. @Anonymitaet - do you know why we're building the python client during the website build? -- This is an

[pulsar] branch master updated (e60e669 -> a1037c7)

2022-03-21 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 e60e669 Version docs for the 2.8.3 release (#14754) add a1037c7 [refactor][proxy] Refactor Proxy code and

[GitHub] [pulsar] michaeljmarshall opened a new issue #14783: Clean up Javadoc Warnings

2022-03-21 Thread GitBox
michaeljmarshall opened a new issue #14783: URL: https://github.com/apache/pulsar/issues/14783 When Pulsar builds in CI, I can see the below warnings printed out (https://github.com/apache/pulsar/runs/5622671591?check_suite_focus=true): ``` 28 warnings Warning: Javadoc

[GitHub] [pulsar] codelipenghui merged pull request #14713: [Proxy] Refactor Proxy code and fix connection stalling by switching to auto read mode

2022-03-21 Thread GitBox
codelipenghui merged pull request #14713: URL: https://github.com/apache/pulsar/pull/14713 -- 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 pull request #14702: [PulsarAdmin] add get active brokers api without cluster name

2022-03-21 Thread GitBox
codelipenghui commented on pull request #14702: URL: https://github.com/apache/pulsar/pull/14702#issuecomment-1074698379 @Technoboy- Please help 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] codelipenghui commented on pull request #14656: [fix][broker]Fix NPE cause by duplicated removeSubscription

2022-03-21 Thread GitBox
codelipenghui commented on pull request #14656: URL: https://github.com/apache/pulsar/pull/14656#issuecomment-1074697911 > The rack condition is already fixes by https://github.com/apache/pulsar/issues/14362. I think. @mattisonchao @gaozhangmin If the race condition has been fixed,

[GitHub] [pulsar] michaeljmarshall commented on pull request #14754: Version docs for the 2.8.3 release

2022-03-21 Thread GitBox
michaeljmarshall commented on pull request #14754: URL: https://github.com/apache/pulsar/pull/14754#issuecomment-1074695868 @Anonymitaet - it looks like the failures started before I merged this. I can look at the logs to understand it more. -- This is an automated message from the

[GitHub] [pulsar] Jason918 commented on a change in pull request #14730: [fix][broker]: Fix cannot delete namespace with system topic

2022-03-21 Thread GitBox
Jason918 commented on a change in pull request #14730: URL: https://github.com/apache/pulsar/pull/14730#discussion_r831733540 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/web/PulsarWebResource.java ## @@ -791,6 +791,11 @@ protected void

[GitHub] [pulsar] ZiyaoWei commented on pull request #14766: Mark CGo client as deprecated in the documentation

2022-03-21 Thread GitBox
ZiyaoWei commented on pull request #14766: URL: https://github.com/apache/pulsar/pull/14766#issuecomment-1074679852 @Anonymitaet - done, thanks! Added a banner in the CGo client docs saying it has been deprecated. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] codelipenghui commented on a change in pull request #14781: [fix][txn]: fix pending ack is recovering throw CursorAlreadyClosedxception

2022-03-21 Thread GitBox
codelipenghui commented on a change in pull request #14781: URL: https://github.com/apache/pulsar/pull/14781#discussion_r831730765 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/transaction/pendingack/impl/MLPendingAckStore.java ## @@ -398,7 +398,7 @@

[GitHub] [pulsar] codelipenghui commented on issue #14779: BadVersionException , unable to consume topic, failing to reconnect

2022-03-21 Thread GitBox
codelipenghui commented on issue #14779: URL: https://github.com/apache/pulsar/issues/14779#issuecomment-1074673985 Hi @cerebrotecnologico thanks for creating the issue, is it able to share the broker logs file? I want to check what happens before the error logs, this will help to

[GitHub] [pulsar] nodece commented on issue #14762: Client must configure AuthenticationTls even if Authentication using TLS is not enabled

2022-03-21 Thread GitBox
nodece commented on issue #14762: URL: https://github.com/apache/pulsar/issues/14762#issuecomment-1074672815 We should split the transport and auth for TLS. For transport, the code can be so like: ``` PulsarClient client = PulsarClient.builder()

[GitHub] [pulsar] momo-jun commented on a change in pull request #14746: [improve][doc] Describe message ID and enhance the description of sequence ID

2022-03-21 Thread GitBox
momo-jun commented on a change in pull request #14746: URL: https://github.com/apache/pulsar/pull/14746#discussion_r831723894 ## File path: site2/docs/concepts-messaging.md ## @@ -20,11 +20,12 @@ Value / data payload | The data carried by the message. All Pulsar messages cont

[GitHub] [pulsar] Anonymitaet commented on pull request #14729: [feat][Java client]: Support Reader Interceptor

2022-03-21 Thread GitBox
Anonymitaet commented on pull request #14729: URL: https://github.com/apache/pulsar/pull/14729#issuecomment-1074667936 @momo-jun a soft reminder: this PR is merged, doc work can be started, thanks. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #14773: [feature][doc][Do-not-merge] Add doc for how to use MultiRolesTokenAuthorizationProvider

2022-03-21 Thread GitBox
Anonymitaet commented on a change in pull request #14773: URL: https://github.com/apache/pulsar/pull/14773#discussion_r831719184 ## File path: site2/docs/security-authorization.md ## @@ -98,3 +98,15 @@ PulsarAdmin admin = PulsarAdmin.builder()

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #14746: [improve][doc] Describe message ID and enhance the description of sequence ID

2022-03-21 Thread GitBox
Anonymitaet commented on a change in pull request #14746: URL: https://github.com/apache/pulsar/pull/14746#discussion_r831718472 ## File path: site2/docs/concepts-messaging.md ## @@ -20,11 +20,12 @@ Value / data payload | The data carried by the message. All Pulsar messages

[GitHub] [pulsar] Jason918 opened a new pull request #14782: [enh][broker] Support dynamic update between non-zero values of topicPublisherThrottlingTickTimeMillis

2022-03-21 Thread GitBox
Jason918 opened a new pull request #14782: URL: https://github.com/apache/pulsar/pull/14782 ### Motivation Apply the change of #14759 to `topicPublisherThrottlingTickTimeMillis`. The config of `topicPublisherThrottlingTickTimeMillis` is marked as dynamic, does not support

[GitHub] [pulsar] Anonymitaet commented on pull request #14754: Version docs for the 2.8.3 release

2022-03-21 Thread GitBox
Anonymitaet commented on pull request #14754: URL: https://github.com/apache/pulsar/pull/14754#issuecomment-1074660372 2.8.3 doc is not shown https://user-images.githubusercontent.com/50226895/159395785-23591d51-40c9-4ba3-b334-15d30fe38943.png;> Pulsar website build is down

[GitHub] [pulsar] gaozhangmin removed a comment on pull request #14776: [fix][test] Fix flaky test LeaderElectionTest

2022-03-21 Thread GitBox
gaozhangmin removed a comment on pull request #14776: URL: https://github.com/apache/pulsar/pull/14776#issuecomment-1074018165 /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

[GitHub] [pulsar] congbobo184 opened a new pull request #14781: [fix][txn]: fix pending ack is recovering throw CursorAlreadyClosedxception

2022-03-21 Thread GitBox
congbobo184 opened a new pull request #14781: URL: https://github.com/apache/pulsar/pull/14781 ### Motivation When Transaction PendingAck recover fail throw CursorAlreadyClosedException, we should stop the recover op. the cursor was been closed, the pendingAck was been closed, so we

[GitHub] [pulsar] nodece commented on pull request #14758: [fix][broker] Fix splitNamespaceBundle api request param

2022-03-21 Thread GitBox
nodece commented on pull request #14758: URL: https://github.com/apache/pulsar/pull/14758#issuecomment-1074650754 /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] Technoboy- commented on issue #14079: Inactive topics cannot be consumed if recreated after automatic deletion

2022-03-21 Thread GitBox
Technoboy- commented on issue #14079: URL: https://github.com/apache/pulsar/issues/14079#issuecomment-1074631964 > Hi @mattisonchao & @Technoboy- , unfortunately I work in a secure env where wouldn't be able to share the logs. This problem is reproducible both in our real environments and

[GitHub] [pulsar] github-actions[bot] commented on issue #14223: sometimes msgbacklog size is not correct when restart

2022-03-21 Thread GitBox
github-actions[bot] commented on issue #14223: URL: https://github.com/apache/pulsar/issues/14223#issuecomment-1074628799 The issue had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] Demogorgon314 commented on a change in pull request #14778: [fix][test] Fix wrong retry behavior in MetadataCacheTest

2022-03-21 Thread GitBox
Demogorgon314 commented on a change in pull request #14778: URL: https://github.com/apache/pulsar/pull/14778#discussion_r831699670 ## File path: pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataCacheTest.java ## @@ -611,7 +611,7 @@ public static void

[GitHub] [pulsar] Demogorgon314 commented on pull request #14776: [fix][test] Fix flaky test LeaderElectionTest

2022-03-21 Thread GitBox
Demogorgon314 commented on pull request #14776: URL: https://github.com/apache/pulsar/pull/14776#issuecomment-1074603041 Please rebase onto the master branch to resolve the compile issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] Technoboy- commented on a change in pull request #14747: [cleanup][broker]: Refactor PulsarAuthorizationProvider.

2022-03-21 Thread GitBox
Technoboy- commented on a change in pull request #14747: URL: https://github.com/apache/pulsar/pull/14747#discussion_r831693036 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java ## @@ -106,63 +104,43 @@

[GitHub] [pulsar] Technoboy- commented on a change in pull request #14747: [cleanup][broker]: Refactor PulsarAuthorizationProvider.

2022-03-21 Thread GitBox
Technoboy- commented on a change in pull request #14747: URL: https://github.com/apache/pulsar/pull/14747#discussion_r831692970 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java ## @@ -178,36 +156,13 @@

svn commit: r53262 [1/2] - in /dev/pulsar/pulsar-2.10.0-candidate-4: ./ DEB/ RPMS/ RPMS/repodata/ connectors/

2022-03-21 Thread penghui
Author: penghui Date: Tue Mar 22 00:54:05 2022 New Revision: 53262 Log: Staging artifacts and signature for Pulsar release 2.10.0 RC4 Added: dev/pulsar/pulsar-2.10.0-candidate-4/ dev/pulsar/pulsar-2.10.0-candidate-4/DEB/ dev/pulsar/pulsar-2.10.0-candidate-4/DEB/Packages.gz (with

svn commit: r53262 [2/2] - in /dev/pulsar/pulsar-2.10.0-candidate-4: ./ DEB/ RPMS/ RPMS/repodata/ connectors/

2022-03-21 Thread penghui
Added: dev/pulsar/pulsar-2.10.0-candidate-4/connectors/pulsar-io-debezium-mysql-2.10.0.nar.asc == --- dev/pulsar/pulsar-2.10.0-candidate-4/connectors/pulsar-io-debezium-mysql-2.10.0.nar.asc (added) +++

[GitHub] [pulsar-client-go] Shoothzj commented on pull request #741: Add schema support to Reader

2022-03-21 Thread GitBox
Shoothzj commented on pull request #741: URL: https://github.com/apache/pulsar-client-go/pull/741#issuecomment-1074555039 @ZiyaoWei #748 is fixed, you can rebase the code for retrigger the ci -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [pulsar-client-go] Shoothzj opened a new pull request #751: [build] make go version consistent

2022-03-21 Thread GitBox
Shoothzj opened a new pull request #751: URL: https://github.com/apache/pulsar-client-go/pull/751 ### Motivation Make dockerfile go version consistent with the `go.mod` file and `README doc` ### Modifications - update go version from 1.13 to 1.15 -- This is an automated

[pulsar-client-go] branch go-version-consistent created (now 1a088b0)

2022-03-21 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a change to branch go-version-consistent in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git. at 1a088b0 [build] make go version consistent This branch includes the following new

[pulsar-client-go] 01/01: [build] make go version consistent

2022-03-21 Thread shoothzj
This is an automated email from the ASF dual-hosted git repository. shoothzj pushed a commit to branch go-version-consistent in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git commit 1a088b0a378bd96426301f4b6fffd7d5af1830ed Author: ZhangJian He AuthorDate: Tue Mar 22

[GitHub] [pulsar-client-go] merlimat closed issue #748: `pulsar-client-go` test cannot pass on apachepulsar/pulsar:latest

2022-03-21 Thread GitBox
merlimat closed issue #748: URL: https://github.com/apache/pulsar-client-go/issues/748 -- 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] merlimat merged pull request #747: Temporarily point ci to pulsar 2.8.2

2022-03-21 Thread GitBox
merlimat merged pull request #747: URL: https://github.com/apache/pulsar-client-go/pull/747 -- 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: Temporarily point ci to pulsar 2.8.2 (#747)

2022-03-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 5c04811 Temporarily point ci to

[GitHub] [pulsar] dlg99 opened a new pull request #14780: Handle kafka sinks that return immutable maps as configs

2022-03-21 Thread GitBox
dlg99 opened a new pull request #14780: URL: https://github.com/apache/pulsar/pull/14780 ### Motivation Some Kafka sinks return immutable map as config (e.g. ones written in Scala) As result KCA sink fail to start/throws exception on attempt to modify such config. ###

[GitHub] [pulsar-client-go] Shoothzj closed pull request #746: [DONOT MERGE] trigger ci use pulsar 2.9.0

2022-03-21 Thread GitBox
Shoothzj closed pull request #746: URL: https://github.com/apache/pulsar-client-go/pull/746 -- 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] cerebrotecnologico opened a new issue #14779: BadVersionException , unable to consume topic, failing to reconnect

2022-03-21 Thread GitBox
cerebrotecnologico opened a new issue #14779: URL: https://github.com/apache/pulsar/issues/14779 **Describe the bug** I am using Pulsar 2.8.0. The topic is partitioned, 3 partitions. The backlog quota is set to 50h The TTL is set to 48h. This is the second time that we

[GitHub] [pulsar-client-go] merlimat commented on issue #748: `pulsar-client-go` test cannot pass on apachepulsar/pulsar:latest

2022-03-21 Thread GitBox
merlimat commented on issue #748: URL: https://github.com/apache/pulsar-client-go/issues/748#issuecomment-1074371439  Makes sense to me -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [pulsar-client-go] ZiyaoWei commented on issue #748: `pulsar-client-go` test cannot pass on apachepulsar/pulsar:latest

2022-03-21 Thread GitBox
ZiyaoWei commented on issue #748: URL: https://github.com/apache/pulsar-client-go/issues/748#issuecomment-1074362257 Actually the tests pass under `lph890127/pulsar:2.10.0-rc4`, so maybe we should temporarily point to `2.8.2` until 2.10 is out? -- This is an automated message from the

[pulsar] branch branch-2.8 updated: [fix][broker] Fixed duplicated delayed messages when all consumers disconnect (#14740)

2022-03-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.8 by this push: new db3ad55 [fix][broker] Fixed duplicated

[GitHub] [pulsar] eolivelli commented on issue #14772: could we add some hook method to allow users set header in pulsar-admin-client

2022-03-21 Thread GitBox
eolivelli commented on issue #14772: URL: https://github.com/apache/pulsar/issues/14772#issuecomment-1074340125 Maybe it may help for some proxy/firewall? -- 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

[pulsar] branch branch-2.9 updated: [fix][broker] Fixed duplicated delayed messages when all consumers disconnect (#14740)

2022-03-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-2.9 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.9 by this push: new 2624a04 [fix][broker] Fixed duplicated

[pulsar] branch branch-2.10 updated: [fix][broker] Fixed duplicated delayed messages when all consumers disconnect (#14740)

2022-03-21 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-2.10 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.10 by this push: new 5428a22 [fix][broker] Fixed

[GitHub] [pulsar] lhotari commented on issue #14401: Speed up CI workflows

2022-03-21 Thread GitBox
lhotari commented on issue #14401: URL: https://github.com/apache/pulsar/issues/14401#issuecomment-1074331619 next attempt after a few iterations: https://github.com/lhotari/pulsar/actions/runs/2018290879 -- This is an automated message from the Apache Git Service. To respond to the

[pulsar] branch master updated (f890017 -> e60e669)

2022-03-21 Thread mmarshall
This is an automated email from the ASF dual-hosted git repository. mmarshall pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from f890017 [fix][test] Fix wrong retry behavior in MetadataCacheTest (#14778) add e60e669 Version docs for

[GitHub] [pulsar] michaeljmarshall commented on pull request #14632: 2.8.3 Release Notes

2022-03-21 Thread GitBox
michaeljmarshall commented on pull request #14632: URL: https://github.com/apache/pulsar/pull/14632#issuecomment-1074238026 > @michaeljmarshall do not forget to write 2.8.3 tech blog https://pulsar.apache.org/blog/ @Anonymitaet - I won't be able to complete a blog for the release.

[GitHub] [pulsar] michaeljmarshall merged pull request #14754: Version docs for the 2.8.3 release

2022-03-21 Thread GitBox
michaeljmarshall merged pull request #14754: URL: https://github.com/apache/pulsar/pull/14754 -- 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 a change in pull request #14632: 2.8.3 Release Notes

2022-03-21 Thread GitBox
michaeljmarshall commented on a change in pull request #14632: URL: https://github.com/apache/pulsar/pull/14632#discussion_r831375233 ## File path: site2/website/release-notes.md ## @@ -1,6 +1,207 @@ ## Apache Pulsar Release Notes +### 2.8.2 + + 2022-03-10 + +###

[GitHub] [pulsar] eolivelli commented on pull request #14754: Version docs for the 2.8.3 release

2022-03-21 Thread GitBox
eolivelli commented on pull request #14754: URL: https://github.com/apache/pulsar/pull/14754#issuecomment-1074196911 @Anonymitaet This is the last step. Officially 2.8.3 is released, the VOTE is closed. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] michaeljmarshall commented on pull request #14754: Version docs for the 2.8.3 release

2022-03-21 Thread GitBox
michaeljmarshall commented on pull request #14754: URL: https://github.com/apache/pulsar/pull/14754#issuecomment-1074185523 @Anonymitaet - can you clarify your statement? When do we consider 2.8.3 released? I would have thought it was already considered "released", as the binaries and the

[GitHub] [pulsar] batilak commented on issue #13267: Pulsar CPP client mem leak

2022-03-21 Thread GitBox
batilak commented on issue #13267: URL: https://github.com/apache/pulsar/issues/13267#issuecomment-1074184173 Any progress / solution / work-around for this? -- 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] lhotari commented on a change in pull request #14713: [Proxy] Refactor Proxy code and fix connection stalling by switching to auto read mode

2022-03-21 Thread GitBox
lhotari commented on a change in pull request #14713: URL: https://github.com/apache/pulsar/pull/14713#discussion_r831319213 ## File path: pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/ProxyConnection.java ## @@ -458,25 +459,6 @@ protected void

[pulsar] branch master updated (2b18fd3 -> f890017)

2022-03-21 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 2b18fd3 [fix][broker] Fixed duplicated delayed messages when all consumers disconnect (#14740) add f890017

[pulsar] branch master updated (1e619ce -> 2b18fd3)

2022-03-21 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 1e619ce Set function channel to idle to prevent DNS resolution of deleted pod (#14750) add 2b18fd3

[pulsar] branch master updated (c97e3f1 -> 1e619ce)

2022-03-21 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 c97e3f1 [enh][broker] Support dynamic update between non-zero values of brokerPublisherThrottlingTickTimeMillis

[GitHub] [pulsar] codelipenghui merged pull request #14778: [fix][test] Fix wrong retry behavior in MetadataCacheTest

2022-03-21 Thread GitBox
codelipenghui merged pull request #14778: URL: https://github.com/apache/pulsar/pull/14778 -- 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 (44f92a8 -> c97e3f1)

2022-03-21 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 44f92a8 [fix][admin-cli]: Remove the trust certs check (#14764) add c97e3f1 [enh][broker] Support dynamic

[GitHub] [pulsar] michaeljmarshall commented on a change in pull request #14713: [Proxy] Refactor Proxy code and fix connection stalling by switching to auto read mode

2022-03-21 Thread GitBox
michaeljmarshall commented on a change in pull request #14713: URL: https://github.com/apache/pulsar/pull/14713#discussion_r831284455 ## File path: pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/ProxyConnection.java ## @@ -458,25 +459,6 @@ protected void

[GitHub] [pulsar] codelipenghui merged pull request #14740: [fix][broker]: fixed duplicated delayed messages when all consumers disconnect

2022-03-21 Thread GitBox
codelipenghui merged pull request #14740: URL: https://github.com/apache/pulsar/pull/14740 -- 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 (b17f86c -> 44f92a8)

2022-03-21 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 b17f86c [Broker] Fix compilation error related to SystemTopicClient (#14774) add 44f92a8 [fix][admin-cli]:

[GitHub] [pulsar] codelipenghui merged pull request #14750: Set function channel to idle to prevent DNS resolution of deleted pod

2022-03-21 Thread GitBox
codelipenghui merged pull request #14750: URL: https://github.com/apache/pulsar/pull/14750 -- 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 merged pull request #14759: [enh][broker] Support dynamic update between non-zero values of brokerPublisherThrottlingTickTimeMillis

2022-03-21 Thread GitBox
codelipenghui merged pull request #14759: URL: https://github.com/apache/pulsar/pull/14759 -- 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 merged pull request #14764: [fix][admin-cli] Remove the trust certs check

2022-03-21 Thread GitBox
codelipenghui merged pull request #14764: URL: https://github.com/apache/pulsar/pull/14764 -- 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 #13975: [managed-ledger] Fix when nextValidLedger is null caused NPE

2022-03-21 Thread GitBox
codelipenghui commented on a change in pull request #13975: URL: https://github.com/apache/pulsar/pull/13975#discussion_r831280068 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedCursorImpl.java ## @@ -1697,7 +1697,8 @@ public void

[GitHub] [pulsar] BewareMyPower commented on a change in pull request #14604: [C++] add chunk message id

2022-03-21 Thread GitBox
BewareMyPower commented on a change in pull request #14604: URL: https://github.com/apache/pulsar/pull/14604#discussion_r831278341 ## File path: pulsar-client-cpp/lib/ConsumerImpl.cc ## @@ -1229,7 +1244,17 @@ void ConsumerImpl::seekAsync(const MessageId& msgId, ResultCallback

[GitHub] [pulsar] A-Wanderer commented on a change in pull request #14604: [C++] add chunk message id

2022-03-21 Thread GitBox
A-Wanderer commented on a change in pull request #14604: URL: https://github.com/apache/pulsar/pull/14604#discussion_r831272247 ## File path: pulsar-client-cpp/lib/ChunkMessageIdImpl.h ## @@ -0,0 +1,57 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one + *

[GitHub] [pulsar] BewareMyPower commented on a change in pull request #14604: [C++] add chunk message id

2022-03-21 Thread GitBox
BewareMyPower commented on a change in pull request #14604: URL: https://github.com/apache/pulsar/pull/14604#discussion_r831247228 ## File path: pulsar-client-cpp/include/pulsar/MessageId.h ## @@ -101,9 +106,15 @@ class PULSAR_PUBLIC MessageId { friend class

[GitHub] [pulsar] lhotari commented on issue #14401: Speed up CI workflows

2022-03-21 Thread GitBox
lhotari commented on issue #14401: URL: https://github.com/apache/pulsar/issues/14401#issuecomment-1074059328 latest run: https://github.com/lhotari/pulsar/actions/runs/2016939054 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] lhotari commented on issue #14401: Speed up CI workflows

2022-03-21 Thread GitBox
lhotari commented on issue #14401: URL: https://github.com/apache/pulsar/issues/14401#issuecomment-1074035688 Here's a build where most jobs pass: https://github.com/lhotari/pulsar/actions/runs/2015721352 . I've been iterating on https://github.com/lhotari/pulsar/pull/59 to resolve the

[GitHub] [pulsar] gaozhangmin commented on pull request #14776: [fix][test] Fix flaky test LeaderElectionTest

2022-03-21 Thread GitBox
gaozhangmin commented on pull request #14776: URL: https://github.com/apache/pulsar/pull/14776#issuecomment-1074018165 /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] Demogorgon314 commented on pull request #14778: [fix][test] Fix wrong retry behavior in MetadataCacheTest

2022-03-21 Thread GitBox
Demogorgon314 commented on pull request #14778: URL: https://github.com/apache/pulsar/pull/14778#issuecomment-1074006054 /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] frankjkelly commented on issue #14329: PIP-142 Increase default numHttpServerThreads value to 200 to prevent Admin API unavailability

2022-03-21 Thread GitBox
frankjkelly commented on issue #14329: URL: https://github.com/apache/pulsar/issues/14329#issuecomment-1073983064 Bump -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [pulsar] nodece removed a comment on pull request #14758: [fix][broker] Fix splitNamespaceBundle api request param

2022-03-21 Thread GitBox
nodece removed a comment on pull request #14758: URL: https://github.com/apache/pulsar/pull/14758#issuecomment-1073947145 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[GitHub] [pulsar] nodece commented on pull request #14758: [fix][broker] Fix splitNamespaceBundle api request param

2022-03-21 Thread GitBox
nodece commented on pull request #14758: URL: https://github.com/apache/pulsar/pull/14758#issuecomment-1073950467 /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] nodece commented on pull request #14758: [fix][broker] Fix splitNamespaceBundle api request param

2022-03-21 Thread GitBox
nodece commented on pull request #14758: URL: https://github.com/apache/pulsar/pull/14758#issuecomment-1073950079 /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 #14758: [fix][broker] Fix splitNamespaceBundle api request param

2022-03-21 Thread GitBox
nodece commented on pull request #14758: URL: https://github.com/apache/pulsar/pull/14758#issuecomment-1073947145 /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] codelipenghui commented on a change in pull request #14747: [cleanup][broker]: Refactor PulsarAuthorizationProvider.

2022-03-21 Thread GitBox
codelipenghui commented on a change in pull request #14747: URL: https://github.com/apache/pulsar/pull/14747#discussion_r831148981 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authorization/PulsarAuthorizationProvider.java ## @@ -178,36 +156,13 @@

[GitHub] [pulsar] gaozhangmin commented on pull request #14776: [fix][test] Fix flaky test LeaderElectionTest

2022-03-21 Thread GitBox
gaozhangmin commented on pull request #14776: URL: https://github.com/apache/pulsar/pull/14776#issuecomment-1073937223 /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] aparajita89 commented on a change in pull request #14102: PIP-144: Making SchemaRegistry implementation configurable

2022-03-21 Thread GitBox
aparajita89 commented on a change in pull request #14102: URL: https://github.com/apache/pulsar/pull/14102#discussion_r831128019 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/schema/DefaultSchemaRegistryService.java ## @@ -71,17 +75,17 @@ }

[pulsar-manager] branch master updated: Upgrade BKVM to 3.1.1 (#453)

2022-03-21 Thread eolivelli
This is an automated email from the ASF dual-hosted git repository. eolivelli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-manager.git The following commit(s) were added to refs/heads/master by this push: new ec186a5 Upgrade BKVM to 3.1.1

  1   2   >