[incubator-pulsar] branch asf-site updated: Updated site at revision f44d367

2018-08-25 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/incubator-pulsar.git The following commit(s) were added to refs/heads/asf-site by this push: new 9e38436 Updated site at revision

[GitHub] srkukarni closed pull request #2426: Added cli commands to get function cluster related information

2018-08-25 Thread GitBox
srkukarni closed pull request #2426: Added cli commands to get function cluster related information URL: https://github.com/apache/incubator-pulsar/pull/2426 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

[incubator-pulsar] branch master updated: Added cli commands to get function cluster related information (#2426)

2018-08-25 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new f44d367 Added cli commands to get

[GitHub] merlimat commented on issue #2400: PIP-22: Dead Letter Topic

2018-08-25 Thread GitBox
merlimat commented on issue #2400: PIP-22: Dead Letter Topic URL: https://github.com/apache/incubator-pulsar/pull/2400#issuecomment-415977386 > @merlimat @rdhabalia can you confirm the summary is correct?  > frankly speaking, I am not sure about the producer concerns:

[GitHub] srkukarni opened a new pull request #2446: Add EventTime interface to Record

2018-08-25 Thread GitBox
srkukarni opened a new pull request #2446: Add EventTime interface to Record URL: https://github.com/apache/incubator-pulsar/pull/2446 ### Motivation Several sources contain information about event time. This pr propagates that information into the pulsar records. ###

[GitHub] codelipenghui commented on issue #2400: PIP-22: Dead Letter Topic

2018-08-25 Thread GitBox
codelipenghui commented on issue #2400: PIP-22: Dead Letter Topic URL: https://github.com/apache/incubator-pulsar/pull/2400#issuecomment-416005474 I have a idea for get the delivery count from broker. When consumer send a redeliveryUnackMessages request then broker process the request,

[GitHub] merlimat commented on a change in pull request #2446: Add EventTime interface to Record

2018-08-25 Thread GitBox
merlimat commented on a change in pull request #2446: Add EventTime interface to Record URL: https://github.com/apache/incubator-pulsar/pull/2446#discussion_r212806146 ## File path: pulsar-io/twitter/src/main/java/org/apache/pulsar/io/twitter/TwitterFireHose.java ## @@

[GitHub] srkukarni commented on issue #2446: Add EventTime interface to Record

2018-08-25 Thread GitBox
srkukarni commented on issue #2446: Add EventTime interface to Record URL: https://github.com/apache/incubator-pulsar/pull/2446#issuecomment-415998240 I can’t be static final since we can only assign it after open call. It can however just be static.

[GitHub] merlimat opened a new pull request #2447: Added producer/consumer properties in Go client

2018-08-25 Thread GitBox
merlimat opened a new pull request #2447: Added producer/consumer properties in Go client URL: https://github.com/apache/incubator-pulsar/pull/2447 ### Motivation Added producer/consumer properties in Go client wrapper

[GitHub] srkukarni commented on issue #2446: Add EventTime interface to Record

2018-08-25 Thread GitBox
srkukarni commented on issue #2446: Add EventTime interface to Record URL: https://github.com/apache/incubator-pulsar/pull/2446#issuecomment-416012557 run integration tests This is an automated message from the Apache Git

[GitHub] srkukarni closed pull request #2446: Add EventTime interface to Record

2018-08-25 Thread GitBox
srkukarni closed pull request #2446: Add EventTime interface to Record URL: https://github.com/apache/incubator-pulsar/pull/2446 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

[incubator-pulsar] branch master updated: Add EventTime interface to Record (#2446)

2018-08-25 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new a16b64a Add EventTime interface

[GitHub] jiazhai commented on a change in pull request #2440: Issue 2313: create a JDBC sink connector

2018-08-25 Thread GitBox
jiazhai commented on a change in pull request #2440: Issue 2313: create a JDBC sink connector URL: https://github.com/apache/incubator-pulsar/pull/2440#discussion_r212812761 ## File path: pulsar-io/jdbc/src/main/java/org/apache/pulsar/io/jdbc/JdbcAbstractSink.java ## @@

[GitHub] srkukarni commented on issue #2446: Add EventTime interface to Record

2018-08-25 Thread GitBox
srkukarni commented on issue #2446: Add EventTime interface to Record URL: https://github.com/apache/incubator-pulsar/pull/2446#issuecomment-416011123 @merlimat Sorry, I mistook your comments. I have made the date format static.

[GitHub] merlimat opened a new pull request #2448: Add multi-topic and regex consumer in Go client

2018-08-25 Thread GitBox
merlimat opened a new pull request #2448: Add multi-topic and regex consumer in Go client URL: https://github.com/apache/incubator-pulsar/pull/2448 ### Motivation Add multi-topic and regex consumer in Go client This

[GitHub] sijie commented on issue #2446: Add EventTime interface to Record

2018-08-25 Thread GitBox
sijie commented on issue #2446: Add EventTime interface to Record URL: https://github.com/apache/incubator-pulsar/pull/2446#issuecomment-416015970 @srkukarni can you label the issue when you merge it please? This is an