[GitHub] [pulsar-client-go] wolfstudy commented on issue #595: panic: runtime error: index out of range [1] with length 1

2021-08-25 Thread GitBox
wolfstudy commented on issue #595: URL: https://github.com/apache/pulsar-client-go/issues/595#issuecomment-905240344 The issue will be fixed in https://github.com/apache/pulsar-client-go/pull/598 -- This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [pulsar-client-go] wolfstudy closed issue #595: panic: runtime error: index out of range [1] with length 1

2021-08-25 Thread GitBox
wolfstudy closed issue #595: URL: https://github.com/apache/pulsar-client-go/issues/595 -- 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-un

[GitHub] [pulsar] lhotari commented on issue #11687: Excessive RAM consumption on kubernetes pulsar bookie pods

2021-08-25 Thread GitBox
lhotari commented on issue #11687: URL: https://github.com/apache/pulsar/issues/11687#issuecomment-905242842 > As far as I know there is no helm charts for 2.7.3 it stops at 2.7.2 right now or maybe I'm not looking where I should be : > https://github.com/apache/pulsar-helm-chart/tags

[GitHub] [pulsar] Anonymitaet commented on pull request #11770: [website] Initial framework for website upgrade

2021-08-25 Thread GitBox
Anonymitaet commented on pull request #11770: URL: https://github.com/apache/pulsar/pull/11770#issuecomment-905243394 @urfreespace many thanks for your contribution! Where are the items in the red box? They are not shown in the preview. ![image](https://user-images.githubusercontent

[GitHub] [pulsar] sunby commented on issue #11768: Failed to restart pulsar standalone

2021-08-25 Thread GitBox
sunby commented on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-905243645 I reproduce it with pulsar 2.8.0. ``` 06:55:34.455 [client-scheduler-OrderedScheduler-2-0] ERROR org.apache.bookkeeper.clients.impl.internal.RootRangeClientImplWithRetries - Re

[GitHub] [pulsar] sunby edited a comment on issue #11768: Failed to restart pulsar standalone

2021-08-25 Thread GitBox
sunby edited a comment on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-905243645 @lhotari I reproduce it with pulsar 2.8.0. ``` 06:55:34.455 [client-scheduler-OrderedScheduler-2-0] ERROR org.apache.bookkeeper.clients.impl.internal.RootRangeClientImplW

[GitHub] [pulsar] lhotari commented on issue #11768: Failed to restart pulsar standalone

