[GitHub] [pulsar] yittg commented on issue #4992: [pulsar-client-cpp/python, dashboard] support parse batch entry

2019-08-21 Thread GitBox
yittg commented on issue #4992: [pulsar-client-cpp/python,dashboard] support parse batch entry URL: https://github.com/apache/pulsar/pull/4992#issuecomment-523746553 run cpp tests This is an automated message from the Apache

[GitHub] [pulsar] tuteng opened a new pull request #5008: [Pulsar IO]Support reload Source and Sink for Pulsar IO

2019-08-21 Thread GitBox
tuteng opened a new pull request #5008: [Pulsar IO]Support reload Source and Sink for Pulsar IO URL: https://github.com/apache/pulsar/pull/5008 Fixes: https://github.com/apache/pulsar/issues/4904 Master Issue: https://github.com/apache/pulsar/issues/4904 ### Motivation

[GitHub] [pulsar] mingfang commented on issue #4957: docker image needs to install 'less'

2019-08-21 Thread GitBox
mingfang commented on issue #4957: docker image needs to install 'less' URL: https://github.com/apache/pulsar/issues/4957#issuecomment-523742771 ```sh docker run -it apachepulsar/pulsar-all:latest bash root@333459a84f05:/pulsar# bin/pulsar sql --server

[GitHub] [pulsar] tuteng commented on issue #4957: docker image needs to install 'less'

2019-08-21 Thread GitBox
tuteng commented on issue #4957: docker image needs to install 'less' URL: https://github.com/apache/pulsar/issues/4957#issuecomment-523741389 @mingfang Can you describe your context in detail? For example, whether the container environment was used or the local environment, what

