[GitHub] ambition119 commented on issue #3368: Issue #3290 Pulsar IO connector for Hbase sink

2019-01-14 Thread GitBox
ambition119 commented on issue #3368: Issue #3290 Pulsar IO connector for Hbase sink URL: https://github.com/apache/pulsar/pull/3368#issuecomment-454285119 run Integration Tests This is an automated message from the Apache Gi

[GitHub] foreversunyao commented on issue #3365: create_topic_with_check_namespaces_cluster

2019-01-14 Thread GitBox
foreversunyao commented on issue #3365: create_topic_with_check_namespaces_cluster URL: https://github.com/apache/pulsar/pull/3365#issuecomment-454252714 @sijie I added a new function(**validateGlobalNamespaceOwnership**) in the **AdminResource.java**(looks like this one supports the valid

[GitHub] lovelle edited a comment on issue #3312: Prevent dup consumers on same client cnx with shared subscription

2019-01-14 Thread GitBox
lovelle edited a comment on issue #3312: Prevent dup consumers on same client cnx with shared subscription URL: https://github.com/apache/pulsar/pull/3312#issuecomment-454201030 @sijie added test exploiting the bug, from my standpoint this pr is ready :+1: ---

[GitHub] ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247713527 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/n

[GitHub] merlimat commented on issue #3371: Fixed links to RPM/Deb package signature files

2019-01-14 Thread GitBox
merlimat commented on issue #3371: Fixed links to RPM/Deb package signature files URL: https://github.com/apache/pulsar/pull/3371#issuecomment-454208585 run integration tests This is an automated message from the Apache Git S

[GitHub] lovelle commented on issue #3312: Prevent dup consumers on same client cnx with shared subscription

2019-01-14 Thread GitBox
lovelle commented on issue #3312: Prevent dup consumers on same client cnx with shared subscription URL: https://github.com/apache/pulsar/pull/3312#issuecomment-454201030 @sijie added tests exploiting the bug, from my standpoint this pr is ready :+1: -

[GitHub] sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247698579 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/net/S

[GitHub] sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247697736 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/net/S

[GitHub] ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247696311 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/n

[GitHub] sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247694562 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/net/S

[GitHub] ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247689067 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/n

[GitHub] sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247684422 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/net/S

[GitHub] ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247683472 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/n

[GitHub] ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247683250 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/n

[GitHub] sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
sijie commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247672889 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/net/S

[GitHub] merlimat opened a new pull request #3371: Fixed links to RPM/Deb package signature files

2019-01-14 Thread GitBox
merlimat opened a new pull request #3371: Fixed links to RPM/Deb package signature files URL: https://github.com/apache/pulsar/pull/3371 ### Motivation Fixes #3370 . The links should point to ASF dist server as the source of truth for package signatures. ---

[GitHub] ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

2019-01-14 Thread GitBox
ivankelly commented on a change in pull request #3249: [client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url URL: https://github.com/apache/pulsar/pull/3249#discussion_r247620616 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/n

[GitHub] ivankelly commented on issue #3218: Pulsar client should be able to connect to brokers without DNS

2019-01-14 Thread GitBox
ivankelly commented on issue #3218: Pulsar client should be able to connect to brokers without DNS URL: https://github.com/apache/pulsar/issues/3218#issuecomment-454124112 > however I removed "metadata format" and "proxy" We should still test these, to ensure they do work. I would e

[pulsar] Diff for: [GitHub] srkukarni merged pull request #3363: Enable stats to be recovered by Kubernetes runtime

2019-01-14 Thread GitBox
diff --git a/pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/JavaInstanceMain.java b/pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/JavaInstanceMain.java index 3a0a404e69..43467ae2b9 100644 --- a/pulsar-functions/runtime/src/main/java/org/a

