[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] 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: produce

[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. ### Modificatio

[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 ## @@ -1

[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] 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, brok

[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] 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 Serv

[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 is

[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 foreign

[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 to

[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 automat