[GitHub] [pulsar] mattisonchao closed pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao closed pull request #13745: URL: https://github.com/apache/pulsar/pull/13745 -- 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] sunshujie1990 edited a comment on issue #13757: invalid config item -Dio.netty.recycler.maxCapacity.default

2022-01-13 Thread GitBox
sunshujie1990 edited a comment on issue #13757: URL: https://github.com/apache/pulsar/issues/13757#issuecomment-1012855809 @merlimat @sijie -- 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 s

[GitHub] [pulsar] sunshujie1990 commented on issue #13737: Replace -XX:+DisableExplicitGC with -XX:+ExplicitGCInvokesConcurrent

2022-01-13 Thread GitBox
sunshujie1990 commented on issue #13737: URL: https://github.com/apache/pulsar/issues/13737#issuecomment-1012859385 @merlimat -- 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 commen

[GitHub] [pulsar] sunshujie1990 commented on issue #13757: invalid config item -Dio.netty.recycler.maxCapacity.default

2022-01-13 Thread GitBox
sunshujie1990 commented on issue #13757: URL: https://github.com/apache/pulsar/issues/13757#issuecomment-1012855809 @merlimat -- 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 commen

[GitHub] [pulsar] sunshujie1990 opened a new issue #13757: invalid config item -Dio.netty.recycler.maxCapacity.default

2022-01-13 Thread GitBox
sunshujie1990 opened a new issue #13757: URL: https://github.com/apache/pulsar/issues/13757 netty version 4.1.66.Final Can not find config item io.netty.recycler.maxCapacity.default in io.netty.util.Recycler. I think it's a invalid config. ![image](https://user-images.githubuserc

[GitHub] [pulsar] Jason918 opened a new issue #13756: [broker] Optimize topic policy with HierarchyTopicPolicies

2022-01-13 Thread GitBox
Jason918 opened a new issue #13756: URL: https://github.com/apache/pulsar/issues/13756 This issue is created to track the process of the serial topic policy optimization with `HierarchyTopicPolicies`. ## Motivation Update topic policy with HierarchyTopicPolicies comes wit

