[GitHub] [pulsar] Demogorgon314 commented on a change in pull request #13332: [pulsar-client] Fix multi topic reader has message available behavior

2021-12-16 Thread GitBox
Demogorgon314 commented on a change in pull request #13332: URL: https://github.com/apache/pulsar/pull/13332#discussion_r771177881 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/MultiTopicsConsumerImpl.java ## @@ -779,7 +779,7 @@ public boolean hasMess

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

2021-12-16 Thread GitBox
lhotari commented on a change in pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#discussion_r771172244 ## File path: docker/pulsar/Dockerfile ## @@ -33,7 +33,13 @@ COPY scripts/pulsar-zookeeper-ruok.sh /pulsar/bin COPY scripts/watch-znode.py /pulsar/bin

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

2021-12-16 Thread GitBox
lhotari commented on a change in pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#discussion_r771169746 ## File path: docker/pulsar/Dockerfile ## @@ -33,7 +33,13 @@ COPY scripts/pulsar-zookeeper-ruok.sh /pulsar/bin COPY scripts/watch-znode.py /pulsar/bin

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

2021-12-16 Thread GitBox
lhotari commented on a change in pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#discussion_r771169746 ## File path: docker/pulsar/Dockerfile ## @@ -33,7 +33,13 @@ COPY scripts/pulsar-zookeeper-ruok.sh /pulsar/bin COPY scripts/watch-znode.py /pulsar/bin

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

2021-12-16 Thread GitBox
lhotari commented on a change in pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#discussion_r771167834 ## File path: docker/pulsar/Dockerfile ## @@ -33,7 +33,13 @@ COPY scripts/pulsar-zookeeper-ruok.sh /pulsar/bin COPY scripts/watch-znode.py /pulsar/bin

[GitHub] [pulsar] codelipenghui commented on a change in pull request #12521: [Transaction]Txn client check timeout

2021-12-16 Thread GitBox
codelipenghui commented on a change in pull request #12521: URL: https://github.com/apache/pulsar/pull/12521#discussion_r771167571 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PartitionedProducerImpl.java ## @@ -191,6 +193,20 @@ private void start(Li

[GitHub] [pulsar] codelipenghui commented on a change in pull request #12521: [Transaction]Txn client check timeout

2021-12-16 Thread GitBox
codelipenghui commented on a change in pull request #12521: URL: https://github.com/apache/pulsar/pull/12521#discussion_r771166529 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/PartitionedProducerImpl.java ## @@ -191,6 +193,20 @@ private void start(Li

[GitHub] [pulsar-client-go] xige-16 opened a new issue #685: Add LastSequenceID interface for consumer

2021-12-16 Thread GitBox
xige-16 opened a new issue #685: URL: https://github.com/apache/pulsar-client-go/issues/685 **Is your feature request related to a problem? Please describe.** A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] **Describe the solution you'd

svn commit: r51531 - /dev/pulsar/pulsar-2.8.2-candidate-2/

2021-12-16 Thread linlin
Author: linlin Date: Fri Dec 17 07:28:04 2021 New Revision: 51531 Log: Remove release 2.8.2 rc2 Removed: dev/pulsar/pulsar-2.8.2-candidate-2/

[GitHub] [pulsar] yuruguo commented on pull request #12351: [pulsar-admin] Add remove-clusters command for namespace

2021-12-16 Thread GitBox
yuruguo commented on pull request #12351: URL: https://github.com/apache/pulsar/pull/12351#issuecomment-996484934 > > @eolivelli @codelipenghui @315157973 Sorry, Could you review this pr again? I have addressed all comments :) > > Please follow eolivelli‘s comment. It may be a risk i

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

2021-12-16 Thread GitBox
michaeljmarshall commented on pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#issuecomment-996484872 Looks like the test failures are related to containers failing to start. I'll take a closer look in the morning. -- This is an automated message from the Apache Git

[pulsar] annotated tag v2.7.4-candidate-1 updated (ab451b8 -> 80630af)

