[GitHub] [pulsar] jerrypeng merged pull request #3990: improving getting version and git SHA

2019-04-07 Thread GitBox
jerrypeng merged pull request #3990: improving getting version and git SHA URL: https://github.com/apache/pulsar/pull/3990 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[pulsar] branch master updated: improving getting version and git SHA (#3990)

2019-04-07 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 c706d45 improving getting version and git SH

[pulsar] branch master updated: fix bug when updating function running in effectively-once (#3994)

2019-04-07 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 5a7b177 fix bug when updating function runni

[GitHub] [pulsar] jerrypeng merged pull request #3994: fix bug when updating function running in effectively-once

2019-04-07 Thread GitBox
jerrypeng merged pull request #3994: fix bug when updating function running in effectively-once URL: https://github.com/apache/pulsar/pull/3994 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [pulsar] jerrypeng merged pull request #3980: fix: bug when function auth is not enabled

2019-04-07 Thread GitBox
jerrypeng merged pull request #3980: fix: bug when function auth is not enabled URL: https://github.com/apache/pulsar/pull/3980 This is an automated message from the Apache Git Service. To respond to the message, please log o

[pulsar] branch master updated: Improve error handling for triggering function when input data does not conform with input topic schema (#3995)

2019-04-07 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 43442e7 Improve error handling for triggerin

[pulsar] branch master updated: fix: bug when function auth is not enabled (#3980)

2019-04-07 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 23926ad fix: bug when function auth is not e

[GitHub] [pulsar] jerrypeng merged pull request #3995: Improve error handling for triggering function when there is a schema mismatch

2019-04-07 Thread GitBox
jerrypeng merged pull request #3995: Improve error handling for triggering function when there is a schema mismatch URL: https://github.com/apache/pulsar/pull/3995 This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] wolfstudy commented on issue #3854: [issue #3767] support go function for pulsar

2019-04-07 Thread GitBox
wolfstudy commented on issue #3854: [issue #3767] support go function for pulsar URL: https://github.com/apache/pulsar/pull/3854#issuecomment-480689747 @srkukarni @jerrypeng remove EO(effectively-once) logic, I will do the separate processing in the next pull request, PTAL again. -

[GitHub] [pulsar] sijie merged pull request #3961: [pulsar-broker] add producer/consumer id in error-logging

2019-04-07 Thread GitBox
sijie merged pull request #3961: [pulsar-broker] add producer/consumer id in error-logging URL: https://github.com/apache/pulsar/pull/3961 This is an automated message from the Apache Git Service. To respond to the message,

[pulsar] branch master updated: [pulsar-broker] add producer/consumer id in error-logging (#3961)

2019-04-07 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 b41a020 [pulsar-broker] add producer/consumer id

[GitHub] [pulsar] jerrypeng commented on issue #3980: fix: bug when function auth is not enabled

2019-04-07 Thread GitBox
jerrypeng commented on issue #3980: fix: bug when function auth is not enabled URL: https://github.com/apache/pulsar/pull/3980#issuecomment-480683312 rerun integration tests This is an automated message from the Apache Git Ser

[GitHub] [pulsar] jerrypeng commented on issue #3990: improving getting version and git SHA

2019-04-07 Thread GitBox
jerrypeng commented on issue #3990: improving getting version and git SHA URL: https://github.com/apache/pulsar/pull/3990#issuecomment-480683229 rerun java8 tests This is an automated message from the Apache Git Service. To re

[GitHub] [pulsar] jerrypeng commented on issue #3990: improving getting version and git SHA

2019-04-07 Thread GitBox
jerrypeng commented on issue #3990: improving getting version and git SHA URL: https://github.com/apache/pulsar/pull/3990#issuecomment-480683196 rerun integration tests This is an automated message from the Apache Git

[GitHub] [pulsar] jerrypeng commented on issue #3995: Improve error handling for triggering function when there is a schema mismatch

2019-04-07 Thread GitBox
jerrypeng commented on issue #3995: Improve error handling for triggering function when there is a schema mismatch URL: https://github.com/apache/pulsar/pull/3995#issuecomment-480683059 rerun integration tests This is an auto

[GitHub] [pulsar] jerrypeng commented on issue #3994: fix bug when updating function running in effectively-once

2019-04-07 Thread GitBox
jerrypeng commented on issue #3994: fix bug when updating function running in effectively-once URL: https://github.com/apache/pulsar/pull/3994#issuecomment-480683102 rerun integration tests This is an automated message from t

[GitHub] [pulsar] jerrypeng commented on issue #3999: Don't pass any auth parameters when authentication is disabled

2019-04-07 Thread GitBox
jerrypeng commented on issue #3999: Don't pass any auth parameters when authentication is disabled URL: https://github.com/apache/pulsar/pull/3999#issuecomment-480682954 rerun integration tests This is an automated message fr

[GitHub] [pulsar] jerrypeng commented on issue #3996: [Kubernetes] Added ingress resource to dashboard

2019-04-07 Thread GitBox
jerrypeng commented on issue #3996: [Kubernetes] Added ingress resource to dashboard URL: https://github.com/apache/pulsar/pull/3996#issuecomment-480683015 rerun integration tests This is an automated message from the Apache

[GitHub] [pulsar] jerrypeng closed issue #3998: Failed to build pulsar docker image

2019-04-07 Thread GitBox
jerrypeng closed issue #3998: Failed to build pulsar docker image URL: https://github.com/apache/pulsar/issues/3998 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[pulsar] branch master updated: Fix integration tests by pinning the version of python package (#4000)

2019-04-07 Thread jerrypeng
This is an automated email from the ASF dual-hosted git repository. jerrypeng 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 acb1352 Fix integration tests by pinning the

[GitHub] [pulsar] jerrypeng merged pull request #4000: Fix integration tests by pinning the version of python package

2019-04-07 Thread GitBox
jerrypeng merged pull request #4000: Fix integration tests by pinning the version of python package URL: https://github.com/apache/pulsar/pull/4000 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] jerrypeng commented on issue #3999: Don't pass any auth parameters when authentication is disabled

2019-04-07 Thread GitBox
jerrypeng commented on issue #3999: Don't pass any auth parameters when authentication is disabled URL: https://github.com/apache/pulsar/pull/3999#issuecomment-480679438 rerun java8 tests This is an automated message from the

[GitHub] [pulsar] tuteng commented on issue #4002: Pulsar standalone start failed

2019-04-07 Thread GitBox
tuteng commented on issue #4002: Pulsar standalone start failed URL: https://github.com/apache/pulsar/issues/4002#issuecomment-480677978 ./bin/pulsar standalone -nss start success This is an automated message from the Apach

[GitHub] [pulsar] tuteng opened a new issue #4002: Pulsar standalone start failed

2019-04-07 Thread GitBox
tuteng opened a new issue #4002: Pulsar standalone start failed URL: https://github.com/apache/pulsar/issues/4002 **Describe the bug** A clear and concise description of what the bug is. **To Reproduce** Steps to reproduce the behavior: 1. Go to pulsar 2. ./bin/pulsar stan

[GitHub] [pulsar] wolfstudy opened a new issue #4001: [Release 2.3.1] go version error in DEB package

2019-04-07 Thread GitBox
wolfstudy opened a new issue #4001: [Release 2.3.1] go version error in DEB package URL: https://github.com/apache/pulsar/issues/4001 **Describe the bug** now, `pulsar-client-go` uses the package management method of `go mod`, `go mod` requires that the version of go must be 1.11 or

[GitHub] [pulsar] sijie commented on issue #4000: Fix integration tests by pinning the version of python package

2019-04-07 Thread GitBox
sijie commented on issue #4000: Fix integration tests by pinning the version of python package URL: https://github.com/apache/pulsar/pull/4000#issuecomment-480672378 Cool! I was looking into the same issue for #3998 This is

[pulsar.wiki] branch master updated: Updated PIP 34: Add new subscribe type Key_Failover (markdown)

2019-04-07 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.wiki.git The following commit(s) were added to refs/heads/master by this push: new 0cd56f3 Updated PIP 34: Add new subscribe

[GitHub] [pulsar] srkukarni opened a new pull request #4000: Fix integration tests by pinning the version of python package

2019-04-07 Thread GitBox
srkukarni opened a new pull request #4000: Fix integration tests by pinning the version of python package URL: https://github.com/apache/pulsar/pull/4000 ### Motivation Integration tests are failing currently because there are issues installing psycopg2. So this pr fixes it by pinning t

[GitHub] [pulsar] jerrypeng opened a new pull request #3999: Don't pass any auth parameters when authentication is disabled

2019-04-07 Thread GitBox
jerrypeng opened a new pull request #3999: Don't pass any auth parameters when authentication is disabled URL: https://github.com/apache/pulsar/pull/3999 ### Motivation When authenticationEnabled is set to false, don't set or pass any of the other configurations related to authe

[GitHub] [pulsar] sijie commented on issue #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on issue #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#issuecomment-480659346 @ConcurrencyPractitioner yes. This is an automated message fr

[GitHub] [pulsar] ConcurrencyPractitioner edited a comment on issue #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
ConcurrencyPractitioner edited a comment on issue #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#issuecomment-480658430 @sijie I want to confirm that seek() should be used for this configuration right? From my understandi

[GitHub] [pulsar] ConcurrencyPractitioner commented on issue #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
ConcurrencyPractitioner commented on issue #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#issuecomment-480658430 @sijie I want to confirm that seek() should be used for this configuration right?

[GitHub] [pulsar] sijie opened a new issue #3998: Failed to build pulsar docker image

2019-04-07 Thread GitBox
sijie opened a new issue #3998: Failed to build pulsar docker image URL: https://github.com/apache/pulsar/issues/3998 **Describe the bug** ``` 2019-04-08\T\01:48:35.008 [INFO] Step 4/14 : RUN pip install uwsgi 'Django<2.0' psycopg2 pytz requests 2019-04-08\T\01:48:35.480 [INFO]

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865568 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/client/api/TopicReaderTest.java ###

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865925 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ReaderConfigurationData

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865762 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ConsumerImpl.java #

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865209 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/client/api/TopicReaderTest.java ###

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865688 ## File path: pulsar-client-api/src/main/java/org/apache/pulsar/client/api/ReaderBuilder.java #

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865817 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ReaderBuilderImpl.java

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272866101 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ReaderImpl.java ##

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865594 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/client/api/TopicReaderTest.java ###

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865274 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/client/api/TopicReaderTest.java ###

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272866060 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ReaderConfigurationData

[GitHub] [pulsar] sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
sijie commented on a change in pull request #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#discussion_r272865870 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/ReaderBuilderImpl.java

[GitHub] [pulsar] wolfstudy commented on issue #3854: [issue #3767] support go function for pulsar

2019-04-07 Thread GitBox
wolfstudy commented on issue #3854: [issue #3767] support go function for pulsar URL: https://github.com/apache/pulsar/pull/3854#issuecomment-480652460 thanks @sijie , good idea, i will fix it This is an automated message from

[GitHub] [pulsar] jiazhai commented on issue #3997: Issue 3655: Kerberos authentication for proxy

2019-04-07 Thread GitBox
jiazhai commented on issue #3997: Issue 3655: Kerberos authentication for proxy URL: https://github.com/apache/pulsar/pull/3997#issuecomment-480651911 run integration tests This is an automated message from the Apache Git Serv

[GitHub] [pulsar] sijie commented on issue #3854: [issue #3767] support go function for pulsar

2019-04-07 Thread GitBox
sijie commented on issue #3854: [issue #3767] support go function for pulsar URL: https://github.com/apache/pulsar/pull/3854#issuecomment-480650051 @srkukarni @wolfstudy : can we agree on followings: ``` - fail the instance (panic) in newGoInstance if ProcessingGuarantee (PG) is E

[GitHub] [pulsar] sijie commented on issue #3904: [go schema] support go schema for pulsar-client-go

2019-04-07 Thread GitBox
sijie commented on issue #3904: [go schema] support go schema for pulsar-client-go URL: https://github.com/apache/pulsar/pull/3904#issuecomment-480649568 ping @merlimat - can you review this again? This is an automated messag

[GitHub] [pulsar] sijie commented on issue #3961: [pulsar-broker] add producer/consumer id in error-logging

2019-04-07 Thread GitBox
sijie commented on issue #3961: [pulsar-broker] add producer/consumer id in error-logging URL: https://github.com/apache/pulsar/pull/3961#issuecomment-480649477 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] sijie commented on issue #3945: [#3944][dashboard]Dashboard authentication support for JWT token

2019-04-07 Thread GitBox
sijie commented on issue #3945: [#3944][dashboard]Dashboard authentication support for JWT token URL: https://github.com/apache/pulsar/pull/3945#issuecomment-480649516 run integration tests This is an automated message from t

[GitHub] [pulsar] sijie commented on issue #3958: [Doc] Update "load distribution" as "load balance"

2019-04-07 Thread GitBox
sijie commented on issue #3958: [Doc] Update "load distribution" as "load balance" URL: https://github.com/apache/pulsar/pull/3958#issuecomment-480649494 run java8 tests This is an automated message from the Apache Git Servic

[GitHub] [pulsar] sijie commented on issue #3962: Feature / Interceptor for negative ack redelivery

2019-04-07 Thread GitBox
sijie commented on issue #3962: Feature / Interceptor for negative ack redelivery URL: https://github.com/apache/pulsar/pull/3962#issuecomment-480649448 @merlimat can you check @lovelle 's latest comment? This is an automated

[GitHub] [pulsar] sijie commented on issue #3977: Provide a mechanism to disable uploading distribution packages

2019-04-07 Thread GitBox
sijie commented on issue #3977: Provide a mechanism to disable uploading distribution packages URL: https://github.com/apache/pulsar/pull/3977#issuecomment-480649328 run cpp tests run integration tests This is an automated

[GitHub] [pulsar] sijie commented on issue #3994: fix bug when updating function running in effectively-once

2019-04-07 Thread GitBox
sijie commented on issue #3994: fix bug when updating function running in effectively-once URL: https://github.com/apache/pulsar/pull/3994#issuecomment-480648814 run integration tests This is an automated message from the Apa

[GitHub] [pulsar] sijie commented on issue #3995: Improve error handling for triggering function when there is a schema mismatch

2019-04-07 Thread GitBox
sijie commented on issue #3995: Improve error handling for triggering function when there is a schema mismatch URL: https://github.com/apache/pulsar/pull/3995#issuecomment-480648760 run java8 tests run integration tests Th

[GitHub] [pulsar] sijie commented on issue #3973: Branch 2.2

2019-04-07 Thread GitBox
sijie commented on issue #3973: Branch 2.2 URL: https://github.com/apache/pulsar/pull/3973#issuecomment-480648437 Closed this pull request since it doesn't look like a valid PR. please reopen it if there is a specific problem you are addressing.

[GitHub] [pulsar] sijie closed pull request #3973: Branch 2.2

2019-04-07 Thread GitBox
sijie closed pull request #3973: Branch 2.2 URL: https://github.com/apache/pulsar/pull/3973 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [pulsar] sijie commented on issue #3996: [Kubernetes] Added ingress resource to dashboard

2019-04-07 Thread GitBox
sijie commented on issue #3996: [Kubernetes] Added ingress resource to dashboard URL: https://github.com/apache/pulsar/pull/3996#issuecomment-480648284 run integration tests This is an automated message from the Apache Git Ser

[GitHub] [pulsar] ConcurrencyPractitioner commented on issue #3983: [Issue#3976] [component/client] Adding config for ReaderImpl

2019-04-07 Thread GitBox
ConcurrencyPractitioner commented on issue #3983: [Issue#3976] [component/client] Adding config for ReaderImpl URL: https://github.com/apache/pulsar/pull/3983#issuecomment-480611462 The test added is broken, although I couldn't pinpoint the exact cause. The location where the consumer seek

[GitHub] [pulsar] eolivelli commented on issue #3997: Issue 3655: Kerberos authentication for proxy

2019-04-07 Thread GitBox
eolivelli commented on issue #3997: Issue 3655: Kerberos authentication for proxy URL: https://github.com/apache/pulsar/pull/3997#issuecomment-480611166 Sure This is an automated message from the Apache Git Service. To respon

[GitHub] [pulsar] jiazhai commented on issue #3997: Issue 3655: Kerberos authentication for proxy

2019-04-07 Thread GitBox
jiazhai commented on issue #3997: Issue 3655: Kerberos authentication for proxy URL: https://github.com/apache/pulsar/pull/3997#issuecomment-480597145 run integration tests This is an automated message from the Apache Git Serv

[GitHub] [pulsar] jiazhai commented on issue #3997: Issue 3655: Kerberos authentication for proxy

2019-04-07 Thread GitBox
jiazhai commented on issue #3997: Issue 3655: Kerberos authentication for proxy URL: https://github.com/apache/pulsar/pull/3997#issuecomment-480592250 @eolivelli If you have time, Would you please help review it? Thanks. This

[GitHub] [pulsar] jiazhai opened a new pull request #3997: Issue 3655: Kerberos authentication for proxy

2019-04-07 Thread GitBox
jiazhai opened a new pull request #3997: Issue 3655: Kerberos authentication for proxy URL: https://github.com/apache/pulsar/pull/3997 Fixes #3655 Master Issue: #3491 ### Motivation add support of Kerberos authentication for proxy ### Modifications add support of