[GitHub] rdhabalia commented on issue #2722: local function-worker use localhost broker service url

2018-10-04 Thread GitBox
rdhabalia commented on issue #2722: local function-worker use localhost broker service url URL: https://github.com/apache/pulsar/pull/2722#issuecomment-427251908 rerun java8 tests This is an automated message from the Apache

[GitHub] rdhabalia commented on a change in pull request #2730: [schema] provide a flag to disable/enable schema validation on broker and change default bytes producer to use `AUTO_PRODUCE_BYTES`

2018-10-04 Thread GitBox
rdhabalia commented on a change in pull request #2730: [schema] provide a flag to disable/enable schema validation on broker and change default bytes producer to use `AUTO_PRODUCE_BYTES` URL: https://github.com/apache/pulsar/pull/2730#discussion_r222895326 ## File path:

[GitHub] hrsakai opened a new issue #2731: Ws client can't receive any more messages when number of timed-out messages reaches receiverQueueSize

2018-10-04 Thread GitBox
hrsakai opened a new issue #2731: Ws client can't receive any more messages when number of timed-out messages reaches receiverQueueSize URL: https://github.com/apache/pulsar/issues/2731 Ws proxy controls message-delivery to ws client by using `pendingMessages`. `pendingMessages`

[pulsar] branch master updated (828e22d -> 90c365c)