[pulsar-client-go] branch master updated: Add some test cases for project (#58)

2019-08-21 Thread zhaijia
This is an automated email from the ASF dual-hosted git repository. zhaijia 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 de147fa Add some test cases for

[GitHub] [pulsar-client-go] jiazhai merged pull request #58: Add some test cases for project

2019-08-21 Thread GitBox
jiazhai merged pull request #58: Add some test cases for project URL: https://github.com/apache/pulsar-client-go/pull/58 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar-client-go] wolfstudy opened a new issue #60: The flow of multi-Partition consumer can't work

2019-08-21 Thread GitBox
wolfstudy opened a new issue #60: The flow of multi-Partition consumer can't work URL: https://github.com/apache/pulsar-client-go/issues/60 Expected behavior The flow of multi-Partition consumer can correct work Actual behavior The receive blocked.

[GitHub] [pulsar] zymap commented on a change in pull request #4993: [Transaction] handle command send with transaction messages

2019-08-21 Thread GitBox
zymap commented on a change in pull request #4993: [Transaction] handle command send with transaction messages URL: https://github.com/apache/pulsar/pull/4993#discussion_r316486474 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java

[GitHub] [pulsar] tuteng commented on issue #4991: [website] Fix the problem that tab click on the page did not response.

2019-08-21 Thread GitBox
tuteng commented on issue #4991: [website] Fix the problem that tab click on the page did not response. URL: https://github.com/apache/pulsar/pull/4991#issuecomment-523734636 run java8 tests This is an automated message from

[GitHub] [pulsar-translation] ireneontheway commented on issue #6: Add workflow-guidelines

2019-08-21 Thread GitBox
ireneontheway commented on issue #6: Add workflow-guidelines URL: https://github.com/apache/pulsar-translation/pull/6#issuecomment-523733548 @Anonymitaet Hello,can you help review this file and merge it ? This is an automated

[GitHub] [pulsar] zymap edited a comment on issue #5001: [Pulsar-sql]Using pulsar SQL query messages will appear `NoSuchLedger` when zk root directory changed

2019-08-21 Thread GitBox
zymap edited a comment on issue #5001: [Pulsar-sql]Using pulsar SQL query messages will appear `NoSuchLedger` when zk root directory changed URL: https://github.com/apache/pulsar/pull/5001#issuecomment-523732129 @sijie I think the integration tests is more suit for testing this scene. But

[GitHub] [pulsar-translation] ireneontheway opened a new pull request #6: Add workflow-guidelines

2019-08-21 Thread GitBox
ireneontheway opened a new pull request #6: Add workflow-guidelines URL: https://github.com/apache/pulsar-translation/pull/6 Add workflow-guidelines This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] zymap commented on issue #5001: [Pulsar-sql]Using pulsar SQL query messages will appear `NoSuchLedger` when zk root directory changed

2019-08-21 Thread GitBox
zymap commented on issue #5001: [Pulsar-sql]Using pulsar SQL query messages will appear `NoSuchLedger` when zk root directory changed URL: https://github.com/apache/pulsar/pull/5001#issuecomment-523732129 @sijie I think the integration tests is more suit for testing this scene. But I need

[GitHub] [pulsar-translation] ireneontheway closed pull request #5: Add translation guidelines

2019-08-21 Thread GitBox
ireneontheway closed pull request #5: Add translation guidelines URL: https://github.com/apache/pulsar-translation/pull/5 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar-translation] ireneontheway opened a new pull request #5: Add translation guidelines

2019-08-21 Thread GitBox
ireneontheway opened a new pull request #5: Add translation guidelines URL: https://github.com/apache/pulsar-translation/pull/5 Add translation guidelines This is an automated message from the Apache Git Service. To respond

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

2019-08-21 Thread GitBox
codelipenghui commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r316479721 ## File path:

[GitHub] [pulsar] zymap commented on a change in pull request #4993: [Transaction] handle command send with transaction messages

2019-08-21 Thread GitBox
zymap commented on a change in pull request #4993: [Transaction] handle command send with transaction messages URL: https://github.com/apache/pulsar/pull/4993#discussion_r316479437 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java

[GitHub] [pulsar] freeznet commented on a change in pull request #4996: [k8s] convert to valid pod name part with k8s function runtime

2019-08-21 Thread GitBox
freeznet commented on a change in pull request #4996: [k8s] convert to valid pod name part with k8s function runtime URL: https://github.com/apache/pulsar/pull/4996#discussion_r316479121 ## File path:

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

2019-08-21 Thread GitBox
codelipenghui commented on issue #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#issuecomment-523725357 @sijie Thanks for the review, i have addressed your comment.

[GitHub] [pulsar] congbobo184 commented on issue #4831: Pulsar schema doesn't do compatibility check for primitive types

2019-08-21 Thread GitBox
congbobo184 commented on issue #4831: Pulsar schema doesn't do compatibility check for primitive types URL: https://github.com/apache/pulsar/issues/4831#issuecomment-523719364 > @congbobo184 to keep simple, we only check compatibilities when using same schema type. different schema types

[GitHub] [pulsar] zymap commented on a change in pull request #4993: [Transaction] handle command send with transaction messages

2019-08-21 Thread GitBox
zymap commented on a change in pull request #4993: [Transaction] handle command send with transaction messages URL: https://github.com/apache/pulsar/pull/4993#discussion_r316471795 ## File path:

[pulsar-client-node] branch master updated (9bc6646 -> 5f8aadb)

2019-08-21 Thread massakam
This is an automated email from the ASF dual-hosted git repository. massakam pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-node.git. from 9bc6646 Merge pull request #45 from massakam/introduce-license-check-and-add new 9ec5690 added

[GitHub] [pulsar] jerrypeng opened a new pull request #5007: Need to reinitialize certain components for externally managed runtimes when moving functions

2019-08-21 Thread GitBox
jerrypeng opened a new pull request #5007: Need to reinitialize certain components for externally managed runtimes when moving functions URL: https://github.com/apache/pulsar/pull/5007 ### Motivation In Pulsar Functions, for externally managed runtimes e.g. KubernetesRuntime,

[GitHub] [pulsar] kezhuw commented on issue #5002: Fixes not owned bundles got selected when broker overloading

2019-08-21 Thread GitBox
kezhuw commented on issue #5002: Fixes not owned bundles got selected when broker overloading URL: https://github.com/apache/pulsar/pull/5002#issuecomment-523692620 run java8 tests This is an automated message from the

[GitHub] [pulsar] sijie commented on issue #4626: Python Pulsar client producer send should return messageId

2019-08-21 Thread GitBox
sijie commented on issue #4626: Python Pulsar client producer send should return messageId URL: https://github.com/apache/pulsar/issues/4626#issuecomment-523660433 @yittg : #4811 is fixing the issue in C++ client and CGo client. For the python fix, it has to wait for #4811 to be merged.

[GitHub] [pulsar] sijie commented on issue #4537: Go client: Missing ability to retrieve MessageID from pulsar.ProducerMessage

2019-08-21 Thread GitBox
sijie commented on issue #4537: Go client: Missing ability to retrieve MessageID from pulsar.ProducerMessage URL: https://github.com/apache/pulsar/issues/4537#issuecomment-523660252 #4811 is fixing the issue. This is an

[GitHub] [pulsar] sijie commented on issue #4831: Pulsar schema doesn't do compatibility check for primitive types

2019-08-21 Thread GitBox
sijie commented on issue #4831: Pulsar schema doesn't do compatibility check for primitive types URL: https://github.com/apache/pulsar/issues/4831#issuecomment-523659836 @congbobo184 to keep simple, we only check compatibilities when using same schema type. different schema types are

[GitHub] [pulsar] sijie commented on issue #5004: RedeliverUnacknowledgedMessages by ids returns more messages than requested

2019-08-21 Thread GitBox
sijie commented on issue #5004: RedeliverUnacknowledgedMessages by ids returns more messages than requested URL: https://github.com/apache/pulsar/issues/5004#issuecomment-523658693 @Lanayx : If it is exclusive or failover, it is the expected current behavior. There is an issue following

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r316411189 ## File path:

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r315511539 ## File path: conf/broker.conf ## @@ -284,6 +284,13 @@

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r316409371 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/systopic/ActionType.java

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r316410694 ## File path:

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r315512537 ## File path:

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r316409421 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/systopic/EventType.java

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r315512692 ## File path:

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r316409633 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/systopic/PulsarEvent.java

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r315511828 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r315511615 ## File path: conf/broker.conf ## @@ -284,6 +284,13 @@

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r316409583 ## File path:

[GitHub] [pulsar] sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4955: [WIP] [PIP-39] Introduce system topic and topic policies service URL: https://github.com/apache/pulsar/pull/4955#discussion_r315512176 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java

[GitHub] [pulsar] sijie commented on a change in pull request #4993: [Transaction] handle command send with transaction messages

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4993: [Transaction] handle command send with transaction messages URL: https://github.com/apache/pulsar/pull/4993#discussion_r316406001 ## File path:

[GitHub] [pulsar] sijie commented on a change in pull request #4993: [Transaction] handle command send with transaction messages

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4993: [Transaction] handle command send with transaction messages URL: https://github.com/apache/pulsar/pull/4993#discussion_r316405602 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java

[GitHub] [pulsar] sijie commented on a change in pull request #4993: [Transaction] handle command send with transaction messages

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4993: [Transaction] handle command send with transaction messages URL: https://github.com/apache/pulsar/pull/4993#discussion_r316404727 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java

[GitHub] [pulsar] sijie commented on a change in pull request #4993: [Transaction] handle command send with transaction messages

2019-08-21 Thread GitBox
sijie commented on a change in pull request #4993: [Transaction] handle command send with transaction messages URL: https://github.com/apache/pulsar/pull/4993#discussion_r316404921 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/ServerCnx.java

[GitHub] [pulsar] Lanayx commented on issue #5004: RedeliverUnacknowledgedMessages by ids returns more messages than requested

2019-08-21 Thread GitBox
Lanayx commented on issue #5004: RedeliverUnacknowledgedMessages by ids returns more messages than requested URL: https://github.com/apache/pulsar/issues/5004#issuecomment-523650777 @sijie Subscription mode is default - exclusive

[GitHub] [pulsar] sijie commented on issue #4995: Admin pod should use broker service to discover broker pods

2019-08-21 Thread GitBox
sijie commented on issue #4995: Admin pod should use broker service to discover broker pods URL: https://github.com/apache/pulsar/pull/4995#issuecomment-523650463 run integration tests This is an automated message from the

[GitHub] [pulsar] KannarFr commented on issue #5006: AvroSchema in pulsar function

2019-08-21 Thread GitBox
KannarFr commented on issue #5006: AvroSchema in pulsar function URL: https://github.com/apache/pulsar/issues/5006#issuecomment-523650339 poke @sijie @merlimat @gmethvin This is an automated message from the Apache Git

[GitHub] [pulsar] KannarFr opened a new issue #5006: AvroSchema in pulsar function

2019-08-21 Thread GitBox
KannarFr opened a new issue #5006: AvroSchema in pulsar function URL: https://github.com/apache/pulsar/issues/5006 **Describe the bug** I'm using pulsar4s (pulsar scala client) and `pulsar-client consume` to debug. When I produce and consume using pulsar4s-avro, I've got the

[GitHub] [pulsar] sijie commented on issue #5004: RedeliverUnacknowledgedMessages by ids returns more messages than requested

2019-08-21 Thread GitBox
sijie commented on issue #5004: RedeliverUnacknowledgedMessages by ids returns more messages than requested URL: https://github.com/apache/pulsar/issues/5004#issuecomment-523647905 @Lanayx what is the subscription mode? This

[GitHub] [pulsar] sijie commented on issue #5003: Python - Exception: Pulsar error: IncompatibleSchema when trying to send message to Pulsar Topic

2019-08-21 Thread GitBox
sijie commented on issue #5003: Python - Exception: Pulsar error: IncompatibleSchema when trying to send message to Pulsar Topic URL: https://github.com/apache/pulsar/issues/5003#issuecomment-523646815 @StormRyders - does the topic exist before you create the producer? - try to

[GitHub] [pulsar] sijie commented on issue #4998: [BUILD FAILURE] pulsar-broker-auth-athenz of 2.5.0-SNAPSHOT[master], due to wrong maven repository

2019-08-21 Thread GitBox
sijie commented on issue #4998: [BUILD FAILURE] pulsar-broker-auth-athenz of 2.5.0-SNAPSHOT[master], due to wrong maven repository URL: https://github.com/apache/pulsar/issues/4998#issuecomment-523646394 @frank-dkvan just FYI. I don't have any problems building the master and also all the

[GitHub] [pulsar] sijie commented on issue #4998: [BUILD FAILURE] pulsar-broker-auth-athenz of 2.5.0-SNAPSHOT[master], due to wrong maven repository

2019-08-21 Thread GitBox
sijie commented on issue #4998: [BUILD FAILURE] pulsar-broker-auth-athenz of 2.5.0-SNAPSHOT[master], due to wrong maven repository URL: https://github.com/apache/pulsar/issues/4998#issuecomment-523645989 @frank-dkvan can you provide more details about what command you are running?

[GitHub] [pulsar] sijie commented on issue #4994: Non-partitioned topic with '-partition-[number]' suffix conflicts with partitioned topic

2019-08-21 Thread GitBox
sijie commented on issue #4994: Non-partitioned topic with '-partition-[number]' suffix conflicts with partitioned topic URL: https://github.com/apache/pulsar/issues/4994#issuecomment-523644616 > If the user created a partitioned topic with the same name after creating a non-partitioned

[GitHub] [pulsar] sijie commented on issue #4991: [website] Fix the problem that tab click on the page did not response.

2019-08-21 Thread GitBox
sijie commented on issue #4991: [website] Fix the problem that tab click on the page did not response. URL: https://github.com/apache/pulsar/pull/4991#issuecomment-523642374 run java8 tests run integration tests This is

[pulsar] branch master updated (e8b4edd -> 4cf3516)

2019-08-21 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 e8b4edd ### Motivation (#4999) add 4cf3516 fix docker image not have cpp client installed (#4986) No new

[GitHub] [pulsar] sijie closed issue #4971: docker image should have C++ client installed

2019-08-21 Thread GitBox
sijie closed issue #4971: docker image should have C++ client installed URL: https://github.com/apache/pulsar/issues/4971 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] sijie merged pull request #4986: [Issue 4971] fix docker image not have cpp client installed

2019-08-21 Thread GitBox
sijie merged pull request #4986: [Issue 4971] fix docker image not have cpp client installed URL: https://github.com/apache/pulsar/pull/4986 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] sijie merged pull request #4999: [Pulsar IO]Completing connector configuration

2019-08-21 Thread GitBox
sijie merged pull request #4999: [Pulsar IO]Completing connector configuration URL: https://github.com/apache/pulsar/pull/4999 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] sijie closed issue #4997: [Pulsar-IO]Nar package for connector not generated for Flume, Redis, Solr and InfluxDB

2019-08-21 Thread GitBox
sijie closed issue #4997: [Pulsar-IO]Nar package for connector not generated for Flume, Redis, Solr and InfluxDB URL: https://github.com/apache/pulsar/issues/4997 This is an automated message from the Apache Git Service. To

[pulsar] branch master updated (5631067 -> e8b4edd)

2019-08-21 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 5631067 [doc] Add a documentation page for metrics reference (#4910) add e8b4edd ### Motivation (#4999) No

[GitHub] [pulsar] tcbuzor commented on issue #4988: Producer create method blocked forever

2019-08-21 Thread GitBox
tcbuzor commented on issue #4988: Producer create method blocked forever URL: https://github.com/apache/pulsar/issues/4988#issuecomment-523602943 When a Producer.create is called, how long does it create request last ? does it wait forever for the producer to be created? Can I pass a

[GitHub] [pulsar] StormRyders opened a new issue #5003: Python - Exception: Pulsar error: IncompatibleSchema when trying to send message to Pulsar Topic

2019-08-21 Thread GitBox
StormRyders opened a new issue #5003: Python - Exception: Pulsar error: IncompatibleSchema when trying to send message to Pulsar Topic URL: https://github.com/apache/pulsar/issues/5003 **Describe the bug** I am trying to create Pulsar Producer in Python and getting below exception:

[GitHub] [pulsar] codelipenghui commented on issue #4987: Trim deleted entries after recover cursor.

2019-08-21 Thread GitBox
codelipenghui commented on issue #4987: Trim deleted entries after recover cursor. URL: https://github.com/apache/pulsar/pull/4987#issuecomment-523472392 retest this please This is an automated message from the Apache Git

[GitHub] [pulsar] codelipenghui commented on issue #4621: [PIP-38] Support batch receive in java client.

2019-08-21 Thread GitBox
codelipenghui commented on issue #4621: [PIP-38] Support batch receive in java client. URL: https://github.com/apache/pulsar/pull/4621#issuecomment-523471418 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] kezhuw opened a new pull request #5002: Fixes not owned bundles got selected when broker overloading

2019-08-21 Thread GitBox
kezhuw opened a new pull request #5002: Fixes not owned bundles got selected when broker overloading URL: https://github.com/apache/pulsar/pull/5002 ### Motivation Fixes not owned bundles got selected when broker overloading ### Modifications Filters out not owned bundles in

[GitHub] [pulsar] jiazhai commented on issue #3414: pulsar-admin - HTTP 401 Authentication required / 500 Internal Server Error

2019-08-21 Thread GitBox
jiazhai commented on issue #3414: pulsar-admin - HTTP 401 Authentication required / 500 Internal Server Error URL: https://github.com/apache/pulsar/issues/3414#issuecomment-523452118 This seems be a dup to #4560 This is an

[GitHub] [pulsar] zymap opened a new pull request #5001: [Pulsar-sql]Using pulsar SQL query messages will appear `NoSuchLedger` when zk root directory changed

2019-08-21 Thread GitBox
zymap opened a new pull request #5001: [Pulsar-sql]Using pulsar SQL query messages will appear `NoSuchLedger` when zk root directory changed URL: https://github.com/apache/pulsar/pull/5001 Fixes #4715 *Motivation* When zookeeper ledgers root path is changed, using

[GitHub] [pulsar] congbobo184 opened a new pull request #5000: Admin api Support the keyValueSchema upload

2019-08-21 Thread GitBox
congbobo184 opened a new pull request #5000: Admin api Support the keyValueSchema upload URL: https://github.com/apache/pulsar/pull/5000 ### Motivation To fix #4840 Does this pull request potentially affect one of the following parts: If yes was chosen, please highlight the

[GitHub] [pulsar] freeznet commented on issue #4986: [Issue 4971] fix docker image not have cpp client installed

2019-08-21 Thread GitBox
freeznet commented on issue #4986: [Issue 4971] fix docker image not have cpp client installed URL: https://github.com/apache/pulsar/pull/4986#issuecomment-523401982 run integration tests This is an automated message from

[GitHub] [pulsar] MrBlueMoo commented on issue #4977: Unable to delete schema from topic after getting schema into a certain state

2019-08-21 Thread GitBox
MrBlueMoo commented on issue #4977: Unable to delete schema from topic after getting schema into a certain state URL: https://github.com/apache/pulsar/issues/4977#issuecomment-523399871 @sijie Thank you, thats good to hear, apologies I didn't notice the existing issue before raising!

[GitHub] [pulsar] yittg edited a comment on issue #4626: Python Pulsar client producer send should return messageId

2019-08-21 Thread GitBox
yittg edited a comment on issue #4626: Python Pulsar client producer send should return messageId URL: https://github.com/apache/pulsar/issues/4626#issuecomment-523389300 @sijie Anyone is working on this? i would like to take it if not.

[GitHub] [pulsar] yittg edited a comment on issue #4992: [pulsar-client-cpp/python, dashboard] support parse batch entry

2019-08-21 Thread GitBox
yittg edited a comment on issue #4992: [pulsar-client-cpp/python,dashboard] support parse batch entry URL: https://github.com/apache/pulsar/pull/4992#issuecomment-523394292 @tuteng > But there are some issues about parsing batch messages. When I test locally mac environment,

[GitHub] [pulsar] yittg commented on issue #4992: [pulsar-client-cpp/python, dashboard] support parse batch entry

2019-08-21 Thread GitBox
yittg commented on issue #4992: [pulsar-client-cpp/python,dashboard] support parse batch entry URL: https://github.com/apache/pulsar/pull/4992#issuecomment-523394292 > But there are some issues about parsing batch messages. When I test locally mac environment, pulsar-client shows that the

[GitHub] [pulsar-client-go] banishee opened a new issue #59: setting the SequenceId can't have an effect on Message deduplication

2019-08-21 Thread GitBox
banishee opened a new issue #59: setting the SequenceId can't have an effect on Message deduplication URL: https://github.com/apache/pulsar-client-go/issues/59 Expected behavior Preparation: Enabling message deduplication. when I initialized the `ProducerMessage{}`, the

[GitHub] [pulsar] yittg commented on issue #4626: Python Pulsar client producer send should return messageId

2019-08-21 Thread GitBox
yittg commented on issue #4626: Python Pulsar client producer send should return messageId URL: https://github.com/apache/pulsar/issues/4626#issuecomment-523389300 @sijie Anyone is working on this? otherwise i would like to take it.

[GitHub] [pulsar] frank-dkvan opened a new issue #4998: [BUILD FAILURE] pulsar-broker-auth-athenz of 2.5.0-SNAPSHOT[master], due to wrong maven repository

2019-08-21 Thread GitBox
frank-dkvan opened a new issue #4998: [BUILD FAILURE] pulsar-broker-auth-athenz of 2.5.0-SNAPSHOT[master], due to wrong maven repository URL: https://github.com/apache/pulsar/issues/4998 **Describe the bug** don't have jar in the maven repository:

[GitHub] [pulsar] tuteng opened a new pull request #4999: [Pulsar IO]Completing connector configuration

2019-08-21 Thread GitBox
tuteng opened a new pull request #4999: [Pulsar IO]Completing connector configuration URL: https://github.com/apache/pulsar/pull/4999 Fixes https://github.com/apache/pulsar/issues/4997 Master Issue: https://github.com/apache/pulsar/issues/4997 ### Motivation In the

[GitHub] [pulsar] tuteng commented on issue #4992: [pulsar-client-cpp/python, dashboard] support parse batch entry

2019-08-21 Thread GitBox
tuteng commented on issue #4992: [pulsar-client-cpp/python,dashboard] support parse batch entry URL: https://github.com/apache/pulsar/pull/4992#issuecomment-523374782 @yittg LGTM But there are some issues about parsing batch messages. When I test locally mac environment, pulsar-client

[GitHub] [pulsar] tuteng commented on issue #4991: [website] Fix the problem that tab click on the page did not response.

2019-08-21 Thread GitBox
tuteng commented on issue #4991: [website] Fix the problem that tab click on the page did not response. URL: https://github.com/apache/pulsar/pull/4991#issuecomment-523366068 run Java8 tests run Integration tests This is

[GitHub] [pulsar] congbobo184 commented on issue #4840: Pulsar schema admin API doesn't support uploading key/value schema

2019-08-21 Thread GitBox
congbobo184 commented on issue #4840: Pulsar schema admin API doesn't support uploading key/value schema URL: https://github.com/apache/pulsar/issues/4840#issuecomment-523365857 #4877 this PR didn't fix it, i will fix ti and move it to 2.5.0

[GitHub] [pulsar] frank-dkvan commented on issue #4976: Pulsar SQL can not query the latest ingested event in a topic , but Pulsar Consumer/Read always can.

2019-08-21 Thread GitBox
frank-dkvan commented on issue #4976: Pulsar SQL can not query the latest ingested event in a topic , but Pulsar Consumer/Read always can. URL: https://github.com/apache/pulsar/issues/4976#issuecomment-523364097 @sijie got it! thanks

[GitHub] [pulsar] tuteng opened a new issue #4997: [Pulsar-IO]Nar package for connector not generated for lume,

2019-08-21 Thread GitBox
tuteng opened a new issue #4997: [Pulsar-IO]Nar package for connector not generated for lume, URL: https://github.com/apache/pulsar/issues/4997 **Describe the bug** A clear and concise description of what the bug is. **To Reproduce** Steps to reproduce the behavior: 1. Go

[GitHub] [pulsar] congbobo184 commented on issue #4831: Pulsar schema doesn't do compatibility check for primitive types

2019-08-21 Thread GitBox
congbobo184 commented on issue #4831: Pulsar schema doesn't do compatibility check for primitive types URL: https://github.com/apache/pulsar/issues/4831#issuecomment-523357885 how we check compatibility for INT8 and INT16? we allow they decode the message for themself or allow INT16 read

[GitHub] [pulsar] freeznet opened a new pull request #4996: [k8s] convert to valid pod name part with k8s function runtime

2019-08-21 Thread GitBox
freeznet opened a new pull request #4996: [k8s] convert to valid pod name part with k8s function runtime URL: https://github.com/apache/pulsar/pull/4996 ### Motivation k8s runtime use tenant, namespace, functionName to create k8s pod name, but functionName might not meet the k8s

[GitHub] [pulsar] guillaume-braibant commented on issue #4978: Pulsar - K8S : admin pod should use the broker service instead of proxy service to discover brokers

2019-08-21 Thread GitBox
guillaume-braibant commented on issue #4978: Pulsar - K8S : admin pod should use the broker service instead of proxy service to discover brokers URL: https://github.com/apache/pulsar/issues/4978#issuecomment-523353870 Pull request : #4995

[GitHub] [pulsar] guillaume-braibant opened a new pull request #4995: Admin pod should use broker service to discover broker pods

2019-08-21 Thread GitBox
guillaume-braibant opened a new pull request #4995: Admin pod should use broker service to discover broker pods URL: https://github.com/apache/pulsar/pull/4995 ### Motivation Add enhancement proposed in #4978 ### Changes

[GitHub] [pulsar] congbobo184 commented on issue #4960: Consumer subscribe with AutoConsumeSchema will throw npe

2019-08-21 Thread GitBox
congbobo184 commented on issue #4960: Consumer subscribe with AutoConsumeSchema will throw npe URL: https://github.com/apache/pulsar/pull/4960#issuecomment-523346421 run cpp tests This is an automated message from the

[GitHub] [pulsar] fxbing opened a new issue #4994: Non-partitioned topic with '-partition-[number]' suffix conflicts with partitioned topic

2019-08-21 Thread GitBox
fxbing opened a new issue #4994: Non-partitioned topic with '-partition-[number]' suffix conflicts with partitioned topic URL: https://github.com/apache/pulsar/issues/4994 **Is your feature request related to a problem? Please describe.** First, create a non-partitioned topic with

[GitHub] [pulsar] codelipenghui commented on issue #4621: [PIP-38] Support batch receive in java client.

2019-08-21 Thread GitBox
codelipenghui commented on issue #4621: [PIP-38] Support batch receive in java client. URL: https://github.com/apache/pulsar/pull/4621#issuecomment-523328823 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] zymap opened a new pull request #4993: [Transaction] handle command send with transaction messages

2019-08-21 Thread GitBox
zymap opened a new pull request #4993: [Transaction] handle command send with transaction messages URL: https://github.com/apache/pulsar/pull/4993 *Motivation* Send transaction messages with `CommansSend` *Modifications* - handle when the send messages are

[GitHub] [pulsar] congbobo184 commented on a change in pull request #4960: Consumer subscribe with AutoConsumeSchema will throw npe

2019-08-21 Thread GitBox
congbobo184 commented on a change in pull request #4960: Consumer subscribe with AutoConsumeSchema will throw npe URL: https://github.com/apache/pulsar/pull/4960#discussion_r316019654 ## File path: