[pulsar] branch asf-site updated: Updated site at revision 144ee74

2018-11-07 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/pulsar.git The following commit(s) were added to refs/heads/asf-site by this push: new d08312b Updated site at revision 144ee74 d0

[GitHub] lgleim opened a new issue #2951: [FAQ] How does Pulsar Differ from Pravega

2018-11-07 Thread GitBox
lgleim opened a new issue #2951: [FAQ] How does Pulsar Differ from Pravega URL: https://github.com/apache/pulsar/issues/2951 While there is much documentation on differences between Kafka and Pulsar, no such information exists for Pulsar and Pravega (https://github.com/pravega/pravega)

[GitHub] HaraldGustafsson opened a new issue #2952: Setting topic permission under load

2018-11-07 Thread GitBox
HaraldGustafsson opened a new issue #2952: Setting topic permission under load URL: https://github.com/apache/pulsar/issues/2952 Expected behavior Do a REST call to admin interface for setting topic permissions like: https://localhost:8081/admin/v2/persistent/tt1/test

[GitHub] jiazhai commented on issue #2430: Namespaces admin commands should be able to distinguish v1 and v2 namespace format

2018-11-07 Thread GitBox
jiazhai commented on issue #2430: Namespaces admin commands should be able to distinguish v1 and v2 namespace format URL: https://github.com/apache/pulsar/issues/2430#issuecomment-436655502 @sijie Seems this is not able to produce. And the code that used in NamespacesImpl.java has already

[GitHub] HaraldGustafsson commented on issue #2952: Setting topic permission under load

2018-11-07 Thread GitBox
HaraldGustafsson commented on issue #2952: Setting topic permission under load URL: https://github.com/apache/pulsar/issues/2952#issuecomment-436658631 Now verified same issue on a full Pulsar setup. This is an automated messa

[GitHub] sijie commented on issue #2430: Namespaces admin commands should be able to distinguish v1 and v2 namespace format

2018-11-07 Thread GitBox
sijie commented on issue #2430: Namespaces admin commands should be able to distinguish v1 and v2 namespace format URL: https://github.com/apache/pulsar/issues/2430#issuecomment-436707584 @jiazhai great! let's close it as "not valid anymore"

[GitHub] sijie closed issue #2430: Namespaces admin commands should be able to distinguish v1 and v2 namespace format

2018-11-07 Thread GitBox
sijie closed issue #2430: Namespaces admin commands should be able to distinguish v1 and v2 namespace format URL: https://github.com/apache/pulsar/issues/2430 This is an automated message from the Apache Git Service. To resp

[GitHub] srkukarni closed pull request #2946: Make sure that the returned object is string for conversion purposes

2018-11-07 Thread GitBox
srkukarni closed pull request #2946: Make sure that the returned object is string for conversion purposes URL: https://github.com/apache/pulsar/pull/2946 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provena

