[GitHub] [pulsar] pantianying commented on a change in pull request #6452: [function] Go function add more producer

2020-03-06 Thread GitBox
pantianying commented on a change in pull request #6452: [function] Go function add more producer URL: https://github.com/apache/pulsar/pull/6452#discussion_r389233341 ## File path: pulsar-function-go/pf/context.go ## @@ -57,6 +57,11 @@ func (c *FunctionContext)

[GitHub] [pulsar] codelipenghui commented on a change in pull request #6471: [Issue #5395][broker] Implement AutoTopicCreation by namespace override

2020-03-06 Thread GitBox
codelipenghui commented on a change in pull request #6471: [Issue #5395][broker] Implement AutoTopicCreation by namespace override URL: https://github.com/apache/pulsar/pull/6471#discussion_r389224183 ## File path:

[GitHub] [pulsar] codelipenghui commented on a change in pull request #6471: [Issue #5395][broker] Implement AutoTopicCreation by namespace override

2020-03-06 Thread GitBox
codelipenghui commented on a change in pull request #6471: [Issue #5395][broker] Implement AutoTopicCreation by namespace override URL: https://github.com/apache/pulsar/pull/6471#discussion_r389224168 ## File path:

[GitHub] [pulsar] lonerzzz opened a new issue #6506: ConsumerImpl class exception on Timeout provides misleading error detail

2020-03-06 Thread GitBox
lonerzzz opened a new issue #6506: ConsumerImpl class exception on Timeout provides misleading error detail URL: https://github.com/apache/pulsar/issues/6506 **Describe the bug** The ConsumerImpl class internalReceive method throws an InterruptedException when the attempt to receive a

[GitHub] [pulsar] rdhabalia commented on issue #6505: Broker fails to load topic and not able to recover data-ledger

2020-03-06 Thread GitBox
rdhabalia commented on issue #6505: Broker fails to load topic and not able to recover data-ledger URL: https://github.com/apache/pulsar/issues/6505#issuecomment-596039235 Created PR in https://github.com/apache/bookkeeper/pull/2281

[GitHub] [pulsar] rdhabalia opened a new issue #6505: Broker fails to load topic and not able to recover data-ledger

2020-03-06 Thread GitBox
rdhabalia opened a new issue #6505: Broker fails to load topic and not able to recover data-ledger URL: https://github.com/apache/pulsar/issues/6505 ### Issue Broker was not able to load the topic due to failure while loading data ledger of the topic. Data ledger has 2

[GitHub] [pulsar] srkukarni opened a new pull request #6504: Use more granular permissions for topics

2020-03-06 Thread GitBox
srkukarni opened a new pull request #6504: Use more granular permissions for topics URL: https://github.com/apache/pulsar/pull/6504 Instead of always using admin access for topic, use read/write/admin access for topic based on call *(If this PR fixes a github issue, please

[GitHub] [pulsar] yjshen commented on issue #6500: [Minor]Remove unused property from pom

2020-03-06 Thread GitBox
yjshen commented on issue #6500: [Minor]Remove unused property from pom URL: https://github.com/apache/pulsar/pull/6500#issuecomment-596021996 /pulsarbot run-failure-checks This is an automated message from the Apache Git

[GitHub] [pulsar] sijie commented on issue #6471: [Issue #5395][broker] Implement AutoTopicCreation by namespace override

2020-03-06 Thread GitBox
sijie commented on issue #6471: [Issue #5395][broker] Implement AutoTopicCreation by namespace override URL: https://github.com/apache/pulsar/pull/6471#issuecomment-596004276 @jiazhai @codelipenghui can you help review this pull request?

[GitHub] [pulsar] sijie commented on issue #6422: [Issue 6283][tiered-storage] Offload policies per namespace

2020-03-06 Thread GitBox
sijie commented on issue #6422: [Issue 6283][tiered-storage] Offload policies per namespace URL: https://github.com/apache/pulsar/pull/6422#issuecomment-596004480 @gaoran10 @codelipenghui can you review this pull request?