[pulsar-client-go] branch master updated: Fix slice bounds out of range for readSingleMessage (#709)

2022-01-13 Thread yong
This is an automated email from the ASF dual-hosted git repository. yong 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 244facf Fix slice bounds out of range f

[GitHub] [pulsar] michaeljmarshall opened a new pull request #12900: [Broker] Return from callback early if producerFuture is complete or …

2022-01-13 Thread GitBox
michaeljmarshall opened a new pull request #12900: URL: https://github.com/apache/pulsar/pull/12900 …connection is closed ### Motivation When producer creation times out on the client side, the producer sends a `CloseProducer` command and then a `Producer` command. Each time i

[GitHub] [pulsar] lhotari closed pull request #12900: [Broker] Return from callback early if producerFuture is complete or …

2022-01-13 Thread GitBox
lhotari closed pull request #12900: URL: https://github.com/apache/pulsar/pull/12900 -- 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

[pulsar] branch master updated (a05694b -> a255992)

2022-01-13 Thread eolivelli
This is an automated email from the ASF dual-hosted git repository. eolivelli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from a05694b [Tests] Workaround flakiness issue in EndToEndMetadataTest.testPublishConsume (#13754) add a255992

[GitHub] [pulsar] lhotari commented on pull request #12900: [Broker] Return from callback early if producerFuture is complete or …

2022-01-13 Thread GitBox
lhotari commented on pull request #12900: URL: https://github.com/apache/pulsar/pull/12900#issuecomment-1012844979 Closing and re-opening to get recent changes from master branch. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [pulsar-client-go] zymap closed issue #702: consume panic

2022-01-13 Thread GitBox
zymap closed issue #702: URL: https://github.com/apache/pulsar-client-go/issues/702 -- 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-unsubs

[GitHub] [pulsar] eolivelli merged pull request #13639: Make LoadBalancer's ResourceUsage class immutable

2022-01-13 Thread GitBox
eolivelli merged pull request #13639: URL: https://github.com/apache/pulsar/pull/13639 -- 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-uns

[GitHub] [pulsar-client-go] zymap merged pull request #709: Fix slice bounds out of range for readSingleMessage

2022-01-13 Thread GitBox
zymap merged pull request #709: URL: https://github.com/apache/pulsar-client-go/pull/709 -- 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] mattisonchao removed a comment on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao removed a comment on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012691423 -- 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 commen

[GitHub] [pulsar] mattisonchao removed a comment on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao removed a comment on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012834435 -- 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 commen

[GitHub] [pulsar] momo-jun commented on pull request #13751: [Docs] Doc updates for PR#12403, PR#13108 and PR#13043

2022-01-13 Thread GitBox
momo-jun commented on pull request #13751: URL: https://github.com/apache/pulsar/pull/13751#issuecomment-1012836846 > @momo-jun before submitting this PR, did you preview the changes in your local environment? If so, can you add some screenshots of the preview in the PR description? So tha

[GitHub] [pulsar] Jason918 commented on pull request #13725: Fix method is not safely accessed by multiple concurrent threads.

2022-01-13 Thread GitBox
Jason918 commented on pull request #13725: URL: https://github.com/apache/pulsar/pull/13725#issuecomment-1012836703 Sorry, I am not getting this yet. In the error part. https://github.com/apache/pulsar/blob/2a7515f9593a76b294bfe2835621a0ab8a904957/pulsar-metadata/src/test/jav

[GitHub] [pulsar] mattisonchao commented on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012834435 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] lhotari closed pull request #13726: Updating dependencies to get rid of CVEs brought in with kafka and log4j-1.2 libs

2022-01-13 Thread GitBox
lhotari closed pull request #13726: URL: https://github.com/apache/pulsar/pull/13726 -- 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] lhotari commented on pull request #13726: Updating dependencies to get rid of CVEs brought in with kafka and log4j-1.2 libs

2022-01-13 Thread GitBox
lhotari commented on pull request #13726: URL: https://github.com/apache/pulsar/pull/13726#issuecomment-1012818296 Closing and re-opening to pick up [fix for CI](https://github.com/apache/pulsar/pull/13754). -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [pulsar] codelipenghui closed pull request #13744: Fix the deadlock while using zookeeper thread to create ledger

2022-01-13 Thread GitBox
codelipenghui closed pull request #13744: URL: https://github.com/apache/pulsar/pull/13744 -- 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

[pulsar] branch master updated (8798392 -> a05694b)

2022-01-13 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 8798392 [broker] Optimize TopicPolicies#compactionThreshold with HierarchyTopicPolicies (#13710) add a05694b

[GitHub] [pulsar] lhotari merged pull request #13754: [Tests] Workaround flakiness issue in EndToEndMetadataTest.testPublishConsume

2022-01-13 Thread GitBox
lhotari merged pull request #13754: URL: https://github.com/apache/pulsar/pull/13754 -- 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] lhotari closed issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
lhotari closed issue #13750: URL: https://github.com/apache/pulsar/issues/13750 -- 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] lhotari commented on pull request #13746: [security] Offloaders - get rid of several CVEs (including Log4J 2.17)

2022-01-13 Thread GitBox
lhotari commented on pull request #13746: URL: https://github.com/apache/pulsar/pull/13746#issuecomment-1012813626 @nicoloboschi it seems that there will be conflicts with #13726 once that is merged. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] Shawyeok commented on pull request #13742: [broker] Fix inefficient forEach loop

2022-01-13 Thread GitBox
Shawyeok commented on pull request #13742: URL: https://github.com/apache/pulsar/pull/13742#issuecomment-1012806121 /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 t

[GitHub] [pulsar] nlu90 commented on pull request #13748: fix checkstyle for tired-storage, websocket and bc modules

2022-01-13 Thread GitBox
nlu90 commented on pull request #13748: URL: https://github.com/apache/pulsar/pull/13748#issuecomment-1012802312 /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 the

[GitHub] [pulsar] mattisonchao commented on a change in pull request #13738: Fix wrong default value of expireTimeOfIncompleteChunkedMessage.

2022-01-13 Thread GitBox
mattisonchao commented on a change in pull request #13738: URL: https://github.com/apache/pulsar/pull/13738#discussion_r784524482 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ConsumerConfigurationData.java ## @@ -114,7 +114,7 @@ public int getMa

[GitHub] [pulsar] mattisonchao commented on a change in pull request #13753: Updated dependencies to get rid of pulsar-io/jdbc related CVE-2020-13692

2022-01-13 Thread GitBox
mattisonchao commented on a change in pull request #13753: URL: https://github.com/apache/pulsar/pull/13753#discussion_r784522728 ## File path: src/owasp-dependency-check-suppressions.xml ## @@ -41,4 +41,13 @@ org\.apache\.zookeeper:.*:3\.6\.2 .* + + + + +

[GitHub] [pulsar] equanz commented on pull request #13644: Rename test file name from `*Test2.java` to `*Test.java` to run all tests correctly

2022-01-13 Thread GitBox
equanz commented on pull request #13644: URL: https://github.com/apache/pulsar/pull/13644#issuecomment-1012793270 /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] gaozhangmin opened a new issue #13755: Flaky-test: [RGUsageMTAggrWaitForAllMsgsTest]. [testMTProduceConsumeRGUsageNonPersistentTopicNamesDifferentTenant]

2022-01-13 Thread GitBox
gaozhangmin opened a new issue #13755: URL: https://github.com/apache/pulsar/issues/13755 https://github.com/apache/pulsar/runs/4812847056?check_suite_focus=true ``` java.lang.AssertionError: expected [400] but found [0] at org.testng.Assert.fail(Assert.java:99) at

[GitHub] [pulsar] lhotari commented on a change in pull request #13753: Updated dependencies to get rid of pulsar-io/jdbc related CVE-2020-13692

2022-01-13 Thread GitBox
lhotari commented on a change in pull request #13753: URL: https://github.com/apache/pulsar/pull/13753#discussion_r784517803 ## File path: src/owasp-dependency-check-suppressions.xml ## @@ -41,4 +41,13 @@ org\.apache\.zookeeper:.*:3\.6\.2 .* + + + + + ^pkg:

[GitHub] [pulsar] liangyepianzhou commented on pull request #13739: [Transaction] Fix recover of TransactionBuffer

2022-01-13 Thread GitBox
liangyepianzhou commented on pull request #13739: URL: https://github.com/apache/pulsar/pull/13739#issuecomment-1012789874 /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] mattisonchao removed a comment on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao removed a comment on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012751138 /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 a

[GitHub] [pulsar] mattisonchao commented on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012788310 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] mattisonchao edited a comment on issue #13741: Consumer already closed with system topic

