[GitHub] [pulsar] equanz commented on pull request #6996: [pulsar-client] Fix incorrect partitioned producer stats

2020-05-27 Thread GitBox
equanz commented on pull request #6996: URL: https://github.com/apache/pulsar/pull/6996#issuecomment-635122287 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] aloyszhang opened a new pull request #7079: fix typos in java doc

2020-05-27 Thread GitBox
aloyszhang opened a new pull request #7079: URL: https://github.com/apache/pulsar/pull/7079 ### Motivation fix typos in java doc This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] aloyszhang opened a new pull request #7078: introduce precise topic publish rate limiting

2020-05-27 Thread GitBox
aloyszhang opened a new pull request #7078: URL: https://github.com/apache/pulsar/pull/7078 Fixes #6975 Master Issue: #6975 ### Motivation Enable precise topic publish rate limit on broker. Now pulsar limits publish rate of topic by a period task runs

[GitHub] [pulsar] codelipenghui commented on pull request #6052: [PIP-54] Support acknowledgment at batch index level

2020-05-27 Thread GitBox
codelipenghui commented on pull request #6052: URL: https://github.com/apache/pulsar/pull/6052#issuecomment-635104385 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] codelipenghui commented on pull request #6791: Use consistent hashing in KeyShared distribution

2020-05-27 Thread GitBox
codelipenghui commented on pull request #6791: URL: https://github.com/apache/pulsar/pull/6791#issuecomment-635104421 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] merlimat opened a new pull request #7077: Fixed increasing number of partitions with attached readers

2020-05-27 Thread GitBox
merlimat opened a new pull request #7077: URL: https://github.com/apache/pulsar/pull/7077 ### Motivation When we're increasing the number of partitions in a topic, we typically also pre-create the subscriptions on those topics. In this case, we're also pre-creating the non-durable