2021-08-25 Thread GitBox
lhotari commented on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-905245666 @sunby what happens if you append `--no-stream-storage` to your startup command in docker-compose.yaml ? (port 4181 is the stream storage port, that's why it might change the result

[GitHub] [pulsar] lhotari commented on issue #11772: Open MLPendingAckStore managedLedger failed

2021-08-25 Thread GitBox
lhotari commented on issue #11772: URL: https://github.com/apache/pulsar/issues/11772#issuecomment-905248708 @xuesongxs the description doesn't explain how to reproduce this issue. Please provide a detailed procedure for reproducing the issue. Providing that information will increase the c

[GitHub] [pulsar] lhotari commented on pull request #11762: [C++] Make some clean up methods thread safe

2021-08-25 Thread GitBox
lhotari commented on pull request #11762: URL: https://github.com/apache/pulsar/pull/11762#issuecomment-905250667 Good work @BewareMyPower -- 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 sp

[GitHub] [pulsar] eolivelli commented on a change in pull request #11771: [Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries

2021-08-25 Thread GitBox
eolivelli commented on a change in pull request #11771: URL: https://github.com/apache/pulsar/pull/11771#discussion_r695476681 ## File path: pulsar-metadata/src/test/java/org/apache/pulsar/metadata/BaseMetadataStoreTest.java ## @@ -19,23 +19,26 @@ package org.apache.pulsar.me

[GitHub] [pulsar] codelipenghui merged pull request #11764: Fixed race condition on multi-topic consumer

2021-08-25 Thread GitBox
codelipenghui merged pull request #11764: URL: https://github.com/apache/pulsar/pull/11764 -- 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 (098ba16 -> f1d66d1)

2021-08-25 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 098ba16 [C++] Make some clean up methods thread safe (#11762) add f1d66d1 Fixed race condition on multi-topic

[GitHub] [pulsar] lhotari commented on a change in pull request #11771: [Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries

2021-08-25 Thread GitBox
lhotari commented on a change in pull request #11771: URL: https://github.com/apache/pulsar/pull/11771#discussion_r695485873 ## File path: pulsar-metadata/src/test/java/org/apache/pulsar/metadata/BaseMetadataStoreTest.java ## @@ -19,23 +19,26 @@ package org.apache.pulsar.meta

[GitHub] [pulsar] hangc0276 merged pull request #11761: Forget to update memory usage on message send timeout

2021-08-25 Thread GitBox
hangc0276 merged pull request #11761: URL: https://github.com/apache/pulsar/pull/11761 -- 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

[pulsar] branch master updated: Forget to update memory usage on message send timeout (#11761)

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang 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 1399eeb Forget to update memory usage on mess

[GitHub] [pulsar] leizhiyuan commented on issue #11757: Failed to load an authentication provider java.lang.NullPointerException

2021-08-25 Thread GitBox
leizhiyuan commented on issue #11757: URL: https://github.com/apache/pulsar/issues/11757#issuecomment-905265025 it use openssl apr1 and crypt algorithm set ``` superUser2:$apr1$foobarmq$kuSZlLgOITksCkRgl57ie/ ``` and set ``` brokerClientAuthenticationPl

[GitHub] [pulsar] leizhiyuan edited a comment on issue #11757: Failed to load an authentication provider java.lang.NullPointerException

2021-08-25 Thread GitBox
leizhiyuan edited a comment on issue #11757: URL: https://github.com/apache/pulsar/issues/11757#issuecomment-905265025 it use openssl apr1 and crypt algorithm set ``` superUser2:$apr1$foobarmq$kuSZlLgOITksCkRgl57ie/ ``` and set ``` authenticationEnabled=

[GitHub] [pulsar] sunby commented on issue #11768: Failed to restart pulsar standalone

2021-08-25 Thread GitBox
sunby commented on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-905268372 > @sunby what happens if you append `--no-stream-storage` to your startup command in docker-compose.yaml ? (port 4181 is the stream storage port, that's why it might change the result

[pulsar] branch branch-2.8 updated (3456269 -> 587548e)

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang pushed a change to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 3456269 Fix the topic in fenced state and can not recover. (#11737) new 2acefde fix the bug, can not up

[pulsar] 02/03: Forget to update memory usage on message send timeout (#11761)

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 8ae8f55cec46d9123d9794aa2121020d7726c25e Author: Shoothzj AuthorDate: Wed Aug 25 15:43:57 2021 +0800 Forget t

[pulsar] 01/03: fix the bug, can not update topic when the update topicName is contained by an existed topic as a part (#11686)

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 2acefde47e61efd84bcd381efdfb5f4b95c76265 Author: Nicklee007 <84127069+nicklee...@users.noreply.github.com> AuthorDa

[pulsar] 03/03: Fixed race condition on multi-topic consumer (#11764)

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang pushed a commit to branch branch-2.8 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit 587548ec5f9e9a9fc6d465aca3cd3fbb76b9ee64 Author: Matteo Merli AuthorDate: Wed Aug 25 00:33:35 2021 -0700 Fixe

[GitHub] [pulsar] leizhiyuan opened a new issue #11773: should we allow users use clear password in AuthenticationProviderBasic

2021-08-25 Thread GitBox
leizhiyuan opened a new issue #11773: URL: https://github.com/apache/pulsar/issues/11773 **Is your enhancement request related to a problem? Please describe.** now AuthenticationProviderBasic only support apr1 and crypt. **Describe the solution you'd like** maybe we can al

[GitHub] [pulsar] sunby edited a comment on issue #11768: Failed to restart pulsar standalone

2021-08-25 Thread GitBox
sunby edited a comment on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-905268372 > @sunby what happens if you append `--no-stream-storage` to your startup command in docker-compose.yaml ? (port 4181 is the stream storage port, that's why it might change the

[GitHub] [pulsar-client-go] cimura opened a new pull request #599: Retry assertion several times in consumer regex test

2021-08-25 Thread GitBox
cimura opened a new pull request #599: URL: https://github.com/apache/pulsar-client-go/pull/599 ### Motivation in `consumer_regex_test.go`, sometimes consumers take more than 2 seconds (waiting time) subscribing, and assertions fail. ### Modifications instead of waiting

[GitHub] [pulsar] sunby edited a comment on issue #11768: Failed to restart pulsar standalone

2021-08-25 Thread GitBox
sunby edited a comment on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-905268372 > @sunby what happens if you append `--no-stream-storage` to your startup command in docker-compose.yaml ? (port 4181 is the stream storage port, that's why it might change the

[GitHub] [pulsar] sunby edited a comment on issue #11768: Failed to restart pulsar standalone

2021-08-25 Thread GitBox
sunby edited a comment on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-905268372 > @sunby what happens if you append `--no-stream-storage` to your startup command in docker-compose.yaml ? (port 4181 is the stream storage port, that's why it might change the

[GitHub] [pulsar] sunby edited a comment on issue #11768: Failed to restart pulsar standalone

2021-08-25 Thread GitBox
sunby edited a comment on issue #11768: URL: https://github.com/apache/pulsar/issues/11768#issuecomment-905268372 > @sunby what happens if you append `--no-stream-storage` to your startup command in docker-compose.yaml ? (port 4181 is the stream storage port, that's why it might change the

[GitHub] [pulsar] eolivelli merged pull request #11771: [Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries

2021-08-25 Thread GitBox
eolivelli merged pull request #11771: URL: https://github.com/apache/pulsar/pull/11771 -- 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] Demogorgon314 commented on pull request #11743: [pulsar-client] Fix: set and return topic name on message api

2021-08-25 Thread GitBox
Demogorgon314 commented on pull request #11743: URL: https://github.com/apache/pulsar/pull/11743#issuecomment-905290647 > I agree with @rdhabalia > > @Demogorgon314 do you want to send a PR for the new MessageBuilder API ? > IIUC we have to write up a small PIP for it, as it is a n

[pulsar] branch master updated: [Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries (#11771)

2021-08-25 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.git The following commit(s) were added to refs/heads/master by this push: new 7de9992 [Tests] Use TestRetrySupport for Bas

[GitHub] [pulsar] leizhiyuan commented on issue #11752: ClientBuilder didn't expose requestTimeoutMs setter for ClientConfigurationData

2021-08-25 Thread GitBox
leizhiyuan commented on issue #11752: URL: https://github.com/apache/pulsar/issues/11752#issuecomment-905291112 it use lombok annotation to generate this method. you can invoke ``` ClientConfigurationData conf = new ClientConfigurationData(); conf.setRequestTi

[GitHub] [pulsar] Richard-Yi opened a new issue #11774: Fail to restart broker Due to org.apache.pulsar.client.api.PulsarClientException$BrokerMetadataException

2021-08-25 Thread GitBox
Richard-Yi opened a new issue #11774: URL: https://github.com/apache/pulsar/issues/11774 **Describe the bug** When I restart broker after installing kafka-source plugin into the running broker, the broker restart failed. the error log as belows: ```log 15:56:47.533 [pulsar-clie

[GitHub] [pulsar] eolivelli commented on a change in pull request #11739: Expose compaction metrics to Prometheus

2021-08-25 Thread GitBox
eolivelli commented on a change in pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#discussion_r695532911 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/stats/CompactionMetricsTest.java ## @@ -0,0 +1,96 @@ +/** + * Licensed to the Apach

[pulsar] branch branch-2.8 updated: Make MetadataCacheTest reliable. (#10877)

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang 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 9ea5180 Make MetadataCacheTest reliab

[GitHub] [pulsar] lhotari commented on issue #11774: Fail to restart broker Due to org.apache.pulsar.client.api.PulsarClientException$BrokerMetadataException

2021-08-25 Thread GitBox
lhotari commented on issue #11774: URL: https://github.com/apache/pulsar/issues/11774#issuecomment-905341313 This problem is most likely fixed by #10968 which will be included in Pulsar 2.8.1 . -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [pulsar] lhotari edited a comment on issue #11774: Fail to restart broker Due to org.apache.pulsar.client.api.PulsarClientException$BrokerMetadataException

2021-08-25 Thread GitBox
lhotari edited a comment on issue #11774: URL: https://github.com/apache/pulsar/issues/11774#issuecomment-905341313 This problem is most likely fixed by #10968 which will be included in Pulsar 2.8.1 . Cherry-picked commit in branch-2.8 is 9c3e904a . -- This is an automated message from t

[GitHub] [pulsar] lhotari commented on issue #11774: Fail to restart broker Due to org.apache.pulsar.client.api.PulsarClientException$BrokerMetadataException

2021-08-25 Thread GitBox
lhotari commented on issue #11774: URL: https://github.com/apache/pulsar/issues/11774#issuecomment-905343447 this issue duplicate #10967 . Closing. -- 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] lhotari closed issue #11774: Fail to restart broker Due to org.apache.pulsar.client.api.PulsarClientException$BrokerMetadataException

2021-08-25 Thread GitBox
lhotari closed issue #11774: URL: https://github.com/apache/pulsar/issues/11774 -- 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 edited a comment on issue #11774: Fail to restart broker Due to org.apache.pulsar.client.api.PulsarClientException$BrokerMetadataException

2021-08-25 Thread GitBox
lhotari edited a comment on issue #11774: URL: https://github.com/apache/pulsar/issues/11774#issuecomment-905343447 this issue duplicates #10967 . Closing. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [pulsar] xuesongxs commented on issue #11772: Open MLPendingAckStore managedLedger failed

2021-08-25 Thread GitBox
xuesongxs commented on issue #11772: URL: https://github.com/apache/pulsar/issues/11772#issuecomment-905344059 > @xuesongxs the description doesn't explain how to reproduce this issue. Please provide a detailed procedure for reproducing the issue. Providing that information will increase t

[GitHub] [pulsar] hangc0276 merged pull request #11769: [Branch-2.8] fix com.squareup.okhttp-okhttp-2.7.4.jar unaccounted for in LICENSE bug

2021-08-25 Thread GitBox
hangc0276 merged pull request #11769: URL: https://github.com/apache/pulsar/pull/11769 -- 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

[pulsar] branch branch-2.8 updated: fix com.squareup.okhttp-okhttp-2.7.4.jar unaccounted for in LICENSE bug (#11769)

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang 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 bf95e2e fix com.squareup.okhttp-okhtt

[GitHub] [pulsar] beyondyinjl2 commented on issue #11757: Failed to load an authentication provider java.lang.NullPointerException

2021-08-25 Thread GitBox
beyondyinjl2 commented on issue #11757: URL: https://github.com/apache/pulsar/issues/11757#issuecomment-905358067 Thank you very much, it has been successful. I set pulsar.auth.basic.conf to modify the pulsar_env.sh file. Is there a better way? -- This is an automated message from the Ap

[GitHub] [pulsar] tuteng commented on pull request #11770: [website] Initial framework for website upgrade

2021-08-25 Thread GitBox
tuteng commented on pull request #11770: URL: https://github.com/apache/pulsar/pull/11770#issuecomment-905360084 > @urfreespace many thanks for your contribution! Where are the items in the red box? They are not shown in the preview. > > ![image](https://user-images.githubuserconten

[GitHub] [pulsar] leizhiyuan commented on issue #11757: Failed to load an authentication provider java.lang.NullPointerException

2021-08-25 Thread GitBox
leizhiyuan commented on issue #11757: URL: https://github.com/apache/pulsar/issues/11757#issuecomment-905378434 > Thank you very much, it has been successful. I set pulsar.auth.basic.conf to modify the pulsar_env.sh file. Is there a better way? it is right, I think you change the PUL

[GitHub] [pulsar] Maxzouz commented on issue #11687: Excessive RAM consumption on kubernetes pulsar bookie pods

2021-08-25 Thread GitBox
Maxzouz commented on issue #11687: URL: https://github.com/apache/pulsar/issues/11687#issuecomment-905394448 Thank you. We were planning on using 2.7.2 for now but I will submit the idea of using 2.7.3 images in our chart. > What do you mean with "stateful YAML"? Your values.yaml

[GitHub] [pulsar] Anonymitaet commented on pull request #11770: [website] Initial framework for website upgrade

2021-08-25 Thread GitBox
Anonymitaet commented on pull request #11770: URL: https://github.com/apache/pulsar/pull/11770#issuecomment-905409046 @tuteng OK, I'm fine with this design, thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [pulsar] shibd closed issue #11742: Enhancement message deduplication

2021-08-25 Thread GitBox
shibd closed issue #11742: URL: https://github.com/apache/pulsar/issues/11742 -- 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] Technoboy- commented on a change in pull request #11739: Expose compaction metrics to Prometheus

2021-08-25 Thread GitBox
Technoboy- commented on a change in pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#discussion_r695748030 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/stats/CompactionMetricsTest.java ## @@ -0,0 +1,96 @@ +/** + * Licensed to the Apac

[GitHub] [pulsar] Technoboy- commented on a change in pull request #11739: Expose compaction metrics to Prometheus

2021-08-25 Thread GitBox
Technoboy- commented on a change in pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#discussion_r695747911 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentTopic.java ## @@ -2120,6 +2115,15 @@ public TopicStat

[GitHub] [pulsar] Technoboy- commented on pull request #11739: Expose compaction metrics to Prometheus

2021-08-25 Thread GitBox
Technoboy- commented on pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#issuecomment-905498946 > very good. > > I left one comment about exposing an "Impl" class in PersistentTopic. > > PTAL Thanks for reviewing. -- This is an automated message f

[GitHub] [pulsar] Anonymitaet commented on pull request #11767: [Doc] add notes for connector admin cli

2021-08-25 Thread GitBox
Anonymitaet commented on pull request #11767: URL: https://github.com/apache/pulsar/pull/11767#issuecomment-905515257 @tuteng @nlu90 Thanks for reporting this issue, could you please help review it? Thanks -- This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [pulsar] urfreespace commented on pull request #11770: [website] Initial framework for website upgrade

2021-08-25 Thread GitBox
urfreespace commented on pull request #11770: URL: https://github.com/apache/pulsar/pull/11770#issuecomment-905556889 > > @urfreespace many thanks for your contribution! Where are the items in the red box? They are not shown in the preview. > > ![image](https://user-images.githubusercon

[GitHub] [pulsar-client-go] pkutilina opened a new pull request #600: Fix missing metrics for topics by registration of existing collector

2021-08-25 Thread GitBox
pkutilina opened a new pull request #600: URL: https://github.com/apache/pulsar-client-go/pull/600 ### Motivation It's a bug: when there're several consumers for several topics, builtin pulsar metrics are collected only for first subscribed topic. It happens so because prometheus co

[GitHub] [pulsar] Technoboy- commented on pull request #11739: Expose compaction metrics to Prometheus

2021-08-25 Thread GitBox
Technoboy- commented on pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#issuecomment-905577181 /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] Demogorgon314 removed a comment on issue #11757: Failed to load an authentication provider java.lang.NullPointerException

2021-08-25 Thread GitBox
Demogorgon314 removed a comment on issue #11757: URL: https://github.com/apache/pulsar/issues/11757#issuecomment-904535266 If you want to use `AuthenticationProviderBasic`, you need to set the environment variable `pulsar.auth.basic.conf` to specify the basic auth users config path.

[GitHub] [pulsar] yuruguo opened a new issue #11775: [pulsar-testclient] --subscriptions size is not equal to --num-subscriptions can cause IndexOutOfBoundsException in consumption

2021-08-25 Thread GitBox
yuruguo opened a new issue #11775: URL: https://github.com/apache/pulsar/issues/11775 **Describe the bug** --subscriptions size is not equal to --num-subscriptions can cause IndexOutOfBoundsException when execute `bin/pulsar-perf consume` **To Reproduce** Steps to reproduce the

[GitHub] [pulsar] dlg99 commented on a change in pull request #11520: Debezium Oracle Source

2021-08-25 Thread GitBox
dlg99 commented on a change in pull request #11520: URL: https://github.com/apache/pulsar/pull/11520#discussion_r695920596 ## File path: pulsar-io/debezium/oracle/src/main/resources/debezium-oracle-source-config.yaml ## @@ -0,0 +1,38 @@ +# +# Licensed to the Apache Software Fo

[GitHub] [pulsar] dlg99 commented on a change in pull request #11520: Debezium Oracle Source

2021-08-25 Thread GitBox
dlg99 commented on a change in pull request #11520: URL: https://github.com/apache/pulsar/pull/11520#discussion_r695920771 ## File path: pulsar-io/debezium/oracle/src/main/resources/debezium-oracle-source-config.yaml ## @@ -0,0 +1,38 @@ +# +# Licensed to the Apache Software Fo

[GitHub] [pulsar] michaeljmarshall opened a new pull request #11776: [Broker] Remove unnecessary futures; fix comment

2021-08-25 Thread GitBox
michaeljmarshall opened a new pull request #11776: URL: https://github.com/apache/pulsar/pull/11776 ### Motivation In reading through the `ServerCnx` class, I noticed that there was some code that could be simplified. ### Modifications * Remove unreachable log statemen

[GitHub] [pulsar-client-go] cckellogg commented on issue #597: Ack batch msgs

2021-08-25 Thread GitBox
cckellogg commented on issue #597: URL: https://github.com/apache/pulsar-client-go/issues/597#issuecomment-905698558 I'm a little confused on what the issue is. Are you suggesting there should be a way to ack all messages within a batch without having to ack all messages within that batch

[GitHub] [pulsar] lhotari merged pull request #11706: [Functions] Support KEY_BASED batch builder for Java based functions and sources

2021-08-25 Thread GitBox
lhotari merged pull request #11706: URL: https://github.com/apache/pulsar/pull/11706 -- 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: [Functions] Support KEY_BASED batch builder for Java based functions and sources (#11706)

2021-08-25 Thread lhotari
This is an automated email from the ASF dual-hosted git repository. lhotari 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 b923af1 [Functions] Support KEY_BASED batch bu

[GitHub] [pulsar] michaeljmarshall opened a new pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-25 Thread GitBox
michaeljmarshall opened a new pull request #11777: URL: https://github.com/apache/pulsar/pull/11777 ### Motivation The AuthenticationService essentially implements the `canLookupAsync` method instead of relying on the provider's `canLookupAsync` method. Before this change, the

[GitHub] [pulsar-client-go] merlimat merged pull request #600: Fix missing metrics for topics by registration of existing collector

2021-08-25 Thread GitBox
merlimat merged pull request #600: URL: https://github.com/apache/pulsar-client-go/pull/600 -- 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: commit

[pulsar-client-go] branch master updated: Fix missing metrics for topics by registration of existing collector (#600)

2021-08-25 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 1a3ad70 Fix missing metrics for topic

[GitHub] [pulsar] lhotari commented on issue #11690: Flaky-test: LockManagerTest.revalidateLockOnDifferentSession

2021-08-25 Thread GitBox
lhotari commented on issue #11690: URL: https://github.com/apache/pulsar/issues/11690#issuecomment-905766899 the solution in #11771 doesn't seem to help. The issue persists. Here's logs from a failed test run: https://gist.github.com/lhotari/5f0176f52a409795ae9134c34efc0f7a . There

[GitHub] [pulsar-client-go] jdkuki commented on a change in pull request #238: Implement TLS VerifyPeerCertificate callback to skip hostname verfication

2021-08-25 Thread GitBox
jdkuki commented on a change in pull request #238: URL: https://github.com/apache/pulsar-client-go/pull/238#discussion_r696050924 ## File path: pulsar/internal/connection.go ## @@ -711,8 +711,46 @@ func (c *connection) getTLSConfig() (*tls.Config, error) { }

[GitHub] [pulsar] lhotari opened a new pull request #11778: [Tests] Fix BaseMetadataStoreTest retries

2021-08-25 Thread GitBox
lhotari opened a new pull request #11778: URL: https://github.com/apache/pulsar/pull/11778 ### Motivation - A `Supplier` must be used for the Zookeeper connection string parameters passed in a TestNG `DataProvider`. - The retried test run will use the same arguments as the fai

[GitHub] [pulsar] lhotari commented on issue #11690: Flaky-test: LockManagerTest.revalidateLockOnDifferentSession

2021-08-25 Thread GitBox
lhotari commented on issue #11690: URL: https://github.com/apache/pulsar/issues/11690#issuecomment-905815283 #11778 mitigates the issue with retries. -- 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] lhotari commented on pull request #11771: [Tests] Use TestRetrySupport for BaseMetadataStoreTests to cleanup state between retries

2021-08-25 Thread GitBox
lhotari commented on pull request #11771: URL: https://github.com/apache/pulsar/pull/11771#issuecomment-905815661 There's an additional change #11778 which is needed for making the test reties work for BaseMetadataStoreTests. -- This is an automated message from the Apache Git Service. T

[GitHub] [pulsar] zbentley opened a new issue #11779: Deleting tenants with active namespaces yields a 500 and stacktrace, not 409 or 405

2021-08-25 Thread GitBox
zbentley opened a new issue #11779: URL: https://github.com/apache/pulsar/issues/11779 **Describe the bug** If I issue an HTTP DELETE for a tenant which has one or more active namespaces, the API docs indicate that I should get either a 405 (`Broker doesn't allow forced deletion of tena

[GitHub] [pulsar] geomagilles opened a new issue #11780: producer.sendAsync is interrupted by client.close()

2021-08-25 Thread GitBox
geomagilles opened a new issue #11780: URL: https://github.com/apache/pulsar/issues/11780 **Describe the bug** If the client closes while sendAsync() is processed, the message is lost. **To Reproduce** see https://github.com/infiniticio/pulsar-bug-clientAsync **Expected

[GitHub] [pulsar] michaeljmarshall commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-25 Thread GitBox
michaeljmarshall commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-905893898 /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] Anonymitaet commented on pull request #11777: [Authorization] AuthorizationService should use provider's canLookupAsync method

2021-08-25 Thread GitBox
Anonymitaet commented on pull request #11777: URL: https://github.com/apache/pulsar/pull/11777#issuecomment-905970769 @michaeljmarshall Thanks for your contribution. Please do not forget to update docs later. And you can ping me to review the docs, thanks. -- This is an automated message

[pulsar] branch branch-2.8 updated: Release 2.8.1

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang 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 694ebb7 Release 2.8.1 694ebb7 is desc

[GitHub] [pulsar] freeznet opened a new issue #11781: [package management service] inequality file issue when having large file content

2021-08-25 Thread GitBox
freeznet opened a new issue #11781: URL: https://github.com/apache/pulsar/issues/11781 **Describe the bug** When uploading a large file to the package management service, the downloaded file is not the same file when uploaded. The downloaded file has the same size as the uploaded file,

[pulsar] annotated tag v2.8.1-candidate-1 updated (694ebb7 -> c799757)

2021-08-25 Thread chenhang
This is an automated email from the ASF dual-hosted git repository. chenhang pushed a change to annotated tag v2.8.1-candidate-1 in repository https://gitbox.apache.org/repos/asf/pulsar.git. *** WARNING: tag v2.8.1-candidate-1 was modified! *** from 694ebb7 (commit) to c799757 (tag)

[GitHub] [pulsar] freeznet opened a new pull request #11782: [package management service] fix `wrappedBuffer` always having the same block of memory

2021-08-25 Thread GitBox
freeznet opened a new pull request #11782: URL: https://github.com/apache/pulsar/pull/11782 Fixes #11781 ### Motivation When uploading file content to bk, the `BookKeeperPackagesStorage` always use the same block of memory to read the file content and build up a bunch of zero

[GitHub] [pulsar] Shoothzj opened a new issue #11783: ConcurrentModificationException occurs when failPendingMessages

2021-08-25 Thread GitBox
Shoothzj opened a new issue #11783: URL: https://github.com/apache/pulsar/issues/11783 ### Describe the bug ![image](https://user-images.githubusercontent.com/12933197/130885099-ecb22a9c-5422-4057-9f00-d85d1c24b84b.png) ### To Reproduce It's hard to reproduce. ### Expect

[GitHub] [pulsar] Technoboy- commented on pull request #11739: Expose compaction metrics to Prometheus

2021-08-25 Thread GitBox
Technoboy- commented on pull request #11739: URL: https://github.com/apache/pulsar/pull/11739#issuecomment-906013201 /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] BewareMyPower closed issue #8357: [docs] Add doc on how to use batching for key_shared subscription

2021-08-25 Thread GitBox
BewareMyPower closed issue #8357: URL: https://github.com/apache/pulsar/issues/8357 -- 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] BewareMyPower merged pull request #11720: [docs] [ISSUE 8357] Add description and examples for key-based batching for Key_Shared subscription mode

2021-08-25 Thread GitBox
BewareMyPower merged pull request #11720: URL: https://github.com/apache/pulsar/pull/11720 -- 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 (b923af1 -> 35a631d)

2021-08-25 Thread xyz
This is an automated email from the ASF dual-hosted git repository. xyz pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from b923af1 [Functions] Support KEY_BASED batch builder for Java based functions and sources (#11706) add 35a631d [d

[GitHub] [pulsar] Shoothzj opened a new pull request #11784: Remove the uncorrect VisableTesting annotation in pulsar-client

2021-08-25 Thread GitBox
Shoothzj opened a new pull request #11784: URL: https://github.com/apache/pulsar/pull/11784 Those method are not used only by test code. Keep the annotation makes people confused. ### Documentation It's an internal change, so we don't need doc. -- This is an autom

[pulsar] 01/01: [Doc] create doc issue template

2021-08-25 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu pushed a commit to branch Anonymitaet-patch-2 in repository https://gitbox.apache.org/repos/asf/pulsar.git commit fb753a6f15ad02904937755f4523751aa6153c3e Author: Anonymitaet <50226895+anonymit...@users.noreply.github.com>

[pulsar] branch Anonymitaet-patch-2 created (now fb753a6)

2021-08-25 Thread liuyu
This is an automated email from the ASF dual-hosted git repository. liuyu pushed a change to branch Anonymitaet-patch-2 in repository https://gitbox.apache.org/repos/asf/pulsar.git. at fb753a6 [Doc] create doc issue template This branch includes the following new commits: new fb753

[GitHub] [pulsar] Anonymitaet commented on pull request #11785: [Doc] create doc issue template

2021-08-25 Thread GitBox
Anonymitaet commented on pull request #11785: URL: https://github.com/apache/pulsar/pull/11785#issuecomment-906021017 @D-2-Ed could you please help review? Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [pulsar] Jennifer88huang commented on issue #7663: doc for deploying python functions with custom dependencies not explained in 2.6.0

2021-08-25 Thread GitBox
Jennifer88huang commented on issue #7663: URL: https://github.com/apache/pulsar/issues/7663#issuecomment-906021677 @sergescevenels-naspers If the [content here](https://pulsar.apache.org/docs/en/next/functions-package/#python) meet your requirements, feel free to close the issue. If you

[GitHub] [pulsar] Anonymitaet removed a comment on pull request #11785: [Doc] create doc issue template

2021-08-25 Thread GitBox
Anonymitaet removed a comment on pull request #11785: URL: https://github.com/apache/pulsar/pull/11785#issuecomment-906021017 @D-2-Ed could you please help review? Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [pulsar] Anonymitaet closed pull request #11785: [Doc] create doc issue template

2021-08-25 Thread GitBox
Anonymitaet closed pull request #11785: URL: https://github.com/apache/pulsar/pull/11785 -- 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] zymap commented on a change in pull request #11782: [package management service] fix `wrappedBuffer` always using the same block of memory

2021-08-25 Thread GitBox
zymap commented on a change in pull request #11782: URL: https://github.com/apache/pulsar/pull/11782#discussion_r696244831 ## File path: pulsar-package-management/bookkeeper-storage/src/main/java/org/apache/pulsar/packages/management/storage/bookkeeper/DLOutputStream.java ## @

[GitHub] [pulsar] Anonymitaet commented on pull request #11786: [Doc Workflow] add doc issue template and optimize PR template

2021-08-25 Thread GitBox
Anonymitaet commented on pull request #11786: URL: https://github.com/apache/pulsar/pull/11786#issuecomment-906040575 @D-2-Ed could you please help review? Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [pulsar] oversearch opened a new issue #11787: Question: copy or move topics between Pulsar clusters

2021-08-25 Thread GitBox
oversearch opened a new issue #11787: URL: https://github.com/apache/pulsar/issues/11787 **Is your feature request related to a problem? Please describe.** I'm wondering if it's currently possible to copy or move a topic that exists in one Pulsar cluster to another independent cluster?

[GitHub] [pulsar] 315157973 opened a new pull request #11788: Fix ConcurrentModificationException occurs when failPendingMessages

2021-08-25 Thread GitBox
315157973 opened a new pull request #11788: URL: https://github.com/apache/pulsar/pull/11788 Fixes #11783 ### Motivation Looks like this issue was introduced by #9650 ### Modifications pendingMessages is a very frequently used queue. In order to avoid GC, I used `ArrayBl

[GitHub] [pulsar-client-go] wolfstudy opened a new pull request #601: Fix panic when scale down partitions

2021-08-25 Thread GitBox
wolfstudy opened a new pull request #601: URL: https://github.com/apache/pulsar-client-go/pull/601 Signed-off-by: xiaolongran ### Motivation When the program is running, if the business is forced to delete certain sub partitions, the following error message will be ca

[GitHub] [pulsar-client-go] wolfstudy commented on pull request #601: Fix panic when scale down partitions

2021-08-25 Thread GitBox
wolfstudy commented on pull request #601: URL: https://github.com/apache/pulsar-client-go/pull/601#issuecomment-906064881 @merlimat @cckellogg @zymap PTAL thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [pulsar] beyondyinjl2 closed issue #11757: Failed to load an authentication provider java.lang.NullPointerException

2021-08-25 Thread GitBox
beyondyinjl2 closed issue #11757: URL: https://github.com/apache/pulsar/issues/11757 -- 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] hangc0276 opened a new issue #11789: java.lang.NoSuchMethodError: java.nio.ByteBuffer.position(I)Ljava/nio/ByteBuffer

2021-08-25 Thread GitBox
hangc0276 opened a new issue #11789: URL: https://github.com/apache/pulsar/issues/11789 **Describe the bug** When create a function, the server side throws NosuchMethodError exception ``` Caused by: java.lang.NoSuchMethodError: java.nio.ByteBuffer.position(I)Ljava/nio/ByteBuffer;

  1   2   >