2022-01-13 Thread GitBox
mattisonchao edited a comment on issue #13741: URL: https://github.com/apache/pulsar/issues/13741#issuecomment-1012779903 @Shawyeok Could you give us more details or context ? - How to get it ? - How to reproduce it ? -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar] mattisonchao edited a comment on issue #13741: Consumer already closed with system topic

2022-01-13 Thread GitBox
mattisonchao edited a comment on issue #13741: URL: https://github.com/apache/pulsar/issues/13741#issuecomment-1012779903 @Shawyeok Could you give us more detail or context ? - How to get it ? - How to reproduce it ? -- This is an automated message from the Apache Git Service. T

[GitHub] [pulsar] mattisonchao commented on issue #13741: Consumer already closed with system topic

2022-01-13 Thread GitBox
mattisonchao commented on issue #13741: URL: https://github.com/apache/pulsar/issues/13741#issuecomment-1012779903 @Shawyeok Could you give us more detail ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [pulsar] mattisonchao commented on pull request #13754: [Tests] Workaround flakiness issue in EndToEndMetadataTest.testPublishConsume

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13754: URL: https://github.com/apache/pulsar/pull/13754#issuecomment-1012778147 > I see, Thanks to your explanation. 👍🏼 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [pulsar] lhotari edited a comment on pull request #13754: [Tests] Workaround flakiness issue in EndToEndMetadataTest.testPublishConsume

2022-01-13 Thread GitBox
lhotari edited a comment on pull request #13754: URL: https://github.com/apache/pulsar/pull/13754#issuecomment-1012777552 > @lhotari > > Do we need to remove the following code ? > > ```java > public BKCluster(String metadataServiceUri, int numBookies) throws Exception {

[GitHub] [pulsar] lhotari commented on pull request #13754: [Tests] Workaround flakiness issue in EndToEndMetadataTest.testPublishConsume