2021-12-16 Thread technoboy
This is an automated email from the ASF dual-hosted git repository. technoboy pushed a change to annotated tag v2.7.4-candidate-1 in repository https://gitbox.apache.org/repos/asf/pulsar.git. *** WARNING: tag v2.7.4-candidate-1 was modified! *** from ab451b8 (commit) to 80630af (tag

[GitHub] [pulsar] 315157973 commented on pull request #12351: [pulsar-admin] Add remove-clusters command for namespace

2021-12-16 Thread GitBox
315157973 commented on pull request #12351: URL: https://github.com/apache/pulsar/pull/12351#issuecomment-996481126 > @eolivelli @codelipenghui @315157973 Sorry, Could you review this pr again? I have addressed all comments :) Please follow eolivelli‘s comment. It may be a risk is to

[pulsar] branch branch-2.7 updated (c643a8a -> ab451b8)

2021-12-16 Thread technoboy
This is an automated email from the ASF dual-hosted git repository. technoboy pushed a change to branch branch-2.7 in repository https://gitbox.apache.org/repos/asf/pulsar.git. from c643a8a [Branch-2.7] Make the unit test and integration test work (#13243) add ab451b8 Release 2.7.4

[pulsar] branch master updated (1a56c85 -> fbe0103)

2021-12-16 Thread bogong
This is an automated email from the ASF dual-hosted git repository. bogong pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 1a56c85 [website][upgrade]feat: website upgrade / docs migration - 2.2.0 / deploy/io/sql (#13318) add fbe0103

[GitHub] [pulsar] congbobo184 merged pull request #13327: [Transaction] Delete the redundant code

2021-12-16 Thread GitBox
congbobo184 merged pull request #13327: URL: https://github.com/apache/pulsar/pull/13327 -- 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: commits-u

[GitHub] [pulsar] congbobo184 removed a comment on pull request #13327: [Transaction] Delete the redundant code

2021-12-16 Thread GitBox
congbobo184 removed a comment on pull request #13327: URL: https://github.com/apache/pulsar/pull/13327#issuecomment-996477841 ### Motivation The problem was resolved, so there is no need to add a wait and retry method again. ### Modification 1. Delete the redundant code 2. Optim

[GitHub] [pulsar] yuruguo removed a comment on pull request #13157: [Authorization] Converge authz of ns policies from super-user to tenant-administrator

2021-12-16 Thread GitBox
yuruguo removed a comment on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-996440639 @codelipenghui @freeznet 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 abo

[GitHub] [pulsar] congbobo184 commented on pull request #13327: [Transaction] Delete the redundant code

2021-12-16 Thread GitBox
congbobo184 commented on pull request #13327: URL: https://github.com/apache/pulsar/pull/13327#issuecomment-996477841 ### Motivation The problem was resolved, so there is no need to add a wait and retry method again. ### Modification 1. Delete the redundant code 2. Optimize some

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

2021-12-16 Thread GitBox
michaeljmarshall commented on pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#issuecomment-996473463 Mailing list thread: https://lists.apache.org/thread/lrt5ycdbms69npblp4j1jp75138wvn62 -- This is an automated message from the Apache Git Service. To respond to the

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

2021-12-16 Thread GitBox
315157973 commented on a change in pull request #13297: URL: https://github.com/apache/pulsar/pull/13297#discussion_r771133205 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/schema/compatibility/SchemaTypeCompatibilityCheckTest.java ## @@ -185,6 +204,7 @@ public v

[GitHub] [pulsar] eolivelli commented on a change in pull request #13332: [pulsar-client] Fix multi topic reader has message available behavior

2021-12-16 Thread GitBox
eolivelli commented on a change in pull request #13332: URL: https://github.com/apache/pulsar/pull/13332#discussion_r771131501 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/MultiTopicsConsumerImpl.java ## @@ -779,7 +779,7 @@ public boolean hasMessageA

[GitHub] [pulsar] michaeljmarshall commented on issue #13363: Producer registe failed when reconnecting

2021-12-16 Thread GitBox
michaeljmarshall commented on issue #13363: URL: https://github.com/apache/pulsar/issues/13363#issuecomment-996468881 @suiyuzeng - thanks for opening this issue. I have been working in this part of the code base recently, so I am happy to take a look. I should be able to sometime in the ne

[pulsar] branch branch-2.7 updated: [Branch-2.7] Make the unit test and integration test work (#13243)

2021-12-16 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui pushed a commit to branch branch-2.7 in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/branch-2.7 by this push: new c643a8a [Branch-2.7] Make the unit tes

[GitHub] [pulsar] codelipenghui merged pull request #13243: [Branch-2.7] Make the unit test and integration test work

2021-12-16 Thread GitBox
codelipenghui merged pull request #13243: URL: https://github.com/apache/pulsar/pull/13243 -- 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: commits

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

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #13376: URL: https://github.com/apache/pulsar/pull/13376#issuecomment-996465491 @michaeljmarshall:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/puls

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

2021-12-16 Thread GitBox
michaeljmarshall opened a new pull request #13376: URL: https://github.com/apache/pulsar/pull/13376 Master Issue: https://github.com/apache/pulsar/issues/11269 ### Motivation In order to increase the overall security of our Pulsar docker images, they should default to run as t

[GitHub] [pulsar] Demogorgon314 commented on pull request #13369: [C++] Adjust clang-format search names

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

[pulsar] branch master updated (e54b9d3 -> 1a56c85)

2021-12-16 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from e54b9d3 [website][upgrade]feat: website upgrade / docs migration - 2.2.0 / started/concepts/functions (#13317)

[GitHub] [pulsar] Anonymitaet merged pull request #13318: [website][upgrade]feat: website upgrade / docs migration - 2.2.0 / deploy/io/sql

2021-12-16 Thread GitBox
Anonymitaet merged pull request #13318: URL: https://github.com/apache/pulsar/pull/13318 -- 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: commits-u

[GitHub] [pulsar] HQebupt commented on pull request #13352: PIP-119: Enable consistent hashing by default on KeyShared subscriptions dispatcher

2021-12-16 Thread GitBox
HQebupt commented on pull request #13352: URL: https://github.com/apache/pulsar/pull/13352#issuecomment-996441413 /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 th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz of ns policies from super-user to tenant-administrator

2021-12-16 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-996440639 @codelipenghui @freeznet 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 above to go

[GitHub] [pulsar] gaozhangmin commented on pull request #13204: support limit message size produced by protocol handlers by maxMessag…

2021-12-16 Thread GitBox
gaozhangmin commented on pull request #13204: URL: https://github.com/apache/pulsar/pull/13204#issuecomment-996438208 /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 t

[GitHub] [pulsar] kwenZh opened a new issue #13375: broker offload use s3 driver, java.io.IOException: No offloader found for driver 'S3'. Please make sure you dropped the offloader nar packages under

2021-12-16 Thread GitBox
kwenZh opened a new issue #13375: URL: https://github.com/apache/pulsar/issues/13375 in pulsar 2.8.1 version, use offload, broker err log: ``` java.io.IOException: No offloader found for driver 'S3'. Please make sure you dropped the offloader nar packages under `${PULSAR_HOME

[GitHub] [pulsar] gaozhangmin closed pull request #13374: bump grpc version to 1.42.1

2021-12-16 Thread GitBox
gaozhangmin closed pull request #13374: URL: https://github.com/apache/pulsar/pull/13374 -- 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: commits-u

[GitHub] [pulsar] github-actions[bot] commented on pull request #13374: bump grpc version to 1.42.1

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #13374: URL: https://github.com/apache/pulsar/pull/13374#issuecomment-996423282 @gaozhangmin:Thanks for providing doc info! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [pulsar] github-actions[bot] commented on pull request #13374: bump grpc version to 1.42.1

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #13374: URL: https://github.com/apache/pulsar/pull/13374#issuecomment-996422713 @gaozhangmin:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/bl

[GitHub] [pulsar] gaozhangmin opened a new pull request #13374: bump grpc version to 1.42.1

2021-12-16 Thread GitBox
gaozhangmin opened a new pull request #13374: URL: https://github.com/apache/pulsar/pull/13374 bump pulsar GRPC version to 1.42.1 since bookkeeper had. Compile error: ``` Could not resolve dependencies for project org.apache.pulsar:pulsar-functions-instance:jar:2.10.0-SNAPSHOT

[GitHub] [pulsar] yuruguo commented on pull request #12351: [pulsar-admin] Add remove-clusters command for namespace

2021-12-16 Thread GitBox
yuruguo commented on pull request #12351: URL: https://github.com/apache/pulsar/pull/12351#issuecomment-996419720 @eolivelli @codelipenghui @315157973 Sorry, Could you review this pr again? I have addressed all comments :) -- This is an automated message from the Apache Git Service. To r

[GitHub] [pulsar] yuruguo removed a comment on pull request #12351: [pulsar-admin] Add remove-clusters command for namespace

2021-12-16 Thread GitBox
yuruguo removed a comment on pull request #12351: URL: https://github.com/apache/pulsar/pull/12351#issuecomment-994563423 @eolivelli @codelipenghui @315157973 Would you help review this pr again? I have addressed all comments :) -- This is an automated message from the Apache Git Service

[GitHub] [pulsar] github-actions[bot] commented on pull request #13373: Support rename the file has been processed.

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

[GitHub] [pulsar] casuallc opened a new pull request #13373: Support rename the file has been processed.

2021-12-16 Thread GitBox
casuallc opened a new pull request #13373: URL: https://github.com/apache/pulsar/pull/13373 ### Motivation Support rename the file has been processed. #13301 ### Modifications Add property 'processedFileSuffix'. If set, do not delete but only rename file that has

svn commit: r51528 - in /dev/pulsar/pulsar-2.8.2-candidate-2: ./ apache-pulsar-2.8.2-bin.tar.gz apache-pulsar-2.8.2-src.tar.gz

2021-12-16 Thread linlin
Author: linlin Date: Fri Dec 17 03:51:36 2021 New Revision: 51528 Log: Staging artifacts and signature for Pulsar release 2.8.2 Added: dev/pulsar/pulsar-2.8.2-candidate-2/ dev/pulsar/pulsar-2.8.2-candidate-2/apache-pulsar-2.8.2-bin.tar.gz (with props) dev/pulsar/pulsar-2.8.2-candid

[GitHub] [pulsar] yuruguo commented on a change in pull request #13157: [Authorization] Converge authz of ns policies from super-user to tenant-administrator

2021-12-16 Thread GitBox
yuruguo commented on a change in pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#discussion_r770186142 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/Namespaces.java ## @@ -122,7 +122,7 @@ @Path("/{property}/{cluster}/{na

[GitHub] [pulsar] yuruguo commented on a change in pull request #13157: [Authorization] Converge authz of ns policies from super-user to tenant-administrator

2021-12-16 Thread GitBox
yuruguo commented on a change in pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#discussion_r770186142 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/Namespaces.java ## @@ -122,7 +122,7 @@ @Path("/{property}/{cluster}/{na

[GitHub] [pulsar] Jason918 commented on issue #13363: Producer registe failed when reconnecting

2021-12-16 Thread GitBox
Jason918 commented on issue #13363: URL: https://github.com/apache/pulsar/issues/13363#issuecomment-996408912 @michaeljmarshall PTAL -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [pulsar] Demogorgon314 commented on pull request #13369: [C++] Adjust clang-format search names

2021-12-16 Thread GitBox
Demogorgon314 commented on pull request #13369: URL: https://github.com/apache/pulsar/pull/13369#issuecomment-996404693 /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] hangc0276 commented on pull request #13316: PIP-121: Pulsar cluster level auto failover

2021-12-16 Thread GitBox
hangc0276 commented on pull request #13316: URL: https://github.com/apache/pulsar/pull/13316#issuecomment-996394985 > > As you mentioned, we use geo-replication to support cluster failover, that means there might be have more than two cluster. In the implementation, we only can configure t

[GitHub] [pulsar] hangc0276 commented on pull request #13316: PIP-121: Pulsar cluster level auto failover

2021-12-16 Thread GitBox
hangc0276 commented on pull request #13316: URL: https://github.com/apache/pulsar/pull/13316#issuecomment-996394629 > As you mentioned, we use geo-replication to support cluster failover, that means there might be have more than two cluster. In the implementation, we only can configure two

[GitHub] [pulsar] zymap commented on a change in pull request #13366: [owasp] Periodic jobs on active branches

2021-12-16 Thread GitBox
zymap commented on a change in pull request #13366: URL: https://github.com/apache/pulsar/pull/13366#discussion_r771063581 ## File path: .github/workflows/ci-owasp-dependency-check-2.7.yaml ## @@ -0,0 +1,62 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +#

[pulsar] branch master updated (c20292c -> e54b9d3)

2021-12-16 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from c20292c Replace serveral deprecated api usages (#13351) add e54b9d3 [website][upgrade]feat: website upgrade / d

[GitHub] [pulsar] hangc0276 commented on a change in pull request #13316: PIP-121: Pulsar cluster level auto failover

2021-12-16 Thread GitBox
hangc0276 commented on a change in pull request #13316: URL: https://github.com/apache/pulsar/pull/13316#discussion_r771062794 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/AutoClusterFailover.java ## @@ -0,0 +1,234 @@ +/** + * Licensed to the Apache

[GitHub] [pulsar] Anonymitaet merged pull request #13317: [website][upgrade]feat: website upgrade / docs migration - 2.2.0 / started/concepts/functions

2021-12-16 Thread GitBox
Anonymitaet merged pull request #13317: URL: https://github.com/apache/pulsar/pull/13317 -- 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: commits-u

[GitHub] [pulsar] hangc0276 commented on a change in pull request #13316: PIP-121: Pulsar cluster level auto failover

2021-12-16 Thread GitBox
hangc0276 commented on a change in pull request #13316: URL: https://github.com/apache/pulsar/pull/13316#discussion_r771061656 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/AutoClusterFailover.java ## @@ -0,0 +1,234 @@ +/** + * Licensed to the Apache

[GitHub] [pulsar] urfreespace opened a new pull request #13372: [website][upgrade]feat: website upgrade / docs migration - 2.1.1 / reference

2021-12-16 Thread GitBox
urfreespace opened a new pull request #13372: URL: https://github.com/apache/pulsar/pull/13372 *(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] urfreespace opened a new pull request #13371: [website][upgrade]feat: website upgrade / docs migration - 2.1.1 / adaptors/cookbooks/develop

2021-12-16 Thread GitBox
urfreespace opened a new pull request #13371: URL: https://github.com/apache/pulsar/pull/13371 *(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] urfreespace opened a new pull request #13370: [website][upgrade]feat: website upgrade / docs migration - 2.1.1 / client/security/admin api

2021-12-16 Thread GitBox
urfreespace opened a new pull request #13370: URL: https://github.com/apache/pulsar/pull/13370 *(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] hangc0276 commented on a change in pull request #13316: PIP-121: Pulsar cluster level auto failover

2021-12-16 Thread GitBox
hangc0276 commented on a change in pull request #13316: URL: https://github.com/apache/pulsar/pull/13316#discussion_r771059962 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/AutoClusterFailover.java ## @@ -0,0 +1,234 @@ +/** + * Licensed to the Apache

[GitHub] [pulsar] Demogorgon314 opened a new pull request #13369: [C++] Adjust clang-format search names

2021-12-16 Thread GitBox
Demogorgon314 opened a new pull request #13369: URL: https://github.com/apache/pulsar/pull/13369 ### Motivation The Pulsar C++ client use `clang-format` 5.0 to format files, we don't need to search for a version lower than 5.0. Additionally, the old version of clang-format mig

[GitHub] [pulsar] codelipenghui commented on pull request #13348: [Transaction] Do not abort when commit a problematic txn, but throw an exception

2021-12-16 Thread GitBox
codelipenghui commented on pull request #13348: URL: https://github.com/apache/pulsar/pull/13348#issuecomment-996382294 @liangyepianzhou We should add more information in the PR description so that all the people in the community can know why we make this change. -- This is an automated

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

2021-12-16 Thread GitBox
RobertIndie commented on pull request #12403: URL: https://github.com/apache/pulsar/pull/12403#issuecomment-996380758 /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 t

[GitHub] [pulsar] urfreespace opened a new pull request #13368: [website][upgrade]feat: website upgrade / docs migration - 2.1.1 / administration/deploy/io

2021-12-16 Thread GitBox
urfreespace opened a new pull request #13368: URL: https://github.com/apache/pulsar/pull/13368 *(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] urfreespace opened a new pull request #13367: [website][upgrade]feat: website upgrade / docs migration - 2.2.1 / started/functions/concepts

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

[GitHub] [pulsar] Jason918 commented on pull request #13361: [Broker] Optimize TopicPolicies#maxConsumerPerTopic with HierarchyTopicPolicies

2021-12-16 Thread GitBox
Jason918 commented on pull request #13361: URL: https://github.com/apache/pulsar/pull/13361#issuecomment-996377996 /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 t

[GitHub] [pulsar] wangjialing218 commented on issue #13315: PIP-121: Pulsar cluster level auto failover

2021-12-16 Thread GitBox
wangjialing218 commented on issue #13315: URL: https://github.com/apache/pulsar/issues/13315#issuecomment-996373128 If cluster B has a different authentication data with cluster A, such as another token string, how to change the authentication setting in client when auto failover happen?

[GitHub] [pulsar] gaozhangmin commented on pull request #13204: support limit message size produced by protocol handlers by maxMessag…

2021-12-16 Thread GitBox
gaozhangmin commented on pull request #13204: URL: https://github.com/apache/pulsar/pull/13204#issuecomment-996374933 /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 t

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

2021-12-16 Thread GitBox
Jason918 commented on a change in pull request #13297: URL: https://github.com/apache/pulsar/pull/13297#discussion_r771045399 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractTopic.java ## @@ -163,6 +168,7 @@ protected void updateTopicPolicy(T

[GitHub] [pulsar] wangjialing218 commented on a change in pull request #13204: support limit message size produced by protocol handlers by maxMessag…

2021-12-16 Thread GitBox
wangjialing218 commented on a change in pull request #13204: URL: https://github.com/apache/pulsar/pull/13204#discussion_r771030071 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java ## @@ -219,6 +208,7 @@ protected MutableLong initialVal

[pulsar] branch master updated (6ddb533 -> c20292c)

2021-12-16 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 6ddb533 [Issue 13285][pulsar-client] Optimize MessageBuilder and SharedBuffer to avoid unnecessary memory copy. (#1

[pulsar] branch master updated (64bbbd4 -> 6ddb533)

2021-12-16 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 64bbbd4 [website][upgrade]feat: website upgrade / docs migration - 2.2.1 / develop/reference (#13314) add 6ddb

[GitHub] [pulsar] merlimat merged pull request #13351: Replace serveral deprecated api usages

2021-12-16 Thread GitBox
merlimat merged pull request #13351: URL: https://github.com/apache/pulsar/pull/13351 -- 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: commits-unsu

[GitHub] [pulsar] merlimat closed issue #13285: [cpp-client] Optimize MessageBuild and SharedBuffer to avoid unnecessary memory copy.

2021-12-16 Thread GitBox
merlimat closed issue #13285: URL: https://github.com/apache/pulsar/issues/13285 -- 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: commits-unsubscr.

[GitHub] [pulsar] merlimat merged pull request #13293: [Issue 13285][pulsar-client] Optimize MessageBuilder and SharedBuffer to avoid unnecessary memory copy.

2021-12-16 Thread GitBox
merlimat merged pull request #13293: URL: https://github.com/apache/pulsar/pull/13293 -- 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: commits-unsu

[GitHub] [pulsar] merlimat commented on a change in pull request #13355: [WIP][Client] Create init subscription while creating DLQ producer

2021-12-16 Thread GitBox
merlimat commented on a change in pull request #13355: URL: https://github.com/apache/pulsar/pull/13355#discussion_r770764494 ## File path: pulsar-client-api/src/main/java/org/apache/pulsar/client/api/DeadLetterPolicy.java ## @@ -49,4 +49,15 @@ */ private String dea

[GitHub] [pulsar] merlimat commented on a change in pull request #13355: [WIP][Client] Create init subscription while creating DLQ producer

2021-12-16 Thread GitBox
merlimat commented on a change in pull request #13355: URL: https://github.com/apache/pulsar/pull/13355#discussion_r770761644 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java ## @@ -1855,10 +1863,20 @@ private void initDeadLetterProduce

[GitHub] [pulsar] michaeljmarshall commented on a change in pull request #13345: Fix typo in functions-work.yaml, add functionWebServiceUrl in yaml

2021-12-16 Thread GitBox
michaeljmarshall commented on a change in pull request #13345: URL: https://github.com/apache/pulsar/pull/13345#discussion_r770753101 ## File path: conf/functions_worker.yml ## @@ -57,6 +57,7 @@ pulsarServiceUrl: pulsar://localhost:6650 # For TLS: # webServiceUrl=https://loca

[GitHub] [pulsar] github-actions[bot] commented on pull request #2609: Initial commit for a go based pulsar cli.

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #2609: URL: https://github.com/apache/pulsar/pull/2609#issuecomment-996015463 @cckellogg:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/m

[GitHub] [pulsar] dave2wave closed pull request #2609: Initial commit for a go based pulsar cli.

2021-12-16 Thread GitBox
dave2wave closed pull request #2609: URL: https://github.com/apache/pulsar/pull/2609 -- 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: commits-unsub

[GitHub] [pulsar] dave2wave commented on pull request #2609: Initial commit for a go based pulsar cli.

2021-12-16 Thread GitBox
dave2wave commented on pull request #2609: URL: https://github.com/apache/pulsar/pull/2609#issuecomment-996015253 This PR is evidently stale or abandoned. Reopen if this is not so. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [pulsar] github-actions[bot] commented on pull request #1183: Apply make format into Python C++ wrapper client

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #1183: URL: https://github.com/apache/pulsar/pull/1183#issuecomment-996013260 @Licht-T:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/mas

[GitHub] [pulsar] dave2wave closed pull request #1183: Apply make format into Python C++ wrapper client

2021-12-16 Thread GitBox
dave2wave closed pull request #1183: URL: https://github.com/apache/pulsar/pull/1183 -- 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: commits-unsub

[GitHub] [pulsar] dave2wave commented on pull request #1183: Apply make format into Python C++ wrapper client

2021-12-16 Thread GitBox
dave2wave commented on pull request #1183: URL: https://github.com/apache/pulsar/pull/1183#issuecomment-996013029 This PR is evidently stale or abandoned. Reopen if this is not so. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [pulsar] github-actions[bot] commented on pull request #1020: Inject system-env variables to pulsar configuration

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #1020: URL: https://github.com/apache/pulsar/pull/1020#issuecomment-996010395 @rdhabalia:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/m

[GitHub] [pulsar] dave2wave commented on pull request #1020: Inject system-env variables to pulsar configuration

2021-12-16 Thread GitBox
dave2wave commented on pull request #1020: URL: https://github.com/apache/pulsar/pull/1020#issuecomment-996010242 This PR is evidently stale or abandoned. Reopen if this is not so. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [pulsar] dave2wave closed pull request #1020: Inject system-env variables to pulsar configuration

2021-12-16 Thread GitBox
dave2wave closed pull request #1020: URL: https://github.com/apache/pulsar/pull/1020 -- 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: commits-unsub

[GitHub] [pulsar] github-actions[bot] commented on pull request #1015: Add customize zookeeper-shell to read binary-managed-ledger

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #1015: URL: https://github.com/apache/pulsar/pull/1015#issuecomment-996008045 @rdhabalia:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/m

[GitHub] [pulsar] dave2wave closed pull request #1015: Add customize zookeeper-shell to read binary-managed-ledger

2021-12-16 Thread GitBox
dave2wave closed pull request #1015: URL: https://github.com/apache/pulsar/pull/1015 -- 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: commits-unsub

[GitHub] [pulsar] dave2wave commented on pull request #1015: Add customize zookeeper-shell to read binary-managed-ledger

2021-12-16 Thread GitBox
dave2wave commented on pull request #1015: URL: https://github.com/apache/pulsar/pull/1015#issuecomment-996007833 This PR is evidently stale or abandoned. Reopen if this is not so. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [pulsar] github-actions[bot] commented on pull request #564: Set timeouts on admin requests

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #564: URL: https://github.com/apache/pulsar/pull/564#issuecomment-996005955 @sschepens:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/mas

[GitHub] [pulsar] dave2wave closed pull request #564: Set timeouts on admin requests

2021-12-16 Thread GitBox
dave2wave closed pull request #564: URL: https://github.com/apache/pulsar/pull/564 -- 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: commits-unsubsc

[GitHub] [pulsar] dave2wave commented on pull request #564: Set timeouts on admin requests

2021-12-16 Thread GitBox
dave2wave commented on pull request #564: URL: https://github.com/apache/pulsar/pull/564#issuecomment-996005735 This PR is evidently stale or abandoned. Reopen if this is not so. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [pulsar] github-actions[bot] commented on pull request #298: Support receiveAsync cancellation

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #298: URL: https://github.com/apache/pulsar/pull/298#issuecomment-996004649 @sschepens:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/mas

[GitHub] [pulsar] dave2wave closed pull request #298: Support receiveAsync cancellation

2021-12-16 Thread GitBox
dave2wave closed pull request #298: URL: https://github.com/apache/pulsar/pull/298 -- 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: commits-unsubsc

[GitHub] [pulsar] dave2wave commented on pull request #298: Support receiveAsync cancellation

2021-12-16 Thread GitBox
dave2wave commented on pull request #298: URL: https://github.com/apache/pulsar/pull/298#issuecomment-996004148 This PR is evidently stale or abandoned. Reopen if this is not so. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [pulsar] github-actions[bot] commented on pull request #234: Enable gzip for broker web service

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #234: URL: https://github.com/apache/pulsar/pull/234#issuecomment-996003399 @sschepens:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/mas

[GitHub] [pulsar] dave2wave closed pull request #234: Enable gzip for broker web service

2021-12-16 Thread GitBox
dave2wave closed pull request #234: URL: https://github.com/apache/pulsar/pull/234 -- 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: commits-unsubsc

[GitHub] [pulsar] dave2wave commented on pull request #234: Enable gzip for broker web service

2021-12-16 Thread GitBox
dave2wave commented on pull request #234: URL: https://github.com/apache/pulsar/pull/234#issuecomment-996003209 This PR is evidently stale or abandoned. Reopen if this is not so. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [pulsar] github-actions[bot] commented on pull request #208: Add LoadBalancer and OwnershipCache factory to support dual write/read to local_zk and data_zk

2021-12-16 Thread GitBox
github-actions[bot] commented on pull request #208: URL: https://github.com/apache/pulsar/pull/208#issuecomment-996001659 @rdhabalia:Thanks for your contribution. For this PR, do we need to update docs? (The [PR template contains info about doc](https://github.com/apache/pulsar/blob/mas

[GitHub] [pulsar] dave2wave commented on pull request #208: Add LoadBalancer and OwnershipCache factory to support dual write/read to local_zk and data_zk

2021-12-16 Thread GitBox
dave2wave commented on pull request #208: URL: https://github.com/apache/pulsar/pull/208#issuecomment-996001537 This PR is evidently stale or abandoned. Reopen if this is not so. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [pulsar] dave2wave closed pull request #208: Add LoadBalancer and OwnershipCache factory to support dual write/read to local_zk and data_zk

2021-12-16 Thread GitBox
dave2wave closed pull request #208: URL: https://github.com/apache/pulsar/pull/208 -- 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: commits-unsubsc

  1   2   >