[pulsar] branch master updated: Enable stats to be recovered by Kubernetes runtime (#3363)

2019-01-14 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 63a0491 Enable stats to be recovered by Kube

[GitHub] srkukarni merged pull request #3363: Enable stats to be recovered by Kubernetes runtime

2019-01-14 Thread GitBox
srkukarni merged pull request #3363: Enable stats to be recovered by Kubernetes runtime URL: https://github.com/apache/pulsar/pull/3363 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, commits@pulsar.apache.org -

[pulsar] Diff for: [GitHub] srkukarni merged pull request #3324: Add Windowfunction interface to functions api

2019-01-14 Thread GitBox
diff --git a/pulsar-functions/instance/src/main/java/org/apache/pulsar/functions/windowing/WindowContext.java b/pulsar-functions/api-java/src/main/java/org/apache/pulsar/functions/api/WindowContext.java similarity index 99% rename from pulsar-functions/instance/src/main/java/org/apache/pulsar/f

[GitHub] srkukarni merged pull request #3324: Add Windowfunction interface to functions api

2019-01-14 Thread GitBox
srkukarni merged pull request #3324: Add Windowfunction interface to functions api URL: https://github.com/apache/pulsar/pull/3324 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, commits@pulsar.apache.org --

[pulsar] branch master updated: Add Windowfunction interface to functions api (#3324)

2019-01-14 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 bf2a9c7 Add Windowfunction interface to func

[pulsar] Diff for: [GitHub] srkukarni merged pull request #3299: Cleanup consumer subscriptions and fix graceful shutdown for functions/sinks

2019-01-14 Thread GitBox
diff --git a/pulsar-broker/src/test/java/org/apache/pulsar/functions/worker/PulsarWorkerAssignmentTest.java b/pulsar-broker/src/test/java/org/apache/pulsar/functions/worker/PulsarWorkerAssignmentTest.java index 5cdf5a446a..4b4597ec34 100644 --- a/pulsar-broker/src/test/java/org/apache/pulsar/fu

[pulsar] branch master updated: Cleanup consumer subscriptions and fix graceful shutdown for functions/sinks (#3299)

2019-01-14 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 034f6ba Cleanup consumer subscriptions and

[GitHub] srkukarni merged pull request #3299: Cleanup consumer subscriptions and fix graceful shutdown for functions/sinks

2019-01-14 Thread GitBox
srkukarni merged pull request #3299: Cleanup consumer subscriptions and fix graceful shutdown for functions/sinks URL: https://github.com/apache/pulsar/pull/3299 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, commits@pulsar.apache.org

[GitHub] srkukarni commented on issue #3368: Issue #3290 Pulsar IO connector for Hbase sink

2019-01-14 Thread GitBox
srkukarni commented on issue #3368: Issue #3290 Pulsar IO connector for Hbase sink URL: https://github.com/apache/pulsar/pull/3368#issuecomment-454098567 rerun java tests This is an automated message from the Apache Git Servi

[GitHub] srkukarni commented on a change in pull request #3324: Add Windowfunction interface to functions api

2019-01-14 Thread GitBox
srkukarni commented on a change in pull request #3324: Add Windowfunction interface to functions api URL: https://github.com/apache/pulsar/pull/3324#discussion_r247592146 ## File path: pulsar-functions/api-java/src/main/java/org/apache/pulsar/functions/api/WindowContext.java #

[GitHub] srkukarni commented on a change in pull request #3299: Cleanup consumer subscriptions and fix graceful shutdown for functions/sinks

2019-01-14 Thread GitBox
srkukarni commented on a change in pull request #3299: Cleanup consumer subscriptions and fix graceful shutdown for functions/sinks URL: https://github.com/apache/pulsar/pull/3299#discussion_r247589416 ## File path: pulsar-functions/instance/src/main/java/org/apache/pulsar/functio

[GitHub] alexandrfox opened a new issue #3370: CPP Client RPM sha512 links are broken

2019-01-14 Thread GitBox
alexandrfox opened a new issue #3370: CPP Client RPM sha512 links are broken URL: https://github.com/apache/pulsar/issues/3370 Hey folks, sha512 signature RPM links for cpp client (the ones here: https://pulsar.apache.org/docs/en/client-libraries-cpp/) are broken on some mirrors, e.

[GitHub] sautran opened a new issue #3369: IndexOutOfBoundsException

2019-01-14 Thread GitBox
sautran opened a new issue #3369: IndexOutOfBoundsException URL: https://github.com/apache/pulsar/issues/3369 Steps: 1. unzip tar ball: tar -xf apache-pulsar-2.2.1-bin.tar.gz , cd the directory 2. ./pulsar standalone 3. change the topic to following in org.apache.pulsar.cli

[GitHub] ambition119 closed pull request #3291: Issue #3290 Pulsar IO connector for Hbase sink

2019-01-14 Thread GitBox
ambition119 closed pull request #3291: Issue #3290 Pulsar IO connector for Hbase sink URL: https://github.com/apache/pulsar/pull/3291 This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] ambition119 opened a new pull request #3368: Issue #3290 Pulsar IO connector for Hbase sink

2019-01-14 Thread GitBox
ambition119 opened a new pull request #3368: Issue #3290 Pulsar IO connector for Hbase sink URL: https://github.com/apache/pulsar/pull/3368 ### Motivation Explain here the context, and why you're making that change. What is the problem you're trying to solve. ### Modificati

[GitHub] ivankelly commented on issue #2880: Unauthorised when deleting namespace with superuser role

2019-01-14 Thread GitBox
ivankelly commented on issue #2880: Unauthorised when deleting namespace with superuser role URL: https://github.com/apache/pulsar/issues/2880#issuecomment-454058868 @merlimat @jai1 could I get some input on my previous comment? -

[GitHub] jiazhai opened a new issue #3367: Dashboard docker shows no data

2019-01-14 Thread GitBox
jiazhai opened a new issue #3367: Dashboard docker shows no data URL: https://github.com/apache/pulsar/issues/3367 **Describe the bug** Some user reported that Dashboard docker shows no data by following instruction of [dashboard](http://pulsar.apache.org/docs/en/administration-dashboa

[GitHub] foreversunyao commented on issue #3365: create_topic_with_check_namespaces_cluster

2019-01-14 Thread GitBox
foreversunyao commented on issue #3365: create_topic_with_check_namespaces_cluster URL: https://github.com/apache/pulsar/pull/3365#issuecomment-453996805 run integration tests This is an automated message from the Apache Git