2022-01-13 Thread GitBox
lhotari commented on pull request #13754: URL: https://github.com/apache/pulsar/pull/13754#issuecomment-1012777552 > @lhotari > > Do we need to remove the following code ? > > ```java > public BKCluster(String metadataServiceUri, int numBookies) throws Exception { >

[GitHub] [pulsar] lhotari edited a comment on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
lhotari edited a comment on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012776992 you can see in BK code that it's not harmful: https://github.com/apache/bookkeeper/blob/bc02d8c487a809fa58d75c477d9e2d5c7dedccec/bookkeeper-server/src/main/java/org/apache

[GitHub] [pulsar] lhotari commented on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
lhotari commented on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012776992 you can see in BK code that it's not harmful: https://github.com/apache/bookkeeper/blob/bc02d8c487a809fa58d75c477d9e2d5c7dedccec/bookkeeper-server/src/main/java/org/apache/bookke

[GitHub] [pulsar] mattisonchao commented on pull request #13754: [Tests] Workaround flakiness issue in EndToEndMetadataTest.testPublishConsume

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13754: URL: https://github.com/apache/pulsar/pull/13754#issuecomment-1012776866 @lhotari Do we need to remove the following code ? ```java public BKCluster(String metadataServiceUri, int numBookies) throws Exception { //

[GitHub] [pulsar] lhotari commented on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
lhotari commented on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012776747 > @lhotari Can we only add the properties for `EndToEndMetadataTest`, looks like other tests will not test different drivers? I already created #13754 . There doesn't seem to

[GitHub] [pulsar] lhotari opened a new pull request #13754: [Tests] Workaround flakiness issue in EndToEndMetadataTest.testPublishConsume

2022-01-13 Thread GitBox
lhotari opened a new pull request #13754: URL: https://github.com/apache/pulsar/pull/13754 Fixes #13750 ### Motivation - If the org.apache.bookkeeper.meta.MetadataDrivers class got loaded before org.apache.pulsar.metadata.bookkeeper.BKCluster constructor is called, things wou

[GitHub] [pulsar] mattisonchao commented on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
mattisonchao commented on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012775250 > The workaround would be to set the system properties for all tests in pulsar-broker using the pom.xml. I can create a PR for that since it shouldn't be harmful. I thi

[GitHub] [pulsar] codelipenghui commented on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
codelipenghui commented on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012774683 @lhotari Can we only add the properties for `EndToEndMetadataTest`, looks like other tests will not test different drivers? -- This is an automated message from the Apache

[GitHub] [pulsar] lhotari edited a comment on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
lhotari edited a comment on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012769783 The workaround would be to set the system properties for all tests in pulsar-broker using the pom.xml. I can create a PR for that since it shouldn't be harmful. Anoth

[GitHub] [pulsar] lhotari commented on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
lhotari commented on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012769783 The workaround would be to set the system properties for all tests in pulsar-broker using the pom.xml. I can create a PR for that since it shouldn't be harmful. -- This is an au

[GitHub] [pulsar] lhotari commented on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
lhotari commented on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012762062 This seems to be related to #12770 . If the org.apache.bookkeeper.meta.MetadataDrivers class gets loaded before org.apache.pulsar.metadata.bookkeeper.BKCluster constructor is ca

[GitHub] [pulsar] gaozhangmin commented on pull request #13720: fix checkReplication NPE

2022-01-13 Thread GitBox
gaozhangmin commented on pull request #13720: URL: https://github.com/apache/pulsar/pull/13720#issuecomment-1012761401 /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] mattisonchao commented on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012751138 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] lhotari commented on issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
lhotari commented on issue #13750: URL: https://github.com/apache/pulsar/issues/13750#issuecomment-1012743433 Similar problem: https://github.com/apache/pulsar/runs/4811252051?check_suite_focus=true#step:10:576 -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] Anonymitaet commented on pull request #13751: [Docs] Doc updates for PR#12403, PR#13108 and PR#13043

2022-01-13 Thread GitBox
Anonymitaet commented on pull request #13751: URL: https://github.com/apache/pulsar/pull/13751#issuecomment-1012733958 @momo-jun before submitting this PR, did you preview the changes in your local environment? If so, can you add some screenshots of the preview in the PR description? So th

[pulsar] branch master updated: [broker] Optimize TopicPolicies#compactionThreshold with HierarchyTopicPolicies (#13710)

2022-01-13 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 8798392 [broker] Optimize TopicPolicies#compac

