[incubator-pulsar] branch master updated: Timeout task for batchMessageAndSendTask is not cancelled when close (#1405)

2018-03-19 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new ef87567 Timeout task for batchMessage

[GitHub] merlimat closed pull request #1405: Timeout task for batchMessageAndSendTask is not cancelled when close

2018-03-19 Thread GitBox
merlimat closed pull request #1405: Timeout task for batchMessageAndSendTask is not cancelled when close URL: https://github.com/apache/incubator-pulsar/pull/1405 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake o

[incubator-pulsar] branch master updated: Added trigger Function capability (#1402)

2018-03-19 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 4cb4f11 Added trigger Function capabi

[GitHub] merlimat closed pull request #1402: Added trigger Function capability

2018-03-19 Thread GitBox
merlimat closed pull request #1402: Added trigger Function capability URL: https://github.com/apache/incubator-pulsar/pull/1402 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: Build batching-awareness into round-robin message router (#1387)

2018-03-19 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new bde6598 Build batching-awareness into

[GitHub] merlimat closed pull request #1387: Build batching-awareness into round-robin message router

2018-03-19 Thread GitBox
merlimat closed pull request #1387: Build batching-awareness into round-robin message router URL: https://github.com/apache/incubator-pulsar/pull/1387 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] zhaijack commented on issue #1397: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-03-19 Thread GitBox
zhaijack commented on issue #1397: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1397#issuecomment-374440973 @XiaoZYang, In CI "Maven clean install", there was a log said: ``` [

[GitHub] rdhabalia closed pull request #1406: Fix JavaSerDeTest package name

2018-03-19 Thread GitBox
rdhabalia closed pull request #1406: Fix JavaSerDeTest package name URL: https://github.com/apache/incubator-pulsar/pull/1406 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 pu

[incubator-pulsar] branch master updated: Fix JavaSerDeTest package name (#1406)

2018-03-19 Thread rdhabalia
This is an automated email from the ASF dual-hosted git repository. rdhabalia 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 9e874cb Fix JavaSerDeTest package

[GitHub] sijie commented on issue #1387: Build batching-awareness into round-robin message router

2018-03-19 Thread GitBox
sijie commented on issue #1387: Build batching-awareness into round-robin message router URL: https://github.com/apache/incubator-pulsar/pull/1387#issuecomment-374362333 ping @merlimat This is an automated message from the A

[GitHub] sijie commented on issue #1397: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-03-19 Thread GitBox
sijie commented on issue #1397: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1397#issuecomment-374362194 @merlimat @rdhabalia if you guys have time, can you review this PR?

[GitHub] sijie commented on a change in pull request #1397: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-03-19 Thread GitBox
sijie commented on a change in pull request #1397: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1397#discussion_r175573617 ## File path: pulsar-client/src/main/java/org/apac

[GitHub] mgodave commented on a change in pull request #1137: Schema registry

2018-03-19 Thread GitBox
mgodave commented on a change in pull request #1137: Schema registry URL: https://github.com/apache/incubator-pulsar/pull/1137#discussion_r175573592 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/SchemasResource.java ## @@ -0,0 +1,241 @@ +/** +

[GitHub] mgodave commented on a change in pull request #1137: Schema registry

2018-03-19 Thread GitBox
mgodave commented on a change in pull request #1137: Schema registry URL: https://github.com/apache/incubator-pulsar/pull/1137#discussion_r175573551 ## File path: bin/pulsar ## @@ -214,6 +214,8 @@ if [ -z "$PULSAR_LOG_CONF" ]; then PULSAR_LOG_CONF=$DEFAULT_LOG_CONF fi

[GitHub] mgodave commented on issue #1381: Schema registry 4/N

2018-03-19 Thread GitBox
mgodave commented on issue #1381: Schema registry 4/N URL: https://github.com/apache/incubator-pulsar/pull/1381#issuecomment-374361391 this one, #1137 is just the tracking branch On Mon, Mar 19, 2018 at 2:24 PM, Sijie Guo wrote: > @mgodave which on

[GitHub] sijie commented on issue #1381: Schema registry 4/N

2018-03-19 Thread GitBox
sijie commented on issue #1381: Schema registry 4/N URL: https://github.com/apache/incubator-pulsar/pull/1381#issuecomment-374360569 @mgodave which one I should review? this one or #1137 ? This is an automated message from the

[GitHub] mgodave commented on issue #1206: Pulsar proxy should forward HTTP requests to brokers

2018-03-19 Thread GitBox
mgodave commented on issue #1206: Pulsar proxy should forward HTTP requests to brokers URL: https://github.com/apache/incubator-pulsar/issues/1206#issuecomment-374360442 okie dokie This is an automated message from the Apach

[GitHub] sijie commented on a change in pull request #1137: Schema registry

2018-03-19 Thread GitBox
sijie commented on a change in pull request #1137: Schema registry URL: https://github.com/apache/incubator-pulsar/pull/1137#discussion_r175572186 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v2/SchemasResource.java ## @@ -0,0 +1,241 @@ +/** + *

[GitHub] sijie commented on a change in pull request #1137: Schema registry

2018-03-19 Thread GitBox
sijie commented on a change in pull request #1137: Schema registry URL: https://github.com/apache/incubator-pulsar/pull/1137#discussion_r175571300 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/schema/BookkeeperSchemaStorage.java ## @@ -377,22 +

[GitHub] sijie commented on a change in pull request #1137: Schema registry

2018-03-19 Thread GitBox
sijie commented on a change in pull request #1137: Schema registry URL: https://github.com/apache/incubator-pulsar/pull/1137#discussion_r175571568 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/schema/SchemaRegistryServiceImpl.java ## @@ -27,6 +

[GitHub] sijie commented on a change in pull request #1137: Schema registry

2018-03-19 Thread GitBox
sijie commented on a change in pull request #1137: Schema registry URL: https://github.com/apache/incubator-pulsar/pull/1137#discussion_r175572279 ## File path: bin/pulsar ## @@ -214,6 +214,8 @@ if [ -z "$PULSAR_LOG_CONF" ]; then PULSAR_LOG_CONF=$DEFAULT_LOG_CONF fi

[GitHub] sijie commented on issue #1206: Pulsar proxy should forward HTTP requests to brokers

2018-03-19 Thread GitBox
sijie commented on issue #1206: Pulsar proxy should forward HTTP requests to brokers URL: https://github.com/apache/incubator-pulsar/issues/1206#issuecomment-374357377 @mgodave I don't think I can't assign to you. let's just use comments here to inform that you are taking this issue. ---

[GitHub] sijie commented on issue #1406: Fix JavaSerDeTest package name

2018-03-19 Thread GitBox
sijie commented on issue #1406: Fix JavaSerDeTest package name URL: https://github.com/apache/incubator-pulsar/pull/1406#issuecomment-374356808 /cc @srkukarni @jerrypeng This is an automated message from the Apache Git Servic

[GitHub] ivankelly commented on issue #1391: Utility to rebatch a message

2018-03-19 Thread GitBox
ivankelly commented on issue #1391: Utility to rebatch a message URL: https://github.com/apache/incubator-pulsar/pull/1391#issuecomment-374332596 retest this please // a c++ test failed :/ This is an automated message from the

[GitHub] rdhabalia opened a new pull request #1406: Fix JavaSerDeTest package name

2018-03-19 Thread GitBox
rdhabalia opened a new pull request #1406: Fix JavaSerDeTest package name URL: https://github.com/apache/incubator-pulsar/pull/1406 ### Motivation To address err-msg: ``` The declared package "org.apache.pulsar.functions.runtime.serde" does not match the expected package

[GitHub] mgodave commented on issue #1206: Pulsar proxy should forward HTTP requests to brokers

2018-03-19 Thread GitBox
mgodave commented on issue #1206: Pulsar proxy should forward HTTP requests to brokers URL: https://github.com/apache/incubator-pulsar/issues/1206#issuecomment-374320090 I'm taking a stab at this. Please assign to me. This i

[GitHub] sijie commented on issue #1387: Build batching-awareness into round-robin message router

2018-03-19 Thread GitBox
sijie commented on issue #1387: Build batching-awareness into round-robin message router URL: https://github.com/apache/incubator-pulsar/pull/1387#issuecomment-374305851 retest this please This is an automated message from th

[GitHub] sijie commented on a change in pull request #1367: Rest endpoint for triggering compaction

2018-03-19 Thread GitBox
sijie commented on a change in pull request #1367: Rest endpoint for triggering compaction URL: https://github.com/apache/incubator-pulsar/pull/1367#discussion_r175528303 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerServiceException.java ###

[GitHub] sijie commented on issue #1347: Enforce timeout on all TestNG tests, thread dump on timeout

2018-03-19 Thread GitBox
sijie commented on issue #1347: Enforce timeout on all TestNG tests, thread dump on timeout URL: https://github.com/apache/incubator-pulsar/pull/1347#issuecomment-374302007 retest this please This is an automated message from

[GitHub] jai1 closed pull request #730: CPP Client: Deprecated few APIs that can't be used with batch messages

2018-03-19 Thread GitBox
jai1 closed pull request #730: CPP Client: Deprecated few APIs that can't be used with batch messages URL: https://github.com/apache/incubator-pulsar/pull/730 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 pr

[GitHub] merlimat commented on a change in pull request #1405: Timeout task for batchMessageAndSendTask is not cancelled when close

2018-03-19 Thread GitBox
merlimat commented on a change in pull request #1405: Timeout task for batchMessageAndSendTask is not cancelled when close URL: https://github.com/apache/incubator-pulsar/pull/1405#discussion_r175497968 ## File path: pulsar-client/src/main/java/org/apache/pulsar/client/impl/Produce

[GitHub] merlimat closed pull request #1404: Make admin client namepsaces work with v2 rest endpoints.

2018-03-19 Thread GitBox
merlimat closed pull request #1404: Make admin client namepsaces work with v2 rest endpoints. URL: https://github.com/apache/incubator-pulsar/pull/1404 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 provenanc

[incubator-pulsar] branch master updated: Make admin client namepsaces work with v2 rest endpoints. (#1404)

2018-03-19 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/incubator-pulsar.git The following commit(s) were added to refs/heads/master by this push: new 440284e Make admin client namepsaces

[GitHub] jiazhai commented on issue #1397: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages

2018-03-19 Thread GitBox
jiazhai commented on issue #1397: Issue 1069: Provide a setting in consumer configuration to specify where to start consuming messages URL: https://github.com/apache/incubator-pulsar/pull/1397#issuecomment-374228301 retest this please ---

[GitHub] ivankelly commented on issue #1399: Integration smoke test for pulsar

2018-03-19 Thread GitBox
ivankelly commented on issue #1399: Integration smoke test for pulsar URL: https://github.com/apache/incubator-pulsar/pull/1399#issuecomment-374193069 retest this please This is an automated message from the Apache Git Service