2018-10-04 Thread mmerli
This is an automated email from the ASF dual-hosted git repository. mmerli pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 828e22d Removed organization affiliations from website (#2715) add 90c365c Removed shading relocations for

[GitHub] merlimat closed pull request #2191: Removed shading relocations for Circe-checksum and lz4 libraries

2018-10-04 Thread GitBox
merlimat closed pull request #2191: Removed shading relocations for Circe-checksum and lz4 libraries URL: https://github.com/apache/pulsar/pull/2191 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:

[GitHub] srkukarni opened a new pull request #2729: Added ability a download dependencies in Kubernetes

2018-10-04 Thread GitBox
srkukarni opened a new pull request #2729: Added ability a download dependencies in Kubernetes URL: https://github.com/apache/pulsar/pull/2729 ### Motivation While pulsar python functions support wheel files, the dependencies don't get extracted using just the zipfile that we

[GitHub] srkukarni commented on issue #2727: Fix download path in Kubernetes

2018-10-04 Thread GitBox
srkukarni commented on issue #2727: Fix download path in Kubernetes URL: https://github.com/apache/pulsar/pull/2727#issuecomment-427219054 run cpp tests This is an automated message from the Apache Git Service. To respond to

[GitHub] sijie commented on issue #2725: Fixed intermittent test failures with "bind error"

2018-10-04 Thread GitBox
sijie commented on issue #2725: Fixed intermittent test failures with "bind error" URL: https://github.com/apache/pulsar/pull/2725#issuecomment-427213621 run integration tests This is an automated message from the Apache Git

[GitHub] sijie commented on a change in pull request #2728: [function][runtime] NPE at RuntimeSpawner

2018-10-04 Thread GitBox
sijie commented on a change in pull request #2728: [function][runtime] NPE at RuntimeSpawner URL: https://github.com/apache/pulsar/pull/2728#discussion_r222867278 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/RuntimeSpawner.java

[GitHub] srkukarni commented on a change in pull request #2728: [function][runtime] NPE at RuntimeSpawner

2018-10-04 Thread GitBox
srkukarni commented on a change in pull request #2728: [function][runtime] NPE at RuntimeSpawner URL: https://github.com/apache/pulsar/pull/2728#discussion_r222866647 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/RuntimeSpawner.java

[pulsar] branch asf-site updated: Updated site at revision 828e22d

2018-10-04 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 56ee21c Updated site at revision 828e22d

[GitHub] jerrypeng commented on issue #2642: Duplicate classes caught by maven-enforcer-plugin

2018-10-04 Thread GitBox
jerrypeng commented on issue #2642: Duplicate classes caught by maven-enforcer-plugin URL: https://github.com/apache/pulsar/issues/2642#issuecomment-427212316 @MarkAddison the multiple version of dependencies has to deal with protobuf 2.0 vs 3.0. In more recent code we started using

[GitHub] srkukarni commented on issue #2726: Mock out the settting up k8 client part

2018-10-04 Thread GitBox
srkukarni commented on issue #2726: Mock out the settting up k8 client part URL: https://github.com/apache/pulsar/pull/2726#issuecomment-427211841 run cpp tests This is an automated message from the Apache Git Service. To

[GitHub] sijie opened a new pull request #2728: [function][runtime] NPE at RuntimeSpawner

2018-10-04 Thread GitBox
sijie opened a new pull request #2728: [function][runtime] NPE at RuntimeSpawner URL: https://github.com/apache/pulsar/pull/2728 *Motivation* getFunctionStat call can happen before spanwer starts the runtime *Changes* Add null checks and also change the order of closing

[GitHub] srkukarni opened a new pull request #2727: Fix download path in Kubernetes

2018-10-04 Thread GitBox
srkukarni opened a new pull request #2727: Fix download path in Kubernetes URL: https://github.com/apache/pulsar/pull/2727 ### Motivation Ensure that user code file is always downloaded at pulsarRootDir. ### Modifications Describe the modifications you've done. ###

[GitHub] srkukarni opened a new pull request #2726: Mock out the settting up k8 client part

2018-10-04 Thread GitBox
srkukarni opened a new pull request #2726: Mock out the settting up k8 client part URL: https://github.com/apache/pulsar/pull/2726 ### Motivation No need to create any client during unittests ### Modifications Describe the modifications you've done. ### Result

[GitHub] merlimat closed pull request #2715: Removed organization affiliations from website

2018-10-04 Thread GitBox
merlimat closed pull request #2715: Removed organization affiliations from website URL: https://github.com/apache/pulsar/pull/2715 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

[pulsar] branch master updated: Removed organization affiliations from website (#2715)

2018-10-04 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 828e22d Removed organization affiliations from

[GitHub] merlimat commented on issue #2725: Fixed intermittent test failures with "bind error"

2018-10-04 Thread GitBox
merlimat commented on issue #2725: Fixed intermittent test failures with "bind error" URL: https://github.com/apache/pulsar/pull/2725#issuecomment-427201186 run integration tests This is an automated message from the Apache

[GitHub] merlimat commented on issue #2725: Fixed intermittent test failures with "bind error"

2018-10-04 Thread GitBox
merlimat commented on issue #2725: Fixed intermittent test failures with "bind error" URL: https://github.com/apache/pulsar/pull/2725#issuecomment-427199804 run integration tests This is an automated message from the Apache

[GitHub] merlimat commented on issue #2191: Removed shading relocations for Circe-checksum and lz4 libraries

2018-10-04 Thread GitBox
merlimat commented on issue #2191: Removed shading relocations for Circe-checksum and lz4 libraries URL: https://github.com/apache/pulsar/pull/2191#issuecomment-427198832 run java8 tests This is an automated message from the

[GitHub] merlimat opened a new pull request #2725: Fixed intermittent test failures with "bind error"

2018-10-04 Thread GitBox
merlimat opened a new pull request #2725: Fixed intermittent test failures with "bind error" URL: https://github.com/apache/pulsar/pull/2725 ### Motivation There are several tests that occasionally are failing with bind error. Eg. from `LoadBalancerTest`: ```

[GitHub] sijie commented on issue #2724: [functions][stats] don't generate function stats at worker service if runtime is k8s

2018-10-04 Thread GitBox
sijie commented on issue #2724: [functions][stats] don't generate function stats at worker service if runtime is k8s URL: https://github.com/apache/pulsar/pull/2724#issuecomment-427198519 This change is based on #2723 This

[GitHub] sijie opened a new pull request #2724: [functions][stats] don't generate function stats at worker service if runtime is k8s

2018-10-04 Thread GitBox
sijie opened a new pull request #2724: [functions][stats] don't generate function stats at worker service if runtime is k8s URL: https://github.com/apache/pulsar/pull/2724 *Motivation* k8s runtime doesn't support generating function stats at worker service right now. so skip it

[GitHub] rdhabalia commented on issue #2720: Improving stop and start function instances

2018-10-04 Thread GitBox
rdhabalia commented on issue #2720: Improving stop and start function instances URL: https://github.com/apache/pulsar/issues/2720#issuecomment-427189673 yes, right now, it is not persisting the state of the function therefore, stop/restart was the transient. I wanted to do it at that time

[GitHub] sijie opened a new pull request #2723: [functions][stats] NPE in FunctionStatsGenerator when worker service is not ready

2018-10-04 Thread GitBox
sijie opened a new pull request #2723: [functions][stats] NPE in FunctionStatsGenerator when worker service is not ready URL: https://github.com/apache/pulsar/pull/2723 *Motivation* NullPointerException was thrown when function worker is running as part of broker and metrics

[GitHub] rdhabalia opened a new pull request #2722: local function-worker use localhost broker service url

2018-10-04 Thread GitBox
rdhabalia opened a new pull request #2722: local function-worker use localhost broker service url URL: https://github.com/apache/pulsar/pull/2722 ### Motivation If pulsar service starts broker and worker in the same process then worker should not use broker-advertised address but

[GitHub] sijie opened a new pull request #2721: [functions][conf] Update functions_worker yml with all the possible settings

2018-10-04 Thread GitBox
sijie opened a new pull request #2721: [functions][conf] Update functions_worker yml with all the possible settings URL: https://github.com/apache/pulsar/pull/2721 *Motivation* People shouldn't be looking into code to understand how to configure a function worker. We should have

[GitHub] jerrypeng opened a new issue #2720: Improving stop and start function instances

2018-10-04 Thread GitBox
jerrypeng opened a new issue #2720: Improving stop and start function instances URL: https://github.com/apache/pulsar/issues/2720 Currently, when a user submits a request to stop or start a function instance, the worker that receives the request attempt to redirect the stop or start

[GitHub] sijie closed pull request #2716: [tests][integration] Fix errors in integration tests

2018-10-04 Thread GitBox
sijie closed pull request #2716: [tests][integration] Fix errors in integration tests URL: https://github.com/apache/pulsar/pull/2716 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

[GitHub] grantwwu opened a new issue #2719: Fix various bugs in the Go client doc examples

2018-10-04 Thread GitBox
grantwwu opened a new issue #2719: Fix various bugs in the Go client doc examples URL: https://github.com/apache/pulsar/issues/2719 Various parts of the Go client doc examples do not work with the existing API. For example,

[GitHub] grantwwu commented on issue #2703: Improve Go/macOS C++ client installation documentation

2018-10-04 Thread GitBox
grantwwu commented on issue #2703: Improve Go/macOS C++ client installation documentation URL: https://github.com/apache/pulsar/issues/2703#issuecomment-427143864 > But the MacOS brew recipe is still installing 2.1.0 C++ client. We need to push the brew recipe to homebrew and make sure

[GitHub] sijie commented on issue #2717: [schema] use AUTO_PRODUCE schema when possible

2018-10-04 Thread GitBox
sijie commented on issue #2717: [schema] use AUTO_PRODUCE schema when possible URL: https://github.com/apache/pulsar/pull/2717#issuecomment-427134428 hold on merging this. I think a better solution is to turn AUTO_PRODUCE_BYTES as the default. I will send out a separate PR for it.

[GitHub] sijie commented on issue #2716: [tests][integration] Fix errors in integration tests

2018-10-04 Thread GitBox
sijie commented on issue #2716: [tests][integration] Fix errors in integration tests URL: https://github.com/apache/pulsar/pull/2716#issuecomment-427124766 run integration tests This is an automated message from the Apache

[GitHub] merlimat commented on issue #2715: Removed organization affiliations from website

2018-10-04 Thread GitBox
merlimat commented on issue #2715: Removed organization affiliations from website URL: https://github.com/apache/pulsar/pull/2715#issuecomment-427096937 run java8 tests This is an automated message from the Apache Git

[pulsar] branch master updated: Allow bk-client to support old bk-protocol (#2718)

2018-10-04 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 24749e1 Allow bk-client to support old

[GitHub] merlimat closed pull request #2707: Add System Property Option for Athenz

2018-10-04 Thread GitBox
merlimat closed pull request #2707: Add System Property Option for Athenz URL: https://github.com/apache/pulsar/pull/2707 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

[pulsar] branch master updated: Add System Property Option for Athenz (#2707)

2018-10-04 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 bb945d3 Add System Property Option for Athenz

[GitHub] HaraldGustafsson commented on issue #2708: Python messageId serialize is broken

2018-10-04 Thread GitBox
HaraldGustafsson commented on issue #2708: Python messageId serialize is broken URL: https://github.com/apache/pulsar/issues/2708#issuecomment-427089658 I wrote a simple protobuf generator based on the MessageId, so that I can serialize them while this is fixed. import itertools

[GitHub] macthestack commented on issue #2697: Offloading fails

2018-10-04 Thread GitBox
macthestack commented on issue #2697: Offloading fails URL: https://github.com/apache/pulsar/issues/2697#issuecomment-427086068 Here we go: [stderr](https://storage.googleapis.com/pulsar-topic-offload/stderr/2018/10/04/15%3A00%3A00_15%3A59%3A59_S0.json)

[GitHub] macthestack commented on issue #2697: Offloading fails

2018-10-04 Thread GitBox
macthestack commented on issue #2697: Offloading fails URL: https://github.com/apache/pulsar/issues/2697#issuecomment-427018049 sure, I'll spin up a fresh cluster, thanks for the help This is an automated message from the

[GitHub] ivankelly commented on issue #2697: Offloading fails

2018-10-04 Thread GitBox
ivankelly commented on issue #2697: Offloading fails URL: https://github.com/apache/pulsar/issues/2697#issuecomment-427016929 Could you post the full broker log? This is an automated message from the Apache Git Service.

[GitHub] macthestack commented on issue #2697: Offloading fails

2018-10-04 Thread GitBox
macthestack commented on issue #2697: Offloading fails URL: https://github.com/apache/pulsar/issues/2697#issuecomment-427014844 I actually set it before the broker start in this sequence: ``` bin/apply-config-from-env.py conf/broker.conf && bin/apply-config-from-env.py

[GitHub] ivankelly commented on issue #2697: Offloading fails

2018-10-04 Thread GitBox
ivankelly commented on issue #2697: Offloading fails URL: https://github.com/apache/pulsar/issues/2697#issuecomment-427009921 @macthestack did you restart the broker after updating the configuration? UnsupportedOperationException looks like its coming from the Null offloader.

[GitHub] macthestack commented on issue #2697: Offloading fails

2018-10-04 Thread GitBox
macthestack commented on issue #2697: Offloading fails URL: https://github.com/apache/pulsar/issues/2697#issuecomment-427006978 ``` ### --- Ledger Offloading --- ### # Driver to use to offload old data to long term storage (Possible values: S3)

[pulsar] branch master updated: Added missing brokerServiceURL config keys in proxy.conf (#2713)

2018-10-04 Thread massakam
This is an automated email from the ASF dual-hosted git repository. massakam 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 3c1ce54 Added missing brokerServiceURL

[GitHub] sijie commented on issue #2716: [tests][integration] Fix errors in integration tests

2018-10-04 Thread GitBox
sijie commented on issue #2716: [tests][integration] Fix errors in integration tests URL: https://github.com/apache/pulsar/pull/2716#issuecomment-426945179 run integration tests This is an automated message from the Apache

[GitHub] nkurihar commented on issue #2707: Add System Property Option for Athenz

2018-10-04 Thread GitBox
nkurihar commented on issue #2707: Add System Property Option for Athenz URL: https://github.com/apache/pulsar/pull/2707#issuecomment-426943003 rerun cpp tests This is an automated message from the Apache Git Service. To

[GitHub] massakam commented on issue #2713: Added missing brokerServiceURL config keys in proxy.conf

2018-10-04 Thread GitBox
massakam commented on issue #2713: Added missing brokerServiceURL config keys in proxy.conf URL: https://github.com/apache/pulsar/pull/2713#issuecomment-426925726 rerun cpp tests This is an automated message from the Apache

[GitHub] rdhabalia opened a new pull request #2718: Allow bk-client to support old bk-protocol

2018-10-04 Thread GitBox
rdhabalia opened a new pull request #2718: Allow bk-client to support old bk-protocol URL: https://github.com/apache/pulsar/pull/2718 ### Motivation Need this config due to two reasons: 1. This config is useful in migration process where bk-client and bookies are on different

[GitHub] sijie opened a new pull request #2717: [schema] use AUTO_PRODUCE schema when possible

2018-10-04 Thread GitBox
sijie opened a new pull request #2717: [schema] use AUTO_PRODUCE schema when possible URL: https://github.com/apache/pulsar/pull/2717 *Motivation* We added schema enforcements. However we don't update some cli tools. For example, if you submit an ExclamationFunction and try to