[GitHub] [pulsar] codelipenghui merged pull request #13710: [broker] Optimize TopicPolicies#compactionThreshold with HierarchyTopicPolicies

2022-01-13 Thread GitBox
codelipenghui merged pull request #13710: URL: https://github.com/apache/pulsar/pull/13710 -- 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] codelipenghui commented on pull request #13742: [broker] Fix inefficient forEach loop

2022-01-13 Thread GitBox
codelipenghui commented on pull request #13742: URL: https://github.com/apache/pulsar/pull/13742#issuecomment-1012725788 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] codelipenghui closed issue #13698: [branch-2.8] Flaky-test: MultiTopicsReaderTest.testMultiTopic

2022-01-13 Thread GitBox
codelipenghui closed issue #13698: URL: https://github.com/apache/pulsar/issues/13698 -- 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

[pulsar] branch master updated: Fix Flaky test BacklogQuotaManagerTest.testConsumerBacklogEvictionTimeQuotaWithEmptyLedger (#13724)

2022-01-13 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 1fd4490 Fix Flaky test BacklogQuotaManagerTes

[GitHub] [pulsar] codelipenghui closed issue #13697: [branch-2.9]Flaky-test: BacklogQuotaManagerTest.testConsumerBacklogEvictionTimeQuotaWithEmptyLedger

2022-01-13 Thread GitBox
codelipenghui closed issue #13697: URL: https://github.com/apache/pulsar/issues/13697 -- 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] codelipenghui commented on issue #13697: [branch-2.9]Flaky-test: BacklogQuotaManagerTest.testConsumerBacklogEvictionTimeQuotaWithEmptyLedger

2022-01-13 Thread GitBox
codelipenghui commented on issue #13697: URL: https://github.com/apache/pulsar/issues/13697#issuecomment-1012724031 @mattisonchao I noticed @gaozhangmin already fixed the issue. Please check other issues. -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [pulsar] codelipenghui commented on pull request #13724: [Flaky-test] BacklogQuotaManagerTest.testConsumerBacklogEvictionTimeQuotaWithEmptyLedger

2022-01-13 Thread GitBox
codelipenghui commented on pull request #13724: URL: https://github.com/apache/pulsar/pull/13724#issuecomment-1012723771 @gaozhangmin Thanks for the quick fix. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[GitHub] [pulsar] codelipenghui merged pull request #13724: [Flaky-test] BacklogQuotaManagerTest.testConsumerBacklogEvictionTimeQuotaWithEmptyLedger

2022-01-13 Thread GitBox
codelipenghui merged pull request #13724: URL: https://github.com/apache/pulsar/pull/13724 -- 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] Anonymitaet removed a comment on pull request #13751: [Docs] Doc updates for PR#12403, PR#13108 and PR#13043

2022-01-13 Thread GitBox
Anonymitaet removed a comment on pull request #13751: URL: https://github.com/apache/pulsar/pull/13751#issuecomment-1012708537 Add docs for https://github.com/apache/pulsar/pull/13077 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] mattisonchao commented on a change in pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao commented on a change in pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#discussion_r784479738 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -1137,26 +1137,29 @@ private void res

[GitHub] [pulsar] gaozhangmin commented on pull request #13720: fix checkReplication NPE

2022-01-13 Thread GitBox
gaozhangmin commented on pull request #13720: URL: https://github.com/apache/pulsar/pull/13720#issuecomment-1012708682 /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] Anonymitaet commented on pull request #13751: [Docs] Doc updates for PR#12403, PR#13108 and PR#13043

2022-01-13 Thread GitBox
Anonymitaet commented on pull request #13751: URL: https://github.com/apache/pulsar/pull/13751#issuecomment-1012708537 Add docs for https://github.com/apache/pulsar/pull/13077 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [pulsar] gaozhangmin commented on pull request #12025: violation of isolation when using bookie affinity group

2022-01-13 Thread GitBox
gaozhangmin commented on pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#issuecomment-1012707710 /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] gaoran10 commented on a change in pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
gaoran10 commented on a change in pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#discussion_r784475519 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java ## @@ -1137,26 +1137,29 @@ private void resumeA

[GitHub] [pulsar] gaozhangmin commented on issue #13713: [2.7.4] NPE while open reader to pulsar topic

2022-01-13 Thread GitBox
gaozhangmin commented on issue #13713: URL: https://github.com/apache/pulsar/issues/13713#issuecomment-1012704149 > Thanks @gaozhangmin I've missed that! > > But still, I think the NPE is just a result, cause the Bookie's full of errors [about 280 unique ledgers] about the ledger is

[GitHub] [pulsar] nodece commented on pull request #13297: Add schema compatibility strategy on topic level

2022-01-13 Thread GitBox
nodece commented on pull request #13297: URL: https://github.com/apache/pulsar/pull/13297#issuecomment-1012701363 /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

[pulsar] branch master updated (a9f810c -> 528c972)

2022-01-13 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 a9f810c [website][upgrade]feat: team page and some bug fix about version switch (#13731) add 528c972 fix no r

[GitHub] [pulsar] congbobo184 closed issue #13654: No response to client when handleSubscribe because PendingAckHandleImpl init fail

2022-01-13 Thread GitBox
congbobo184 closed issue #13654: URL: https://github.com/apache/pulsar/issues/13654 -- 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-unsubs

[GitHub] [pulsar] congbobo184 merged pull request #13655: fix no response to client when handleSubscribe because PendingAckHandleImpl init fail

2022-01-13 Thread GitBox
congbobo184 merged pull request #13655: URL: https://github.com/apache/pulsar/pull/13655 -- 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 commented on a change in pull request #13481: [Transaction] PendingAckHandleImpl handle isInCacheRequest

2022-01-13 Thread GitBox
congbobo184 commented on a change in pull request #13481: URL: https://github.com/apache/pulsar/pull/13481#discussion_r784461740 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java ## @@ -2324,13 +2324,6 @@ private String tr

[GitHub] [pulsar] mattisonchao commented on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012691423 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] mattisonchao commented on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012689700 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] mattisonchao removed a comment on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao removed a comment on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012688937 /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 a