[pulsar] branch master updated: Fixed updating partitioned topic without subscriptions (#7064)

2020-05-27 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.git The following commit(s) were added to refs/heads/master by this push: new 55f7fd6 Fixed updating partitioned topic

[GitHub] [pulsar] merlimat merged pull request #7064: Added test for updating partitioned topic without subscriptions

2020-05-27 Thread GitBox
merlimat merged pull request #7064: URL: https://github.com/apache/pulsar/pull/7064 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] ckdarby commented on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ckdarby commented on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-635081278 @ashwallace Thanks for everything. I'll nuke the cluster & change the type with a baseline higher and let you know. Here are my other comments... >In your graphs, bookie

[GitHub] [pulsar] equanz commented on pull request #6996: [pulsar-client] Fix incorrect partitioned producer stats

2020-05-27 Thread GitBox
equanz commented on pull request #6996: URL: https://github.com/apache/pulsar/pull/6996#issuecomment-635076014 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] codelipenghui commented on pull request #6052: [PIP-54] Support acknowledgment at batch index level

2020-05-27 Thread GitBox
codelipenghui commented on pull request #6052: URL: https://github.com/apache/pulsar/pull/6052#issuecomment-635075286 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] DGBStarSky opened a new issue #7076: Pulsar SQL cannot get the latest message

2020-05-27 Thread GitBox
DGBStarSky opened a new issue #7076: URL: https://github.com/apache/pulsar/issues/7076 **Describe the bug** Pulsar SQL cannot get the latest message of topic. **To Reproduce** Steps to reproduce the behavior: I use pulsar sql query messages, but cannot get the latest

[GitHub] [pulsar] DGBStarSky closed issue #7076: Pulsar SQL cannot get the latest message

2020-05-27 Thread GitBox
DGBStarSky closed issue #7076: URL: https://github.com/apache/pulsar/issues/7076 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] ashwallace commented on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ashwallace commented on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-635066612 > > > @ashwallace But that doesn't explain why when on the pod itself using `dd` I'm able to access the EBS at 200 mbyte/s. > > I will change the cluster to one that

[GitHub] [pulsar] gh777777 commented on issue #4937: Retrieve schema instance from schema info for type 'NONE'

2020-05-27 Thread GitBox
gh77 commented on issue #4937: URL: https://github.com/apache/pulsar/issues/4937#issuecomment-635062518 > @gh77 what version of python function are you using? Sorry to reply so late. I use python 3.8.2. Now the issue has gone. I suspect the problem has to do with

[GitHub] [pulsar] merlimat opened a new pull request #7075: Fix flaky ReplicatorTest.testUpdateGlobalTopicPartition

2020-05-27 Thread GitBox
merlimat opened a new pull request #7075: URL: https://github.com/apache/pulsar/pull/7075 ### Motivation The test has been failing frequently in CI with this error: ``` [ERROR] testUpdateGlobalTopicPartition(org.apache.pulsar.broker.service.ReplicatorTest) Time elapsed:

[GitHub] [pulsar] codelipenghui commented on pull request #6052: [PIP-54] Support acknowledgment at batch index level

2020-05-27 Thread GitBox
codelipenghui commented on pull request #6052: URL: https://github.com/apache/pulsar/pull/6052#issuecomment-635059385 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[pulsar] branch master updated (adf650e -> afd3296)

2020-05-27 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from adf650e fixes RESTAPIs and Cli button not working. (#7038) add afd3296 Fix the flaky function test.

[GitHub] [pulsar] srkukarni merged pull request #7057: Fix the flaky function test.

2020-05-27 Thread GitBox
srkukarni merged pull request #7057: URL: https://github.com/apache/pulsar/pull/7057 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] ckdarby edited a comment on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ckdarby edited a comment on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-635048127 @ashwallace But that doesn't explain why when on the pod itself using `dd` I'm able to access the EBS at 200 mbyte/s. I will change the cluster to one that has

[GitHub] [pulsar] ckdarby commented on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ckdarby commented on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-635048127 @ashwallace But that doesn't explain why when on the pod itself using `dd` I'm able to access the EBS at 200 mbyte/s. I will change the cluster to one that has throughput has

[GitHub] [pulsar] merlimat opened a new pull request #7074: Allow to grant permissions when the authorization is disabled

2020-05-27 Thread GitBox
merlimat opened a new pull request #7074: URL: https://github.com/apache/pulsar/pull/7074 ### Motivation Currently if authorization is not enforced, the AuthZ plugin is not loaded at all. That makes it impossible to switch a cluster from no-authz to authz-enforced with no downtime.

[GitHub] [pulsar] sijie commented on issue #7056: pulsar client 2.5.2 cli does not support subscription-position when running consumer

2020-05-27 Thread GitBox
sijie commented on issue #7056: URL: https://github.com/apache/pulsar/issues/7056#issuecomment-635043934 @sakthi-prasad I think the documentation of release 2.5.2 was cut by using the latest master. @jiazhai is looking into fixing the issue.

[GitHub] [pulsar-client-go] wolfstudy merged pull request #263: [Issue 240] Add check for max message size

2020-05-27 Thread GitBox
wolfstudy merged pull request #263: URL: https://github.com/apache/pulsar-client-go/pull/263 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [pulsar-client-go] wolfstudy closed issue #240: Check max message size in go client

2020-05-27 Thread GitBox
wolfstudy closed issue #240: URL: https://github.com/apache/pulsar-client-go/issues/240 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

[pulsar-client-go] branch master updated: [Issue 240] Add check for max message size (#263)

2020-05-27 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new bf248fd [Issue 240] Add check for max

[GitHub] [pulsar] sijie commented on issue #7055: Unable to Produce Messages: Topic is stuck on a ledger that doesn't exist: "No such ledger exists on Metadata Server"

2020-05-27 Thread GitBox
sijie commented on issue #7055: URL: https://github.com/apache/pulsar/issues/7055#issuecomment-635041688 @Saswatibhoi How do you deploy your pulsar cluster? Did you happen to run any bookkeeper commands like `metaformat` to format the bookkeeper metadata?

[GitHub] [pulsar-client-go] wolfstudy commented on issue #264: [Improve] Unified processing of error information in the project

2020-05-27 Thread GitBox
wolfstudy commented on issue #264: URL: https://github.com/apache/pulsar-client-go/issues/264#issuecomment-635039803 @keithnull will track this task This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] merlimat opened a new pull request #7073: Fix race in MessageDispatchThrottlingTest

2020-05-27 Thread GitBox
merlimat opened a new pull request #7073: URL: https://github.com/apache/pulsar/pull/7073 ### Motivation There was a race in testRelativeMessageRateLimitingThrottling where the test was sending and receiving a message to warmup the broker, and then testing whether messages

[GitHub] [pulsar] equanz commented on pull request #6996: [pulsar-client] Fix incorrect partitioned producer stats

2020-05-27 Thread GitBox
equanz commented on pull request #6996: URL: https://github.com/apache/pulsar/pull/6996#issuecomment-635038553 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar-client-go] wolfstudy commented on a change in pull request #263: [Issue 240] Add check for max message size

2020-05-27 Thread GitBox
wolfstudy commented on a change in pull request #263: URL: https://github.com/apache/pulsar-client-go/pull/263#discussion_r431529838 ## File path: pulsar/producer_partition.go ## @@ -236,6 +239,16 @@ func (p *partitionProducer) internalSend(request *sendRequest) {

[GitHub] [pulsar] merlimat opened a new pull request #7072: Clear backlog quota before test in ReplicatorTest

2020-05-27 Thread GitBox
merlimat opened a new pull request #7072: URL: https://github.com/apache/pulsar/pull/7072 ### Motivation All tests in Replicator test use the same instance of pulsar and same namespaces. Depending on the order the tests run in, which is somewhat random, the backlog quota may be

[GitHub] [pulsar-client-go] wolfstudy commented on a change in pull request #263: [Issue 240] Add check for max message size

2020-05-27 Thread GitBox
wolfstudy commented on a change in pull request #263: URL: https://github.com/apache/pulsar-client-go/pull/263#discussion_r431529838 ## File path: pulsar/producer_partition.go ## @@ -236,6 +239,16 @@ func (p *partitionProducer) internalSend(request *sendRequest) {

[GitHub] [pulsar-client-go] wolfstudy commented on a change in pull request #263: [Issue 240] Add check for max message size

2020-05-27 Thread GitBox
wolfstudy commented on a change in pull request #263: URL: https://github.com/apache/pulsar-client-go/pull/263#discussion_r431529838 ## File path: pulsar/producer_partition.go ## @@ -236,6 +239,16 @@ func (p *partitionProducer) internalSend(request *sendRequest) {

[GitHub] [pulsar-client-go] wolfstudy opened a new issue #264: [Improve] Unified processing of error information in the project

2020-05-27 Thread GitBox
wolfstudy opened a new issue #264: URL: https://github.com/apache/pulsar-client-go/issues/264 Currently, there are some inconsistencies in the handling of errors. In order to ensure the structured output of the log, we should use withError() instead of Error(). E.g: ```

[GitHub] [pulsar] merlimat opened a new pull request #7071: C++ tests should use sensible timeouts for receive

2020-05-27 Thread GitBox
merlimat opened a new pull request #7071: URL: https://github.com/apache/pulsar/pull/7071 ### Motivation A bunch of tests were using 100ms as the timeout for receive. 100ms is way too short. The broker for these tests is standalone, so everything is running in one process on

[GitHub] [pulsar-client-go] wolfstudy commented on a change in pull request #263: [Issue 240] Add check for max message size

2020-05-27 Thread GitBox
wolfstudy commented on a change in pull request #263: URL: https://github.com/apache/pulsar-client-go/pull/263#discussion_r431529838 ## File path: pulsar/producer_partition.go ## @@ -236,6 +239,16 @@ func (p *partitionProducer) internalSend(request *sendRequest) {

[GitHub] [pulsar] merlimat opened a new pull request #7070: [C++] PIP-55: Refresh Authentication Credentials

2020-05-27 Thread GitBox
merlimat opened a new pull request #7070: URL: https://github.com/apache/pulsar/pull/7070 ### Motivation This adds the support for refreshing the auth credentials in C++ clients for PIP-55: https://github.com/apache/pulsar/wiki/PIP-55%3A-Refresh-Authentication-Credentials .

[GitHub] [pulsar] merlimat opened a new pull request #7069: Allow BasicEndToEndTest to run multiple times locally

2020-05-27 Thread GitBox
merlimat opened a new pull request #7069: URL: https://github.com/apache/pulsar/pull/7069 ### Motivation The topics used in a bunch of the cases in `BasicEndToEndTest` had no unique identifier, so if you ran it multiple times against the same standalone cluster, they would

[GitHub] [pulsar] merlimat opened a new pull request #7068: [C++] ClientImpl::handleClose shouldn't use statics

2020-05-27 Thread GitBox
merlimat opened a new pull request #7068: URL: https://github.com/apache/pulsar/pull/7068 ### Motivation `ClientImpl::handleClose` was using static variables to record the first error on closing a client. This is just wrong. A static stack variable in c++ acts like a global. So

[GitHub] [pulsar] merlimat opened a new pull request #7067: Brokers must advertize on localhost to avoid problems creating client

2020-05-27 Thread GitBox
merlimat opened a new pull request #7067: URL: https://github.com/apache/pulsar/pull/7067 ### Motivation This works around the issue where if the hostname of the machine has an underscore, the test will fail to run

[GitHub] [pulsar] ashwallace edited a comment on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ashwallace edited a comment on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-635024332 @ckdarby @sijie EC2 r5.large has a baseline EBS throughput of ~81 MB/s at 128 KB IO sizes. Looks like you're at around that number more or less. Because of this fact,

[GitHub] [pulsar] ashwallace edited a comment on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ashwallace edited a comment on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-635024332 @ckdarby @sijie EC2 r5.large has a baseline EBS throughput of ~81 MB/s at 128 KB IO sizes. Looks like you're at around that number more or less. Because of this fact,

[GitHub] [pulsar] merlimat opened a new pull request #7066: Don't run Runtime#halt in tests

2020-05-27 Thread GitBox
merlimat opened a new pull request #7066: URL: https://github.com/apache/pulsar/pull/7066 ### Motivation When you trigger Runtime#halt from a test, as was happening on ZK session loss, it kills the test without reporting a result. We shouldn't do that. Instead, reserve halt for

[GitHub] [pulsar] ashwallace commented on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ashwallace commented on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-635024332 EC2 r5.large has a baseline EBS throughput of ~81 MB/s at 128 KB IO sizes. Looks like you're at around that number more or less. Because of this fact, you may not be a benefit

[GitHub] [pulsar] merlimat opened a new pull request #7065: Remove the bundle load data info before performing the split

2020-05-27 Thread GitBox
merlimat opened a new pull request #7065: URL: https://github.com/apache/pulsar/pull/7065 ### Motivation When a namespace bundle split operation fails by timeout, we're not cleaning up the current in-memory metadata of the old bundles. This lead to the broker to treat the operation

[GitHub] [pulsar-client-go] keithnull commented on issue #231: Race in partitionProducer Send on err access

2020-05-27 Thread GitBox
keithnull commented on issue #231: URL: https://github.com/apache/pulsar-client-go/issues/231#issuecomment-635021070 Seems this has already been fixed by #229? This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] zhanghaou commented on issue #7049: Make WebSocketService.MaxTextFrameSize configurable

2020-05-27 Thread GitBox
zhanghaou commented on issue #7049: URL: https://github.com/apache/pulsar/issues/7049#issuecomment-635020327 Thanks for your feedback, I will fix it. This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] merlimat opened a new pull request #7064: Added test for updating partitioned topic without subscriptions

2020-05-27 Thread GitBox
merlimat opened a new pull request #7064: URL: https://github.com/apache/pulsar/pull/7064 ### Motivation The underlying issue was already fixed in master, though currently there is no test to validate that we can increase partitions on a topic without subscriptions.

[GitHub] [pulsar] merlimat opened a new pull request #7063: Remove double @Test annotations

2020-05-27 Thread GitBox
merlimat opened a new pull request #7063: URL: https://github.com/apache/pulsar/pull/7063 ### Motivation Some tests had @Test at the class level and also at the method level. This resulted in these tests running twice.

[GitHub] [pulsar] merlimat opened a new pull request #7062: Regex consumer shared queue should be set to configured value

2020-05-27 Thread GitBox
merlimat opened a new pull request #7062: URL: https://github.com/apache/pulsar/pull/7062 ### Motivation C++ consumers are using a queue size of 1000 when consuming from multiple topics (eg: regex), irrespectively of the setting configured in the `ConsumerConfiguration` that is

[GitHub] [pulsar] merlimat opened a new pull request #7061: Don't assume that we are running in docker for py instance tests

2020-05-27 Thread GitBox
merlimat opened a new pull request #7061: URL: https://github.com/apache/pulsar/pull/7061 ### Motivation The run-unit-tests.sh assumed that the build tree was at /pulsar as it is in the docker based build. This isn't necessarily the case when the script is being called

[GitHub] [pulsar] merlimat opened a new pull request #7060: Allow managed cursor to specify max read batch in bytes

2020-05-27 Thread GitBox
merlimat opened a new pull request #7060: URL: https://github.com/apache/pulsar/pull/7060 ### Motivation In order to have a predictable memory footprint in brokers, we need to cap the amount of bytes that we read in single range read request from BK. Since BK doesn't take a

[GitHub] [pulsar] ckdarby commented on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ckdarby commented on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-635011079 Talked to @cdbartholomew today and he mentioned that he might be having what appears to be the same issue, but in Azure. Read performance is nowhere close to the disk

[GitHub] [pulsar] merlimat commented on pull request #6791: Use consistent hashing in KeyShared distribution

2020-05-27 Thread GitBox
merlimat commented on pull request #6791: URL: https://github.com/apache/pulsar/pull/6791#issuecomment-635000206 @sijie @codelipenghui Added configuration switch for consistent hashing, by default turned off. PTAL. This is

[GitHub] [pulsar] irairdon opened a new issue #7059: Pulsar Producing Hangs when Kubernertes Node Restarts

2020-05-27 Thread GitBox
irairdon opened a new issue #7059: URL: https://github.com/apache/pulsar/issues/7059 When running pulsar on Kubernetes if you reboot a Kubernetes node any client producing will hand and be unable to produce new messages. **To Reproduce** Steps to reproduce the behavior: 1.

[GitHub] [pulsar] sijie closed issue #7002: There are invalid links in the document

2020-05-27 Thread GitBox
sijie closed issue #7002: URL: https://github.com/apache/pulsar/issues/7002 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

[pulsar] branch master updated: fixes RESTAPIs and Cli button not working. (#7038)

2020-05-27 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie 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 adf650e fixes RESTAPIs and Cli button not

[GitHub] [pulsar] sijie merged pull request #7038: [website][issue7002]fixes RESTAPIs and Cli button not working.

2020-05-27 Thread GitBox
sijie merged pull request #7038: URL: https://github.com/apache/pulsar/pull/7038 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] ckdarby commented on issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ckdarby commented on issue #7058: URL: https://github.com/apache/pulsar/issues/7058#issuecomment-634959473 @sijie I was finally able to compile the issue that I mentioned on Slack. This is an automated message from the

[GitHub] [pulsar] ckdarby opened a new issue #7058: Pulsar on EBS having poor performance

2020-05-27 Thread GitBox
ckdarby opened a new issue #7058: URL: https://github.com/apache/pulsar/issues/7058 **Describe the bug** A clear and concise description of what the bug is. **To Reproduce** Steps to reproduce the behavior: 1. We're using Pulsar helm install on AWS EKS

[GitHub] [pulsar] rdhabalia commented on pull request #6787: Using Readers still causes backlog quota to be observed

2020-05-27 Thread GitBox
rdhabalia commented on pull request #6787: URL: https://github.com/apache/pulsar/pull/6787#issuecomment-634940827 > We can do here: > > a) use a feature flag to control whether include reader cursors in the storage quota consideration; b) if you don’t want to introduce the feature

[GitHub] [pulsar] jerrypeng merged pull request #7052: Remove function files stored in BK when function is de-registered

2020-05-27 Thread GitBox
jerrypeng merged pull request #7052: URL: https://github.com/apache/pulsar/pull/7052 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: Remove function files stored in BK when function is de-registered (#7052)

2020-05-27 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 fb182c0 Remove function files stored in BK

[GitHub] [pulsar] rdhabalia commented on pull request #6787: Using Readers still causes backlog quota to be observed

2020-05-27 Thread GitBox
rdhabalia commented on pull request #6787: URL: https://github.com/apache/pulsar/pull/6787#issuecomment-634895127 Pulsar reader is a non-durable cursor which means 1. Broker should not persist data because of non-durable cursor's existence but broker should allow non-durable cursor to

[GitHub] [pulsar] sijie commented on pull request #6787: Using Readers still causes backlog quota to be observed

2020-05-27 Thread GitBox
sijie commented on pull request #6787: URL: https://github.com/apache/pulsar/pull/6787#issuecomment-634892636 > I have already heard from members of industry/community that there are too many tunables in Pulsar and its hard for people to navigate. Well, "tuneables" are

[GitHub] [pulsar] jerrypeng commented on pull request #6787: Using Readers still causes backlog quota to be observed

2020-05-27 Thread GitBox
jerrypeng commented on pull request #6787: URL: https://github.com/apache/pulsar/pull/6787#issuecomment-634870390 @sijie @merlimat maybe we can ask others for opinions? @rdhabalia @joefk can you share your opinion on this issue as well?

[GitHub] [pulsar] srkukarni commented on pull request #7057: Fix the flaky function test.

2020-05-27 Thread GitBox
srkukarni commented on pull request #7057: URL: https://github.com/apache/pulsar/pull/7057#issuecomment-634836819 @jerrypeng done This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [pulsar] jerrypeng commented on pull request #7057: Fix the flaky function test.

2020-05-27 Thread GitBox
jerrypeng commented on pull request #7057: URL: https://github.com/apache/pulsar/pull/7057#issuecomment-634831106 @srkukarni the test should also be deleted flaky tests https://github.com/apache/pulsar/blob/master/.github/workflows/ci-unit-flaky.yaml#L84 and removed from:

[GitHub] [pulsar] merlimat merged pull request #7034: [C++] Fix Result can't be serialized to string inside the library

2020-05-27 Thread GitBox
merlimat merged pull request #7034: URL: https://github.com/apache/pulsar/pull/7034 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

[pulsar] branch master updated (d68829c -> a50b75a)

2020-05-27 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 d68829c Add properties for managed-ledger. (#7054) add a50b75a Fix Result can't be serialized to string

[GitHub] [pulsar] srkukarni opened a new pull request #7057: Fix the flaky function test.

2020-05-27 Thread GitBox
srkukarni opened a new pull request #7057: URL: https://github.com/apache/pulsar/pull/7057 *(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-client-go] rueian commented on a change in pull request #248: Clean callbacks of connection after run loop stopped (#239)

2020-05-27 Thread GitBox
rueian commented on a change in pull request #248: URL: https://github.com/apache/pulsar-client-go/pull/248#discussion_r431291341 ## File path: pulsar/internal/connection.go ## @@ -656,18 +662,20 @@ func (c *connection) Close() { c.log.Info("Connection closed")

[GitHub] [pulsar] Saswatibhoi closed issue #6644: Websocket Proxy Connectivity Issue (500 Server Error)

2020-05-27 Thread GitBox
Saswatibhoi closed issue #6644: URL: https://github.com/apache/pulsar/issues/6644 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] sakthi-prasad opened a new issue #7056: pulsar client 2.5.2 cli does not support subscription-position when running consumer

2020-05-27 Thread GitBox
sakthi-prasad opened a new issue #7056: URL: https://github.com/apache/pulsar/issues/7056 When running pulsar client cli command for consumer, --subscription-position is shown as unknown option Note pulsar client version: 2.5.2 Run the below command to start a consumer

[GitHub] [pulsar] BewareMyPower commented on issue #6945: Key-based batching and hashingScheme parameter for Python client

2020-05-27 Thread GitBox
BewareMyPower commented on issue #6945: URL: https://github.com/apache/pulsar/issues/6945#issuecomment-634739076 You could create multiple producers and deliver the message to the right producer associated with the key. **However, I suspected that it's broker's problem.** Here is my

[GitHub] [pulsar] Saswatibhoi opened a new issue #7055: Unable to Produce Messages: Topic is stuck on a ledger that doesn't exist

2020-05-27 Thread GitBox
Saswatibhoi opened a new issue #7055: URL: https://github.com/apache/pulsar/issues/7055 **Describe the bug** While producing messages on a topic, it is trying to reach to a ledger that doesn't exists. The produce operation fails throwing an error message "No such ledger exists on

[GitHub] [pulsar] codelipenghui commented on pull request #4955: [PIP-39] Introduce system topic and topic policies service

2020-05-27 Thread GitBox
codelipenghui commented on pull request #4955: URL: https://github.com/apache/pulsar/pull/4955#issuecomment-634647772 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[pulsar] branch master updated: Add properties for managed-ledger. (#7054)

2020-05-27 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 d68829c Add properties for managed-ledger.

[GitHub] [pulsar] codelipenghui merged pull request #7054: Add properties for managed-ledger.

2020-05-27 Thread GitBox
codelipenghui merged pull request #7054: URL: https://github.com/apache/pulsar/pull/7054 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] nicolo-paganin removed a comment on pull request #6895: [#6003][pulsar-functions] Possibility to add builtin Functions

2020-05-27 Thread GitBox
nicolo-paganin removed a comment on pull request #6895: URL: https://github.com/apache/pulsar/pull/6895#issuecomment-634630347 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] nicolo-paganin commented on pull request #6895: [#6003][pulsar-functions] Possibility to add builtin Functions

2020-05-27 Thread GitBox
nicolo-paganin commented on pull request #6895: URL: https://github.com/apache/pulsar/pull/6895#issuecomment-634630347 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] hangc0276 commented on pull request #6719: fix consumer fetch message number maps to read entry number bug and expose avgMessagesPerEntry metric

2020-05-27 Thread GitBox
hangc0276 commented on pull request #6719: URL: https://github.com/apache/pulsar/pull/6719#issuecomment-634624662 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] nicolo-paganin removed a comment on pull request #6895: [#6003][pulsar-functions] Possibility to add builtin Functions

2020-05-27 Thread GitBox
nicolo-paganin removed a comment on pull request #6895: URL: https://github.com/apache/pulsar/pull/6895#issuecomment-634592624 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] nicolo-paganin commented on pull request #6895: [#6003][pulsar-functions] Possibility to add builtin Functions

2020-05-27 Thread GitBox
nicolo-paganin commented on pull request #6895: URL: https://github.com/apache/pulsar/pull/6895#issuecomment-634592624 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] Anonymitaet commented on pull request #6942: [pulsar-proxy] add advertised address option for proxy

2020-05-27 Thread GitBox
Anonymitaet commented on pull request #6942: URL: https://github.com/apache/pulsar/pull/6942#issuecomment-634566358 Doc added https://github.com/apache/pulsar/pull/7051. This is an automated message from the Apache Git

[GitHub] [pulsar] codelipenghui edited a comment on pull request #6977: Introduce messages fence mechanism for Key_Shared subscription

2020-05-27 Thread GitBox
codelipenghui edited a comment on pull request #6977: URL: https://github.com/apache/pulsar/pull/6977#issuecomment-634564244 @feeblefakie #6791 introduces a consistent hash approach for the consumer selector. The current implementation is a hash range split approach. This PR is to

[GitHub] [pulsar] Anonymitaet commented on pull request #7051: Add advertisedAddress and subscriptionExpirationTimeMinutes conf for broker and proxy

2020-05-27 Thread GitBox
Anonymitaet commented on pull request #7051: URL: https://github.com/apache/pulsar/pull/7051#issuecomment-634565117 @codelipenghui thank you This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] codelipenghui commented on pull request #6977: Introduce messages fence mechanism for Key_Shared subscription

2020-05-27 Thread GitBox
codelipenghui commented on pull request #6977: URL: https://github.com/apache/pulsar/pull/6977#issuecomment-634564244 @feeblefakie #6791 introduces a consistent hash approach for the consumer selector. The current implementation is a hash range split approach.

[GitHub] [pulsar] feeblefakie commented on pull request #6977: Introduce messages fence mechanism for Key_Shared subscription

2020-05-27 Thread GitBox
feeblefakie commented on pull request #6977: URL: https://github.com/apache/pulsar/pull/6977#issuecomment-634546905 @codelipenghui Just a question. So this and #6791 are solving the same issue with different approaches but the both will be applied ? #6791 looks like more

[GitHub] [pulsar] Huanli-Meng commented on pull request #6977: Introduce messages fence mechanism for Key_Shared subscription

2020-05-27 Thread GitBox
Huanli-Meng commented on pull request #6977: URL: https://github.com/apache/pulsar/pull/6977#issuecomment-634532636 should the key_shared subscription in messaging section be updated? This is an automated message from the

[GitHub] [pulsar] codelipenghui opened a new pull request #7054: Add properties for managed-ledger.

2020-05-27 Thread GitBox
codelipenghui opened a new pull request #7054: URL: https://github.com/apache/pulsar/pull/7054 ### Motivation Add properties for managed-ledger. ### Verifying this change New unit tests added ### Does this pull request potentially affect one of the following

[GitHub] [pulsar] Huanli-Meng commented on pull request #7053: [doc] Update Pulsar 2.5.2 sidebar doc.

2020-05-27 Thread GitBox
Huanli-Meng commented on pull request #7053: URL: https://github.com/apache/pulsar/pull/7053#issuecomment-634522017 Update the Pulsar 2.5.2 sidebar doc to 1. add Run Pulsar in Kubernetes doc to Get started section 2: remove the blank page for Connector Admin CLI from the Reference

[GitHub] [pulsar] Huanli-Meng opened a new pull request #7053: [doc] Update Pulsar 2.5.2 sidebar doc.

2020-05-27 Thread GitBox
Huanli-Meng opened a new pull request #7053: URL: https://github.com/apache/pulsar/pull/7053 *(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] codelipenghui commented on pull request #4955: [PIP-39] Introduce system topic and topic policies service

2020-05-27 Thread GitBox
codelipenghui commented on pull request #4955: URL: https://github.com/apache/pulsar/pull/4955#issuecomment-634495498 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] codelipenghui commented on pull request #6347: PIP 57: Improve Zookeeper Session Timeout Handling

2020-05-27 Thread GitBox
codelipenghui commented on pull request #6347: URL: https://github.com/apache/pulsar/pull/6347#issuecomment-634495475 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[pulsar] branch master updated: Add advertisedAddress and subscriptionExpirationTimeMinutes conf for broker and proxy

2020-05-27 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 8ffee4c Add advertisedAddress and

[GitHub] [pulsar] codelipenghui merged pull request #7051: Add advertisedAddress and subscriptionExpirationTimeMinutes conf for broker and proxy

2020-05-27 Thread GitBox
codelipenghui merged pull request #7051: URL: https://github.com/apache/pulsar/pull/7051 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] huangdx0726 commented on pull request #5716: [Issues 5709]remove the namespace checking

2020-05-27 Thread GitBox
huangdx0726 commented on pull request #5716: URL: https://github.com/apache/pulsar/pull/5716#issuecomment-634494100 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] huangdx0726 commented on pull request #7042: [Issue 6957][pulsar-io-debezium] Support CDC Connector for SqlServer

2020-05-27 Thread GitBox
huangdx0726 commented on pull request #7042: URL: https://github.com/apache/pulsar/pull/7042#issuecomment-634494191 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

  1   2   >