[pulsar] branch master updated: Make sure that the returned object is string for conversion purposes (#2946)

2018-11-07 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk 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 810b95f Make sure that the returned object i

[GitHub] srkukarni commented on a change in pull request #2943: Issue #2657: change function cli getstate to use REST endpoint

2018-11-07 Thread GitBox
srkukarni commented on a change in pull request #2943: Issue #2657: change function cli getstate to use REST endpoint URL: https://github.com/apache/pulsar/pull/2943#discussion_r231605507 ## File path: pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdFunctions.java

[GitHub] rdhabalia opened a new pull request #2953: Avoid debug noise by consumer ack-tracker

2018-11-07 Thread GitBox
rdhabalia opened a new pull request #2953: Avoid debug noise by consumer ack-tracker URL: https://github.com/apache/pulsar/pull/2953 ### Motivation Consumer Ack-tracker continuously logs below debug-log and flushing without any acking activity. ``` 10:16:44.094 [pulsar-client

[GitHub] srkukarni commented on issue #2944: Make Log a logger instead of Logger Adapter

2018-11-07 Thread GitBox
srkukarni commented on issue #2944: Make Log a logger instead of Logger Adapter URL: https://github.com/apache/pulsar/pull/2944#issuecomment-436751288 rerun integration tests This is an automated message from the Apache Git Se

[GitHub] rdhabalia opened a new pull request #2954: Broker considers fail-over consumer priority-level

2018-11-07 Thread GitBox
rdhabalia opened a new pull request #2954: Broker considers fail-over consumer priority-level URL: https://github.com/apache/pulsar/pull/2954 ### Motivation Right now, Broker selects failover consumer based on lexicographical sorting on consumer name. However, one of the user has us

[GitHub] srkukarni commented on issue #2944: Make Log a logger instead of Logger Adapter

2018-11-07 Thread GitBox
srkukarni commented on issue #2944: Make Log a logger instead of Logger Adapter URL: https://github.com/apache/pulsar/pull/2944#issuecomment-436767405 rerun integration tests This is an automated message from the Apache Git Se

[GitHub] jerrypeng commented on issue #2930: Expose metrics via http port in function instance

2018-11-07 Thread GitBox
jerrypeng commented on issue #2930: Expose metrics via http port in function instance URL: https://github.com/apache/pulsar/pull/2930#issuecomment-436759784 rerun integration tests This is an automated message from the Apache

[GitHub] sijie commented on issue #2953: Avoid debug noise by consumer ack-tracker

2018-11-07 Thread GitBox
sijie commented on issue #2953: Avoid debug noise by consumer ack-tracker URL: https://github.com/apache/pulsar/pull/2953#issuecomment-436760387 run java8 tests run integration tests This is an automated message from the Ap

[GitHub] rdhabalia opened a new pull request #2955: Remove conflicting log4j2 policies

2018-11-07 Thread GitBox
rdhabalia opened a new pull request #2955: Remove conflicting log4j2 policies URL: https://github.com/apache/pulsar/pull/2955 ### Motivation After moving to log4j2, issue was that sometimes our log files were not rolling over properly and rolled-over archived file was not complete an

[GitHub] sijie commented on issue #2936: [io][docs] introduce annotations for generating connector yaml config files

2018-11-07 Thread GitBox
sijie commented on issue #2936: [io][docs] introduce annotations for generating connector yaml config files URL: https://github.com/apache/pulsar/pull/2936#issuecomment-436761112 run integration tests This is an automated mes

[GitHub] merlimat commented on issue #2898: Remove guava usage in pulsar-storm

2018-11-07 Thread GitBox
merlimat commented on issue #2898: Remove guava usage in pulsar-storm URL: https://github.com/apache/pulsar/pull/2898#issuecomment-436769120 run integration tests This is an automated message from the Apache Git Service. To re

[GitHub] rdhabalia commented on issue #2954: Broker considers fail-over consumer priority-level

2018-11-07 Thread GitBox
rdhabalia commented on issue #2954: Broker considers fail-over consumer priority-level URL: https://github.com/apache/pulsar/pull/2954#issuecomment-436771567 > Can you also clarify on the ConsumerBuilder javadoc how the priority is being used? yes, added. --

[GitHub] merlimat commented on issue #2935: Ensure broker is fully boostrapped before load manager register itself

2018-11-07 Thread GitBox
merlimat commented on issue #2935: Ensure broker is fully boostrapped before load manager register itself URL: https://github.com/apache/pulsar/pull/2935#issuecomment-436779833 run integration tests This is an automat

[GitHub] merlimat closed pull request #2898: Remove guava usage in pulsar-storm

2018-11-07 Thread GitBox
merlimat closed pull request #2898: Remove guava usage in pulsar-storm URL: https://github.com/apache/pulsar/pull/2898 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull requ

[pulsar] branch master updated: Remove guava usage in pulsar-storm (#2898)

2018-11-07 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 6fd7406 Remove guava usage in pulsar-storm (#28

[GitHub] sijie commented on issue #2952: Setting topic permission under load

2018-11-07 Thread GitBox
sijie commented on issue #2952: Setting topic permission under load URL: https://github.com/apache/pulsar/issues/2952#issuecomment-436786902 I think the problem might be related to concurrent updates on namespace policy. we can probably handle the bad version and retry, or put all the upda

[GitHub] srkukarni commented on issue #2944: Make Log a logger instead of Logger Adapter

2018-11-07 Thread GitBox
srkukarni commented on issue #2944: Make Log a logger instead of Logger Adapter URL: https://github.com/apache/pulsar/pull/2944#issuecomment-436787301 rerun integration tests This is an automated message from the Apache Git Se

[GitHub] srkukarni closed pull request #2947: Check for presence of namespace/tenant before creating function

2018-11-07 Thread GitBox
srkukarni closed pull request #2947: Check for presence of namespace/tenant before creating function URL: https://github.com/apache/pulsar/pull/2947 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[pulsar] branch master updated: Check for presence of namespace/tenant before creating function (#2947)

2018-11-07 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk 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 ec170b0 Check for presence of namespace/tena

[GitHub] rdhabalia opened a new pull request #2956: Derive np topic worker threads based on system config

2018-11-07 Thread GitBox
rdhabalia opened a new pull request #2956: Derive np topic worker threads based on system config URL: https://github.com/apache/pulsar/pull/2956 ### Motivation Derive np topic worker threads based on system config This

[GitHub] sijie closed issue #2343: Enable state in cluster mode

2018-11-07 Thread GitBox
sijie closed issue #2343: Enable state in cluster mode URL: https://github.com/apache/pulsar/issues/2343 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the UR

[GitHub] sijie commented on issue #2343: Enable state in cluster mode

2018-11-07 Thread GitBox
sijie commented on issue #2343: Enable state in cluster mode URL: https://github.com/apache/pulsar/issues/2343#issuecomment-436789375 This is done by #2706 This is an automated message from the Apache Git Service. To respond

[GitHub] sijie closed issue #2314: TLS auth not supported for https lookup in C++ client

2018-11-07 Thread GitBox
sijie closed issue #2314: TLS auth not supported for https lookup in C++ client URL: https://github.com/apache/pulsar/issues/2314 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] sijie commented on issue #2314: TLS auth not supported for https lookup in C++ client

2018-11-07 Thread GitBox
sijie commented on issue #2314: TLS auth not supported for https lookup in C++ client URL: https://github.com/apache/pulsar/issues/2314#issuecomment-436789853 this is fixed by #2350 This is an automated message from the Apac

[GitHub] sijie closed issue #2289: Broker suddenly goes down

2018-11-07 Thread GitBox
sijie closed issue #2289: Broker suddenly goes down URL: https://github.com/apache/pulsar/issues/2289 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL a

[GitHub] HaraldGustafsson commented on issue #2952: Setting topic permission under load

2018-11-07 Thread GitBox
HaraldGustafsson commented on issue #2952: Setting topic permission under load URL: https://github.com/apache/pulsar/issues/2952#issuecomment-436793483 Yes, the zookeeper bad version implies that pulsar have concurrent updates. At least a retry is needed. Don't know if a queue would help,

[GitHub] srkukarni closed pull request #2944: Make Log a logger instead of Logger Adapter

2018-11-07 Thread GitBox
srkukarni closed pull request #2944: Make Log a logger instead of Logger Adapter URL: https://github.com/apache/pulsar/pull/2944 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[pulsar] branch master updated: Make Log a logger instead of Logger Adapter (#2944)

2018-11-07 Thread sanjeevrk
This is an automated email from the ASF dual-hosted git repository. sanjeevrk 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 988b55d Make Log a logger instead of Logger

[pulsar] branch master updated: Remove conflicting log4j2 policies (#2955)

2018-11-07 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 43da26d Remove conflicting log4j2 policies (#29

[GitHub] merlimat closed pull request #2955: Remove conflicting log4j2 policies

2018-11-07 Thread GitBox
merlimat closed pull request #2955: Remove conflicting log4j2 policies URL: https://github.com/apache/pulsar/pull/2955 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull requ

[GitHub] aahmed-se commented on issue #2891: Expose read and connect timeout in PulsarAdmin

2018-11-07 Thread GitBox
aahmed-se commented on issue #2891: Expose read and connect timeout in PulsarAdmin URL: https://github.com/apache/pulsar/issues/2891#issuecomment-436807651 https://github.com/apache/pulsar/pull/2957 This is an automated messa

[GitHub] aahmed-se opened a new pull request #2957: Add Connect and Read timeout to pulsar admin

2018-11-07 Thread GitBox
aahmed-se opened a new pull request #2957: Add Connect and Read timeout to pulsar admin URL: https://github.com/apache/pulsar/pull/2957 We add a timeout to the admin client of 30 seconds for both connection and server response. -

[GitHub] aahmed-se commented on issue #2891: Expose read and connect timeout in PulsarAdmin

2018-11-07 Thread GitBox
aahmed-se commented on issue #2891: Expose read and connect timeout in PulsarAdmin URL: https://github.com/apache/pulsar/issues/2891#issuecomment-436807709 Will make it configurable also. This is an automated message from the

[GitHub] merlimat commented on a change in pull request #2930: Expose metrics via http port in function instance

2018-11-07 Thread GitBox
merlimat commented on a change in pull request #2930: Expose metrics via http port in function instance URL: https://github.com/apache/pulsar/pull/2930#discussion_r231718007 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/RuntimeUtils.java

[GitHub] jerrypeng commented on a change in pull request #2930: Expose metrics via http port in function instance

2018-11-07 Thread GitBox
jerrypeng commented on a change in pull request #2930: Expose metrics via http port in function instance URL: https://github.com/apache/pulsar/pull/2930#discussion_r231718205 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/RuntimeUtils.java

[GitHub] rdhabalia commented on issue #2916: Fix zk latency metrics collector

2018-11-07 Thread GitBox
rdhabalia commented on issue #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916#issuecomment-436817399 run integration tests This is an automated message from the Apache Git Service. To res

[GitHub] rdhabalia commented on issue #2911: Fix pulsar function-metrics artifacts deploy

2018-11-07 Thread GitBox
rdhabalia commented on issue #2911: Fix pulsar function-metrics artifacts deploy URL: https://github.com/apache/pulsar/pull/2911#issuecomment-436817652 rerun java8 tests This is an automated message from the Apache Git Service

[GitHub] jiazhai commented on issue #2949: default enable cpp client producer batch

2018-11-07 Thread GitBox
jiazhai commented on issue #2949: default enable cpp client producer batch URL: https://github.com/apache/pulsar/pull/2949#issuecomment-436823715 @merlimat Thanks, will check and do the fix This is an automated message from th

[GitHub] rdhabalia closed pull request #2916: Fix zk latency metrics collector

2018-11-07 Thread GitBox
rdhabalia closed pull request #2916: Fix zk latency metrics collector URL: https://github.com/apache/pulsar/pull/2916 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull reque

[pulsar] branch master updated: Fix zk latency metrics collector (#2916)

2018-11-07 Thread rdhabalia
This is an automated email from the ASF dual-hosted git repository. rdhabalia 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 7756231 Fix zk latency metrics collector (#2

[GitHub] rdhabalia opened a new pull request #2958: Add partitioned-topic stats internal admin-api

2018-11-07 Thread GitBox
rdhabalia opened a new pull request #2958: Add partitioned-topic stats internal admin-api URL: https://github.com/apache/pulsar/pull/2958 ### Motivation Soemtimes user has partition-topic with large number of partition (> 100) and in that case, user wants admin-api to get internal-s

[GitHub] merlimat commented on issue #2852: Add connection timeout client configuration option

2018-11-07 Thread GitBox
merlimat commented on issue #2852: Add connection timeout client configuration option URL: https://github.com/apache/pulsar/pull/2852#issuecomment-436836287 run integration tests This is an automated message from the Apache G

[GitHub] rdhabalia commented on issue #2956: Derive np topic worker threads based on system config

2018-11-07 Thread GitBox
rdhabalia commented on issue #2956: Derive np topic worker threads based on system config URL: https://github.com/apache/pulsar/pull/2956#issuecomment-436844181 run integration tests This is an automated message from the Apac

[GitHub] rdhabalia commented on issue #2954: Broker considers fail-over consumer priority-level

2018-11-07 Thread GitBox
rdhabalia commented on issue #2954: Broker considers fail-over consumer priority-level URL: https://github.com/apache/pulsar/pull/2954#issuecomment-436844285 run java8 tests This is an automated message from the Apache Git Se

[GitHub] merlimat commented on issue #2954: Broker considers fail-over consumer priority-level

2018-11-07 Thread GitBox
merlimat commented on issue #2954: Broker considers fail-over consumer priority-level URL: https://github.com/apache/pulsar/pull/2954#issuecomment-436850973 @rdhabalia One more question: how would this impact the assignment across multiple partitions? -

[GitHub] merlimat opened a new pull request #2959: Fixed race condition in schema initialization in partitioned topics

2018-11-07 Thread GitBox
merlimat opened a new pull request #2959: Fixed race condition in schema initialization in partitioned topics URL: https://github.com/apache/pulsar/pull/2959 ### Motivation There is a race condition when producers and consumers are connecting to a new partitioned topic concurrently

[GitHub] k2la opened a new pull request #2960: Fix Get Function Cluster in Master Docs

2018-11-07 Thread GitBox
k2la opened a new pull request #2960: Fix Get Function Cluster in Master Docs URL: https://github.com/apache/pulsar/pull/2960 ### Motivation About #2948 , fix path in master docs. ### Modifications Change `curl -s http://localhost:8080/admin/v2/functions/cluster` to `curl

[GitHub] nanguawu commented on issue #2901: pulsar-io/hdfs Maven compilation failed

2018-11-07 Thread GitBox
nanguawu commented on issue #2901: pulsar-io/hdfs Maven compilation failed URL: https://github.com/apache/pulsar/issues/2901#issuecomment-436889613 ok thanks a lot. This is an automated message from the Apache Git Service. To