[GitHub] [pulsar] mattisonchao commented on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012688937 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] mattisonchao removed a comment on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao removed a comment on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012646837 /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 a

[GitHub] [pulsar] dlg99 opened a new pull request #13753: Updated dependencies to get rid of pulsar-io/jdbc related CVE-2020-13692

2022-01-13 Thread GitBox
dlg99 opened a new pull request #13753: URL: https://github.com/apache/pulsar/pull/13753 ### Motivation `mvn clean install verify -Powasp-dependency-check -DskipTests` found various CVEs ### Modifications Updated dependencies to get rid of pulsar-io/jdbc related CVE-202

[GitHub] [pulsar] gaozhangmin commented on pull request #13720: fix checkReplication NPE

2022-01-13 Thread GitBox
gaozhangmin commented on pull request #13720: URL: https://github.com/apache/pulsar/pull/13720#issuecomment-1012684243 /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] gaozhangmin commented on pull request #12025: violation of isolation when using bookie affinity group

2022-01-13 Thread GitBox
gaozhangmin commented on pull request #12025: URL: https://github.com/apache/pulsar/pull/12025#issuecomment-1012683671 /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 abov

[GitHub] [pulsar] nlu90 commented on a change in pull request #13748: fix checkstyle for tired-storage, websocket and bc modules

2022-01-13 Thread GitBox
nlu90 commented on a change in pull request #13748: URL: https://github.com/apache/pulsar/pull/13748#discussion_r784458722 ## File path: pulsar-websocket/src/main/java/org/apache/pulsar/websocket/stats/JvmMetrics.java ## @@ -18,27 +18,21 @@ */ package org.apache.pulsar.webs

[GitHub] [pulsar] Shawyeok commented on pull request #13742: [broker] Fix inefficient forEach loop

2022-01-13 Thread GitBox
Shawyeok commented on pull request #13742: URL: https://github.com/apache/pulsar/pull/13742#issuecomment-1012680345 /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 t

[GitHub] [pulsar] urfreespace opened a new pull request #13752: [website][upgrade]feat: resources page and powered-by pages

2022-01-13 Thread GitBox
urfreespace opened a new pull request #13752: URL: https://github.com/apache/pulsar/pull/13752 *(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 #13751: [Docs] Doc updates for PR#12403, PR#13108 and PR#13043