[GitHub] [pulsar] sijie commented on issue #6475: [pulsar-common] Remove duplicate RestException references

2020-03-06 Thread GitBox
sijie commented on issue #6475: [pulsar-common] Remove duplicate RestException references URL: https://github.com/apache/pulsar/pull/6475#issuecomment-596003892 /pulsarbot run-failure-checks This is an automated message from

[GitHub] [pulsar] sijie merged pull request #6483: Fix some async method problems at PersistentTopicsBase.

2020-03-06 Thread GitBox
sijie merged pull request #6483: Fix some async method problems at PersistentTopicsBase. URL: https://github.com/apache/pulsar/pull/6483 This is an automated message from the Apache Git Service. To respond to the message,

[pulsar] branch master updated: Fix some async method problems at PersistentTopicsBase. (#6483)

2020-03-06 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 47ca8e6 Fix some async method problems at

[pulsar] branch master updated (f9ada10 -> 329e231)

2020-03-06 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from f9ada10 [pulsar-client] remove duplicate cnx method (#6490) add 329e231 [proxy] Fix proxy routing to functions

[GitHub] [pulsar] sijie closed pull request #6297: [Issue 4612][proxy] Fix proxy to forward V3 API request to functions worker cluster

2020-03-06 Thread GitBox
sijie closed pull request #6297: [Issue 4612][proxy] Fix proxy to forward V3 API request to functions worker cluster URL: https://github.com/apache/pulsar/pull/6297 This is an automated message from the Apache Git Service.

[GitHub] [pulsar] sijie commented on issue #6297: [Issue 4612][proxy] Fix proxy to forward V3 API request to functions worker cluster

2020-03-06 Thread GitBox
sijie commented on issue #6297: [Issue 4612][proxy] Fix proxy to forward V3 API request to functions worker cluster URL: https://github.com/apache/pulsar/pull/6297#issuecomment-596002261 This was fixed by #6486 This is an

[GitHub] [pulsar] sijie merged pull request #6486: [proxy] Fix proxy routing to functions worker

2020-03-06 Thread GitBox
sijie merged pull request #6486: [proxy] Fix proxy routing to functions worker URL: https://github.com/apache/pulsar/pull/6486 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] sijie commented on a change in pull request #6489: [broker] Timeout API calls in BrokerService

2020-03-06 Thread GitBox
sijie commented on a change in pull request #6489: [broker] Timeout API calls in BrokerService URL: https://github.com/apache/pulsar/pull/6489#discussion_r389185936 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java ## @@ -732,6

[pulsar] branch master updated (ffe9a92 -> f9ada10)

2020-03-06 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from ffe9a92 Fix Topic metrics documentation (#6495) add f9ada10 [pulsar-client] remove duplicate cnx method

[GitHub] [pulsar] sijie merged pull request #6490: [pulsar-client] remove duplicate cnx method

2020-03-06 Thread GitBox
sijie merged pull request #6490: [pulsar-client] remove duplicate cnx method URL: https://github.com/apache/pulsar/pull/6490 This is an automated message from the Apache Git Service. To respond to the message, please log on

[pulsar] branch master updated: Fix Topic metrics documentation (#6495)

2020-03-06 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 ffe9a92 Fix Topic metrics documentation (#6495)

[GitHub] [pulsar] sijie merged pull request #6495: [docs]: fix wrong metric name in `Topic metrics`

2020-03-06 Thread GitBox
sijie merged pull request #6495: [docs]: fix wrong metric name in `Topic metrics` URL: https://github.com/apache/pulsar/pull/6495 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [pulsar] sijie commented on issue #6499: Start namespace service and schema registry service before start broker.

2020-03-06 Thread GitBox
sijie commented on issue #6499: Start namespace service and schema registry service before start broker. URL: https://github.com/apache/pulsar/pull/6499#issuecomment-595963762 /pulsarbot run-failure-checks This is an

[GitHub] [pulsar] lonerzzz opened a new issue #6503: Vague error message when pulsar zookeeper namespace information is missing

2020-03-06 Thread GitBox
lonerzzz opened a new issue #6503: Vague error message when pulsar zookeeper namespace information is missing URL: https://github.com/apache/pulsar/issues/6503 **Describe the bug** When the client cannot access the /namespace path on the zookeeper server, it generates a vague error

[pulsar] branch master updated: Refactor tests in pulsar client tools test (#6472)

2020-03-06 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 73e8beb Refactor tests in pulsar client tools

[GitHub] [pulsar] sijie merged pull request #6472: Refactor tests in pulsar client tools test

2020-03-06 Thread GitBox
sijie merged pull request #6472: Refactor tests in pulsar client tools test URL: https://github.com/apache/pulsar/pull/6472 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [pulsar] lonerzzz commented on issue #6493: [pulsar-broker] Policies not found

2020-03-06 Thread GitBox
lonerzzz commented on issue #6493: [pulsar-broker] Policies not found URL: https://github.com/apache/pulsar/issues/6493#issuecomment-595959554 Was just looking into a similar issue for my own system and saw your post... I identified the problem by connecting with the zookeeper client to

[GitHub] [pulsar] addisonj opened a new pull request #6502: [functions] WIP Introduce `RichFunction` interface for more control

2020-03-06 Thread GitBox
addisonj opened a new pull request #6502: [functions] WIP Introduce `RichFunction` interface for more control URL: https://github.com/apache/pulsar/pull/6502 Early commit! Not intended for merge, just discussion. This commit introduces a new interface `RichFunction` (name subject to

[GitHub] [pulsar] junquero edited a comment on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0

2020-03-06 Thread GitBox
junquero edited a comment on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 URL: https://github.com/apache/pulsar/issues/6501#issuecomment-595862750 ### NOTE: this comment is not correct. The issue described is because volumes mounted in the container

[GitHub] [pulsar] junquero edited a comment on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0

2020-03-06 Thread GitBox
junquero edited a comment on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 URL: https://github.com/apache/pulsar/issues/6501#issuecomment-595880538 I have to say that with a clean environment, it has worked with latest version. I just removed previous

[GitHub] [pulsar] junquero commented on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0

2020-03-06 Thread GitBox
junquero commented on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 URL: https://github.com/apache/pulsar/issues/6501#issuecomment-595880538 I have to say that with a clean environment, it has worked with latest version. I just removed previous volumes

[GitHub] [pulsar] junquero edited a comment on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0

2020-03-06 Thread GitBox
junquero edited a comment on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 URL: https://github.com/apache/pulsar/issues/6501#issuecomment-595877931 I edited the description of the issue to add two volumes creation commands before running the container.

[GitHub] [pulsar] junquero commented on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0

2020-03-06 Thread GitBox
junquero commented on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 URL: https://github.com/apache/pulsar/issues/6501#issuecomment-595877931 I edited the description of the issue to add two volumes creation commands before running the container.

[GitHub] [pulsar] addisonj commented on issue #6486: [proxy] Fix proxy routing to functions worker

2020-03-06 Thread GitBox
addisonj commented on issue #6486: [proxy] Fix proxy routing to functions worker URL: https://github.com/apache/pulsar/pull/6486#issuecomment-595872311 /pulsarbot run-failure-checks This is an automated message from the

[GitHub] [pulsar] junquero edited a comment on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0

2020-03-06 Thread GitBox
junquero edited a comment on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 URL: https://github.com/apache/pulsar/issues/6501#issuecomment-595862750 Tested also with latest right now. It behaves diferently: show schemas does work right, but show

[GitHub] [pulsar] junquero commented on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0

2020-03-06 Thread GitBox
junquero commented on issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 URL: https://github.com/apache/pulsar/issues/6501#issuecomment-595862750 Tested also with latest right now. It behaves difeerently: show schemas does work right, but show tables

[GitHub] [pulsar] addisonj commented on issue #6486: [proxy] Fix proxy routing to functions worker

2020-03-06 Thread GitBox
addisonj commented on issue #6486: [proxy] Fix proxy routing to functions worker URL: https://github.com/apache/pulsar/pull/6486#issuecomment-595832091 oh yep, missed that one as it isn't part of the v3 group This is an

[GitHub] [pulsar] junquero opened a new issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0

2020-03-06 Thread GitBox
junquero opened a new issue #6501: Pulsar SQL Fails Out of the Box with Default Docker Deployment v2.5.0 URL: https://github.com/apache/pulsar/issues/6501 **Describe the bug** Run pulsar docker **To Reproduce** Steps to reproduce the behavior (commands in bold): 1. `docker

[GitHub] [pulsar] yjshen opened a new pull request #6500: [Minor]Remove unused property from pom

2020-03-06 Thread GitBox
yjshen opened a new pull request #6500: [Minor]Remove unused property from pom URL: https://github.com/apache/pulsar/pull/6500 This PR is a follow-up of #6494 This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] codelipenghui opened a new pull request #6499: Start namespace service and schema registry service before start broker.

2020-03-06 Thread GitBox
codelipenghui opened a new pull request #6499: Start namespace service and schema registry service before start broker. URL: https://github.com/apache/pulsar/pull/6499 ### Motivation If the broker service is started, the client can connect to the broker and send requests depends on

[GitHub] [pulsar] k2la commented on issue #6498: [pulsar-client-cpp] Fix Redelivery of Messages on UnackedMessageTracker When Ack Messages .

2020-03-06 Thread GitBox
k2la commented on issue #6498: [pulsar-client-cpp] Fix Redelivery of Messages on UnackedMessageTracker When Ack Messages . URL: https://github.com/apache/pulsar/pull/6498#issuecomment-595779041 /pulsarbot run-failure-checks

[GitHub] [pulsar] Lanayx edited a comment on issue #6486: [proxy] Fix proxy routing to functions worker

2020-03-06 Thread GitBox
Lanayx edited a comment on issue #6486: [proxy] Fix proxy routing to functions worker URL: https://github.com/apache/pulsar/pull/6486#issuecomment-595729346 Shouldn't `admin/worker`, `admin/v2/worker` and `admin/v3/worker` routes be redirected to worker as well?

[GitHub] [pulsar] Lanayx commented on issue #6486: [proxy] Fix proxy routing to functions worker

2020-03-06 Thread GitBox
Lanayx commented on issue #6486: [proxy] Fix proxy routing to functions worker URL: https://github.com/apache/pulsar/pull/6486#issuecomment-595729346 Shouldn't `admin/v2/worker` and `admin/v3/worker` routes be redirected to worker as well?

[GitHub] [pulsar] k2la opened a new pull request #6498: [pulsar-clientc-cpp] Fix Redelivery of Messages on UnackedMessageTracker When Ack Messages .

2020-03-06 Thread GitBox
k2la opened a new pull request #6498: [pulsar-clientc-cpp] Fix Redelivery of Messages on UnackedMessageTracker When Ack Messages . URL: https://github.com/apache/pulsar/pull/6498 ### Motivation Because of #6391 , acked messages were counted as unacked messages. Although messages

[pulsar] branch master updated (19ccfd5 -> a3e1efc)

2020-03-06 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 19ccfd5 Fix create partitioned topic with a substring of an existing topic name. (#6478) add a3e1efc Bump

[GitHub] [pulsar] sijie merged pull request #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module

2020-03-06 Thread GitBox
sijie merged pull request #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module URL: https://github.com/apache/pulsar/pull/6494 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] congbobo184 commented on issue #6462: getLastMessageId from PulsarAdmin doesn't work with BatchMessageIdImpl

2020-03-06 Thread GitBox
congbobo184 commented on issue #6462: getLastMessageId from PulsarAdmin doesn't work with BatchMessageIdImpl URL: https://github.com/apache/pulsar/issues/6462#issuecomment-595689426 @kevin-kosmos broker don't store the batchIndex, client only loop set batchIndex.

[GitHub] [pulsar] liujiejie opened a new issue #6497: create the producer whill core in c++

2020-03-06 Thread GitBox
liujiejie opened a new issue #6497: create the producer whill core in c++ URL: https://github.com/apache/pulsar/issues/6497 **Describe the bug** A clear and concise description of what the bug is. When I start the system to create the producer,The program will lose its core。The

[GitHub] [pulsar] yjshen removed a comment on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module

2020-03-06 Thread GitBox
yjshen removed a comment on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module URL: https://github.com/apache/pulsar/pull/6494#issuecomment-595675630 /pulsarbot run-failure-checks This is an automated

[GitHub] [pulsar] yjshen commented on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module

2020-03-06 Thread GitBox
yjshen commented on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module URL: https://github.com/apache/pulsar/pull/6494#issuecomment-595682389 cc @sijie @jiazhai for review. This is an automated message

[GitHub] [pulsar] yjshen commented on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module

2020-03-06 Thread GitBox
yjshen commented on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module URL: https://github.com/apache/pulsar/pull/6494#issuecomment-595675630 /pulsarbot run-failure-checks This is an automated message

[GitHub] [pulsar] yjshen removed a comment on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module

2020-03-06 Thread GitBox
yjshen removed a comment on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module URL: https://github.com/apache/pulsar/pull/6494#issuecomment-595674945 /pulsarbot run-failure-tests This is an automated

[GitHub] [pulsar] yjshen removed a comment on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module

2020-03-06 Thread GitBox
yjshen removed a comment on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module URL: https://github.com/apache/pulsar/pull/6494#issuecomment-595667061 /pulsarbot run-failure-tests This is an automated

[GitHub] [pulsar] yjshen commented on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module

2020-03-06 Thread GitBox
yjshen commented on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module URL: https://github.com/apache/pulsar/pull/6494#issuecomment-595674945 /pulsarbot run-failure-tests This is an automated message

[GitHub] [pulsar] wonko opened a new issue #6496: ReceiverQueueSize: -1 prefetches one message

2020-03-06 Thread GitBox
wonko opened a new issue #6496: ReceiverQueueSize: -1 prefetches one message URL: https://github.com/apache/pulsar/issues/6496 **Describe the bug** When subscribing to a shared topic, and pulling messages one by one with a settings of `ReceiverQueueSize: -1` (golang client), it seems

[GitHub] [pulsar] yjshen commented on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module

2020-03-06 Thread GitBox
yjshen commented on issue #6494: Bump jcloud version to 2.2.0 and remove jcloud-shade module URL: https://github.com/apache/pulsar/pull/6494#issuecomment-595667061 /pulsarbot run-failure-tests This is an automated message

[pulsar] branch master updated: Fix create partitioned topic with a substring of an existing topic name. (#6478)

2020-03-06 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 19ccfd5 Fix create partitioned topic with a

[GitHub] [pulsar] codelipenghui closed issue #6468: Unable to create partitioned topic which is a substring of an existing topic

2020-03-06 Thread GitBox
codelipenghui closed issue #6468: Unable to create partitioned topic which is a substring of an existing topic URL: https://github.com/apache/pulsar/issues/6468 This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] yjshen commented on issue #6484: pulsar-client-admin-2.5.0.jar contains an unshaded protobuf dependency

2020-03-06 Thread GitBox
yjshen commented on issue #6484: pulsar-client-admin-2.5.0.jar contains an unshaded protobuf dependency URL: https://github.com/apache/pulsar/issues/6484#issuecomment-595666076 I will check on this later This is an automated

[GitHub] [pulsar] codelipenghui merged pull request #6478: Fix create partitioned topic with a substring of an existing topic name.

2020-03-06 Thread GitBox
codelipenghui merged pull request #6478: Fix create partitioned topic with a substring of an existing topic name. URL: https://github.com/apache/pulsar/pull/6478 This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] congbobo184 commented on issue #6464: Topics with replicated subscription are not deleted in GC

2020-03-06 Thread GitBox
congbobo184 commented on issue #6464: Topics with replicated subscription are not deleted in GC URL: https://github.com/apache/pulsar/issues/6464#issuecomment-595658111 @hrsakai do you set-retention for the namespace from the topic?