2022-01-13 Thread GitBox
github-actions[bot] commented on pull request #13751: URL: https://github.com/apache/pulsar/pull/13751#issuecomment-1012677458 @momo-jun: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] momo-jun opened a new pull request #13751: [Docs] Doc updates for PR#12403, PR#13108 and PR#13043

2022-01-13 Thread GitBox
momo-jun opened a new pull request #13751: URL: https://github.com/apache/pulsar/pull/13751 1. PR#12403: restructure and optimize content for message chunking. 2. PR#13108: add description for how to disable threshold check for parameter “loadBalancerNamespaceBundleMaxSessions”. 3. PR

[GitHub] [pulsar] Shoothzj commented on pull request #12430: remove unused filed

2022-01-13 Thread GitBox
Shoothzj commented on pull request #12430: URL: https://github.com/apache/pulsar/pull/12430#issuecomment-1012672676 @eolivelli Please take a look again -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] Shoothzj commented on a change in pull request #13749: Enable CheckStyle Plugin in auth module

2022-01-13 Thread GitBox
Shoothzj commented on a change in pull request #13749: URL: https://github.com/apache/pulsar/pull/13749#discussion_r784448956 ## File path: pulsar-client-auth-athenz/src/main/java/org/apache/pulsar/client/impl/auth/package-info.java ## @@ -0,0 +1,23 @@ +/** + * Licensed to the

[GitHub] [pulsar] codelipenghui opened a new issue #13750: Flaky-test: [EndToEndMetadataTest].[testPublishConsume]

2022-01-13 Thread GitBox
codelipenghui opened a new issue #13750: URL: https://github.com/apache/pulsar/issues/13750 [EndToEndMetadataTest] is flaky. The [testPublishConsume] test method fails sporadically. https://github.com/apache/pulsar/runs/4805692281?check_suite_focus=true ``` Error: Tests ru

[pulsar] branch master updated (2955430 -> a9f810c)

2022-01-13 Thread urfree
This is an automated email from the ASF dual-hosted git repository. urfree pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 2955430 [Transaction] Duplicate TYPE of Prometheus metrics (#13704) add a9f810c [website][upgrade]feat: team p

[GitHub] [pulsar] urfreespace merged pull request #13731: [website][upgrade]feat: team page and some bug fix about version switch

2022-01-13 Thread GitBox
urfreespace merged pull request #13731: URL: https://github.com/apache/pulsar/pull/13731 -- 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] liudezhi2098 opened a new pull request #13749: Enable CheckStyle Plugin in auth module

2022-01-13 Thread GitBox
liudezhi2098 opened a new pull request #13749: URL: https://github.com/apache/pulsar/pull/13749 Motivation enable checkStyle_plugin in pulsar-broker-auth-athen module. enable checkStyle_plugin in pulsar-broker-auth-sasl module. enable checkStyle_plugin in pulsar-client-auth-athe

[GitHub] [pulsar] Shoothzj commented on a change in pull request #13748: fix checkstyle for tired-storage, websocket and bc modules

2022-01-13 Thread GitBox
Shoothzj commented on a change in pull request #13748: URL: https://github.com/apache/pulsar/pull/13748#discussion_r784436624 ## File path: pulsar-websocket/src/main/java/org/apache/pulsar/websocket/stats/JvmMetrics.java ## @@ -18,27 +18,21 @@ */ package org.apache.pulsar.w

[GitHub] [pulsar] dlg99 commented on pull request #13747: Upgraded ElasticSearch to get rid of CVEs.

2022-01-13 Thread GitBox
dlg99 commented on pull request #13747: URL: https://github.com/apache/pulsar/pull/13747#issuecomment-1012652014 /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 the

[GitHub] [pulsar] mattisonchao commented on pull request #13745: Fix call sync method in async rest API for ``internalGetSubscriptionsForNonPartitionedTopics``

2022-01-13 Thread GitBox
mattisonchao commented on pull request #13745: URL: https://github.com/apache/pulsar/pull/13745#issuecomment-1012646837 /pulsarbot rerun-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] nicoloboschi commented on pull request #13746: [security] Offloaders - get rid of several CVEs (including Log4J 2.17)

2022-01-13 Thread GitBox
nicoloboschi commented on pull request #13746: URL: https://github.com/apache/pulsar/pull/13746#issuecomment-1012646578 /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

  1   2   >