[GitHub] [pulsar] sijie commented on a change in pull request #5429: fix: java doc on MessageListener - acknowledgement part

2019-10-24 Thread GitBox
sijie commented on a change in pull request #5429: fix: java doc on MessageListener - acknowledgement part URL: https://github.com/apache/pulsar/pull/5429#discussion_r338435302 ## File path:

[GitHub] [pulsar] sijie commented on issue #5411: Add test for links to other markdown docs.

2019-10-24 Thread GitBox
sijie commented on issue #5411: Add test for links to other markdown docs. URL: https://github.com/apache/pulsar/pull/5411#issuecomment-545805278 run cpp tests This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] sijie commented on a change in pull request #5357: [functions][Issue:5350]Fix pulsar can't load the customized SerDe

2019-10-24 Thread GitBox
sijie commented on a change in pull request #5357: [functions][Issue:5350]Fix pulsar can't load the customized SerDe URL: https://github.com/apache/pulsar/pull/5357#discussion_r338459546 ## File path:

[GitHub] [pulsar] liketic commented on a change in pull request #5364: [broker] Fix incorrect log format

2019-10-24 Thread GitBox
liketic commented on a change in pull request #5364: [broker] Fix incorrect log format URL: https://github.com/apache/pulsar/pull/5364#discussion_r338480135 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/BrokerStatsBase.java ## @@ -167,7

[GitHub] [pulsar] jiazhai commented on issue #5453: [Cpp] Fixed negative ack tracker constructor sequence

2019-10-24 Thread GitBox
jiazhai commented on issue #5453: [Cpp] Fixed negative ack tracker constructor sequence URL: https://github.com/apache/pulsar/pull/5453#issuecomment-545839018 @merlimat Thanks for the fix. seems [ci](https://builds.apache.org/job/pulsar_precommit_cpp/10208/console) failed with

[GitHub] [pulsar] jiazhai commented on issue #5443: [pulsar-client] Fix message corruption on OOM for batch messages

2019-10-24 Thread GitBox
jiazhai commented on issue #5443: [pulsar-client] Fix message corruption on OOM for batch messages URL: https://github.com/apache/pulsar/pull/5443#issuecomment-545839689 run java8 tests This is an automated message

[pulsar-client-go] branch master updated: fix: avoid assert panic (#73)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 8a2ff05 fix: avoid assert panic (#73)

[pulsar-client-go] branch master updated: refactor: gofmt code (#75)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 84fb576 refactor: gofmt code (#75)

[pulsar-client-go] branch master updated: fix: avoid assert panic (#73)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 8a2ff05 fix: avoid assert panic (#73)

[pulsar-client-go] branch master updated: refactor: gofmt code (#75)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 84fb576 refactor: gofmt code (#75)

[GitHub] [pulsar] sijie merged pull request #5435: Add support for partitioned topic consumer seek by time.

2019-10-24 Thread GitBox
sijie merged pull request #5435: Add support for partitioned topic consumer seek by time. URL: https://github.com/apache/pulsar/pull/5435 This is an automated message from the Apache Git Service. To respond to the message,

[pulsar] branch master updated (997a128 -> a95bea6)

2019-10-24 Thread sijie
This is an automated email from the ASF dual-hosted git repository. sijie pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from 997a128 [Docs] Add functions-worker commands for pulsar admin CLI (#5405) add a95bea6 Add support for

[GitHub] [pulsar] sijie commented on issue #5278: Fixed reader cursor seek with disconnection

2019-10-24 Thread GitBox
sijie commented on issue #5278: Fixed reader cursor seek with disconnection URL: https://github.com/apache/pulsar/pull/5278#issuecomment-545826384 @merlimat If we don't disconnect the consumers, how do consumers reset their state? e.g. pending queue, acknowledgement, and etc.

[GitHub] [pulsar] sijie commented on issue #5320: Expose metrics of Caffeine caches

2019-10-24 Thread GitBox
sijie commented on issue #5320: Expose metrics of Caffeine caches URL: https://github.com/apache/pulsar/pull/5320#issuecomment-545830380 run integration tests This is an automated message from the Apache Git Service. To

[GitHub] [pulsar-client-go] wolfstudy closed issue #64: producer pending queue peek last with empty queue cause panic

2019-10-24 Thread GitBox
wolfstudy closed issue #64: producer pending queue peek last with empty queue cause panic URL: https://github.com/apache/pulsar-client-go/issues/64 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar-client-go] wolfstudy merged pull request #73: [ISSUE #64]fix: avoid assert panic

2019-10-24 Thread GitBox
wolfstudy merged pull request #73: [ISSUE #64]fix: avoid assert panic URL: https://github.com/apache/pulsar-client-go/pull/73 This is an automated message from the Apache Git Service. To respond to the message, please log on

[pulsar-client-go] branch master updated: refactor: gofmt code (#75)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 84fb576 refactor: gofmt code (#75)

[pulsar-client-go] branch master updated: fix: avoid assert panic (#73)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 8a2ff05 fix: avoid assert panic (#73)

[GitHub] [pulsar] aahmed-se commented on issue #5455: Update test docker scripts and library

2019-10-24 Thread GitBox
aahmed-se commented on issue #5455: Update test docker scripts and library URL: https://github.com/apache/pulsar/pull/5455#issuecomment-545811594 run integration tests This is an automated message from the Apache Git Service.

[GitHub] [pulsar] sijie commented on issue #5361: [proxy] Fix proxy to be able to re-send request body

2019-10-24 Thread GitBox
sijie commented on issue #5361: [proxy] Fix proxy to be able to re-send request body URL: https://github.com/apache/pulsar/pull/5361#issuecomment-545817355 run java8 tests run integration tests This is an automated

[GitHub] [pulsar] sijie commented on issue #5356: [TIEREDSTORAGE] Only seek when reading unexpected entry

2019-10-24 Thread GitBox
sijie commented on issue #5356: [TIEREDSTORAGE] Only seek when reading unexpected entry URL: https://github.com/apache/pulsar/pull/5356#issuecomment-545823044 I reverted `info` logging to `debug` logging. once it passes CI, it is ready to merge.

[pulsar-client-go] branch master updated: fix: avoid assert panic (#73)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 8a2ff05 fix: avoid assert panic (#73)

[GitHub] [pulsar] codelipenghui commented on issue #5451: SchemaUpdateStrategyTest is broken and not run

2019-10-24 Thread GitBox
codelipenghui commented on issue #5451: SchemaUpdateStrategyTest is broken and not run URL: https://github.com/apache/pulsar/issues/5451#issuecomment-545765396 @congbobo184 Can you help take a look this issue? This is an

[GitHub] [pulsar] codelipenghui commented on issue #5421: [Broker] Fix log message bug

2019-10-24 Thread GitBox
codelipenghui commented on issue #5421: [Broker] Fix log message bug URL: https://github.com/apache/pulsar/pull/5421#issuecomment-545769650 run integration tests This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] sijie commented on issue #5382: Update client-libraries-go.md

2019-10-24 Thread GitBox
sijie commented on issue #5382: Update client-libraries-go.md URL: https://github.com/apache/pulsar/pull/5382#issuecomment-545812825 run integration tests run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] sijie commented on issue #5279: Support for python native logging from python wrapper

2019-10-24 Thread GitBox
sijie commented on issue #5279: Support for python native logging from python wrapper URL: https://github.com/apache/pulsar/pull/5279#issuecomment-545826686 run integration tests This is an automated message from the Apache

[GitHub] [pulsar-client-go] wolfstudy merged pull request #75: [ISSUE #74]refactor: gofmt code

2019-10-24 Thread GitBox
wolfstudy merged pull request #75: [ISSUE #74]refactor: gofmt code URL: https://github.com/apache/pulsar-client-go/pull/75 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar-client-go] wolfstudy closed issue #74: fmt code

2019-10-24 Thread GitBox
wolfstudy closed issue #74: fmt code URL: https://github.com/apache/pulsar-client-go/issues/74 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [pulsar] codelipenghui commented on issue #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery*

2019-10-24 Thread GitBox
codelipenghui commented on issue #5293: [ISSUE #4675][docs] Add *Delayed Message Delivery* URL: https://github.com/apache/pulsar/pull/5293#issuecomment-545770764 ping @merlimat please help review the documentation for Delayed Message Delivery

[GitHub] [pulsar] sijie commented on issue #5456: [integration-tests]Enable messaging tests to integration tests.

2019-10-24 Thread GitBox
sijie commented on issue #5456: [integration-tests]Enable messaging tests to integration tests. URL: https://github.com/apache/pulsar/pull/5456#issuecomment-545798832 run cpp tests run integration tests This is an

[GitHub] [pulsar] sijie commented on issue #5453: [Cpp] Fixed negative ack tracker constructor sequence

2019-10-24 Thread GitBox
sijie commented on issue #5453: [Cpp] Fixed negative ack tracker constructor sequence URL: https://github.com/apache/pulsar/pull/5453#issuecomment-545798468 run cpp tests This is an automated message from the Apache Git

[GitHub] [pulsar] sijie commented on issue #5444: [pulsar-client] print unknown exception stacktrace

2019-10-24 Thread GitBox
sijie commented on issue #5444: [pulsar-client] print unknown exception stacktrace URL: https://github.com/apache/pulsar/pull/5444#issuecomment-545798204 run java8 tests This is an automated message from the Apache Git

[pulsar-client-go] branch master updated: fix: avoid assert panic (#73)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 8a2ff05 fix: avoid assert panic (#73)

[pulsar-client-go] branch master updated: refactor: gofmt code (#75)

2019-10-24 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 84fb576 refactor: gofmt code (#75)

[GitHub] [pulsar] codelipenghui commented on issue #5441: [pulsar-perf] record message failure and avoid exiting from process on publish failure

2019-10-24 Thread GitBox
codelipenghui commented on issue #5441: [pulsar-perf] record message failure and avoid exiting from process on publish failure URL: https://github.com/apache/pulsar/pull/5441#issuecomment-545769302 run java8 tests This is an

[GitHub] [pulsar] liketic commented on issue #5363: Merge module pulsar-client-tools-test into pulsar-client-tools

2019-10-24 Thread GitBox
liketic commented on issue #5363: Merge module pulsar-client-tools-test into pulsar-client-tools URL: https://github.com/apache/pulsar/pull/5363#issuecomment-545842684 @sijie The purpose is move the test and source code into one module. However, seems I missed the dependency problem which

[GitHub] [pulsar] liketic closed pull request #5363: Merge module pulsar-client-tools-test into pulsar-client-tools

2019-10-24 Thread GitBox
liketic closed pull request #5363: Merge module pulsar-client-tools-test into pulsar-client-tools URL: https://github.com/apache/pulsar/pull/5363 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar-client-go] wolfstudy closed pull request #65: [Issue 64] fix bug: type assert cause panic

2019-10-24 Thread GitBox
wolfstudy closed pull request #65: [Issue 64] fix bug: type assert cause panic URL: https://github.com/apache/pulsar-client-go/pull/65 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar] sijie commented on issue #5439: Multiple unit tests improvements

2019-10-24 Thread GitBox
sijie commented on issue #5439: Multiple unit tests improvements URL: https://github.com/apache/pulsar/pull/5439#issuecomment-545797300 retest this please This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar] sijie commented on issue #5441: [pulsar-perf] record message failure and avoid exiting from process on publish failure

2019-10-24 Thread GitBox
sijie commented on issue #5441: [pulsar-perf] record message failure and avoid exiting from process on publish failure URL: https://github.com/apache/pulsar/pull/5441#issuecomment-545797699 run java8 tests This is an

[GitHub] [pulsar] sijie commented on issue #5400: [functions] Allow functions to pass runtime specific options

2019-10-24 Thread GitBox
sijie commented on issue #5400: [functions] Allow functions to pass runtime specific options URL: https://github.com/apache/pulsar/pull/5400#issuecomment-545808890 @addisonj Can you rebase this pull request? This is an

[GitHub] [pulsar] sijie commented on issue #5363: Merge module pulsar-client-tools-test into pulsar-client-tools

2019-10-24 Thread GitBox
sijie commented on issue #5363: Merge module pulsar-client-tools-test into pulsar-client-tools URL: https://github.com/apache/pulsar/pull/5363#issuecomment-545815789 what is the motivation of this change? are there any benefits?

[GitHub] [pulsar] sijie commented on a change in pull request #5364: [broker] Fix incorrect log format

2019-10-24 Thread GitBox
sijie commented on a change in pull request #5364: [broker] Fix incorrect log format URL: https://github.com/apache/pulsar/pull/5364#discussion_r338453453 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/BrokerStatsBase.java ## @@ -167,7

[GitHub] [pulsar] jiazhai commented on issue #5456: [integration-tests]Enable messaging tests to integration tests.

2019-10-24 Thread GitBox
jiazhai commented on issue #5456: [integration-tests]Enable messaging tests to integration tests. URL: https://github.com/apache/pulsar/pull/5456#issuecomment-545837696 run java8 tests This is an automated message from the

[GitHub] [pulsar] codelipenghui commented on issue #5443: [pulsar-client] Fix message corruption on OOM for batch messages

2019-10-24 Thread GitBox
codelipenghui commented on issue #5443: [pulsar-client] Fix message corruption on OOM for batch messages URL: https://github.com/apache/pulsar/pull/5443#issuecomment-545768544 run java8 tests This is an automated message

[GitHub] [pulsar] codelipenghui commented on issue #5075: [pulsar-client] support set read-position based on timestamp

2019-10-24 Thread GitBox
codelipenghui commented on issue #5075: [pulsar-client] support set read-position based on timestamp URL: https://github.com/apache/pulsar/pull/5075#issuecomment-545768835 rerun java8 tests This is an automated message from

[GitHub] [pulsar] tuteng commented on issue #5456: [integration-tests]Enable messaging tests to integration tests.

2019-10-24 Thread GitBox
tuteng commented on issue #5456: [integration-tests]Enable messaging tests to integration tests. URL: https://github.com/apache/pulsar/pull/5456#issuecomment-545970319 run integration tests This is an automated

[GitHub] [pulsar] merlimat merged pull request #5385: Standalone should use crc32c by default as well

2019-10-24 Thread GitBox
merlimat merged pull request #5385: Standalone should use crc32c by default as well URL: https://github.com/apache/pulsar/pull/5385 This is an automated message from the Apache Git Service. To respond to the message, please

[pulsar] branch master updated (a71ac20 -> 978efaf)

2019-10-24 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 a71ac20 Fix website io connector 404 (#5347) add 978efaf [proxy] Fix proxy to be able to re-send request body

[GitHub] [pulsar] tuteng edited a comment on issue #5454: mysql JDBC Sink - consumer error

2019-10-24 Thread GitBox
tuteng edited a comment on issue #5454: mysql JDBC Sink - consumer error URL: https://github.com/apache/pulsar/issues/5454#issuecomment-545966420 http://pulsar.apache.org/docs/en/next/io-connectors/ It is already being fixed. https://github.com/apache/pulsar/pull/5347

[pulsar] branch master updated (c5c44a9 -> b4e5d15)

2019-10-24 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 c5c44a9 Add test for links to other markdown docs. (#5411) add b4e5d15 Standalone should use crc32c by

[pulsar] branch master updated (c5c44a9 -> b4e5d15)

2019-10-24 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 c5c44a9 Add test for links to other markdown docs. (#5411) add b4e5d15 Standalone should use crc32c by

[GitHub] [pulsar] jm7647 commented on issue #5454: mysql JDBC Sink - consumer error

2019-10-24 Thread GitBox
jm7647 commented on issue #5454: mysql JDBC Sink - consumer error URL: https://github.com/apache/pulsar/issues/5454#issuecomment-545940153 I was hoping you have some python example. There is nothing I could find on your web side regarding producer for that sink. All links related to

[GitHub] [pulsar] tuteng commented on issue #5454: mysql JDBC Sink - consumer error

2019-10-24 Thread GitBox
tuteng commented on issue #5454: mysql JDBC Sink - consumer error URL: https://github.com/apache/pulsar/issues/5454#issuecomment-545966420 http://pulsar.apache.org/docs/en/next/io-connectors/ It is already being fixed.

[GitHub] [pulsar] merlimat commented on issue #5342: Use simple GC settings when running integration tests

2019-10-24 Thread GitBox
merlimat commented on issue #5342: Use simple GC settings when running integration tests URL: https://github.com/apache/pulsar/pull/5342#issuecomment-545974819 run integration tests This is an automated message from

[pulsar] branch master updated (c5c44a9 -> b4e5d15)

2019-10-24 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 c5c44a9 Add test for links to other markdown docs. (#5411) add b4e5d15 Standalone should use crc32c by

[GitHub] [pulsar] merlimat merged pull request #5411: Add test for links to other markdown docs.

2019-10-24 Thread GitBox
merlimat merged pull request #5411: Add test for links to other markdown docs. URL: https://github.com/apache/pulsar/pull/5411 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] merlimat merged pull request #5406: [Doc] Add *authenticateMetricsEndpoint* option in administration-prox…

2019-10-24 Thread GitBox
merlimat merged pull request #5406: [Doc] Add *authenticateMetricsEndpoint* option in administration-prox… URL: https://github.com/apache/pulsar/pull/5406 This is an automated message from the Apache Git Service. To respond

[pulsar] branch master updated (ba6dfaf -> c5c44a9)

2019-10-24 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 ba6dfaf [Doc] Add *authenticateMetricsEndpoint* option in administration-proxy.md (#5406) add c5c44a9 Add

[pulsar] branch master updated (a95bea6 -> ba6dfaf)

2019-10-24 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 a95bea6 Add support for partitioned topic consumer seek by time. (#5435) add ba6dfaf [Doc] Add

[GitHub] [pulsar] merlimat commented on issue #5043: [zookeeper] bump zookeeper to version 3.5.5

2019-10-24 Thread GitBox
merlimat commented on issue #5043: [zookeeper] bump zookeeper to version 3.5.5 URL: https://github.com/apache/pulsar/pull/5043#issuecomment-545980449 > Fyi now 3.5.6 is available :) Yes, and it includes fix for upgrading ZK with missing snapshot:

[GitHub] [pulsar] merlimat closed issue #5360: Proxy doesn't send request body on redirects

2019-10-24 Thread GitBox
merlimat closed issue #5360: Proxy doesn't send request body on redirects URL: https://github.com/apache/pulsar/issues/5360 This is an automated message from the Apache Git Service. To respond to the message, please log on

[pulsar] branch master updated (b4e5d15 -> a71ac20)

2019-10-24 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 b4e5d15 Standalone should use crc32c by default as well (#5385) add a71ac20 Fix website io connector 404

[GitHub] [pulsar] merlimat merged pull request #5347: [Docs]Fix website io connector link 404

2019-10-24 Thread GitBox
merlimat merged pull request #5347: [Docs]Fix website io connector link 404 URL: https://github.com/apache/pulsar/pull/5347 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [pulsar] codelipenghui commented on issue #5457: [transaction-coordinator] Ownership change listeners

2019-10-24 Thread GitBox
codelipenghui commented on issue #5457: [transaction-coordinator] Ownership change listeners URL: https://github.com/apache/pulsar/pull/5457#issuecomment-545907008 @jiazhai @sijie According to our discussion offline, please help take a look, Is this PR suitable for kop?

[GitHub] [pulsar] codelipenghui opened a new pull request #5457: [transaction-coordinator] Ownership change listeners

2019-10-24 Thread GitBox
codelipenghui opened a new pull request #5457: [transaction-coordinator] Ownership change listeners URL: https://github.com/apache/pulsar/pull/5457 ### Motivation This is the first part to implement transaction coordinator startup, this PR added ownership change listeners, using

[GitHub] [pulsar] codelipenghui commented on issue #5378: Trim messages which less than mark delete position for message redelivery

2019-10-24 Thread GitBox
codelipenghui commented on issue #5378: Trim messages which less than mark delete position for message redelivery URL: https://github.com/apache/pulsar/pull/5378#issuecomment-545908928 run integration Tests This is an

[GitHub] [pulsar] codelipenghui commented on issue #5456: [integration-tests]Enable messaging tests to integration tests.

2019-10-24 Thread GitBox
codelipenghui commented on issue #5456: [integration-tests]Enable messaging tests to integration tests. URL: https://github.com/apache/pulsar/pull/5456#issuecomment-545893754 run java8 tests run integration tests This is

[GitHub] [pulsar] codelipenghui commented on issue #5378: Trim messages which less than mark delete position for message redelivery

2019-10-24 Thread GitBox
codelipenghui commented on issue #5378: Trim messages which less than mark delete position for message redelivery URL: https://github.com/apache/pulsar/pull/5378#issuecomment-545908413 run java8 tests This is an automated

[GitHub] [pulsar] jm7647 commented on issue #5454: mysql JDBC Sink - consumer error

2019-10-24 Thread GitBox
jm7647 commented on issue #5454: mysql JDBC Sink - consumer error URL: https://github.com/apache/pulsar/issues/5454#issuecomment-546101921 This is producer code I used based on your sample - still same error: producer = client.create_producer(

[GitHub] [pulsar] aahmed-se commented on issue #5447: Apache Beam support for Pulsar

2019-10-24 Thread GitBox
aahmed-se commented on issue #5447: Apache Beam support for Pulsar URL: https://github.com/apache/pulsar/issues/5447#issuecomment-546110903 this should be a request in the beam repo. This is an automated message from the

[GitHub] [pulsar] merlimat commented on issue #5453: [Cpp] Fixed negative ack tracker constructor sequence

2019-10-24 Thread GitBox
merlimat commented on issue #5453: [Cpp] Fixed negative ack tracker constructor sequence URL: https://github.com/apache/pulsar/pull/5453#issuecomment-546120867 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] jerrypeng commented on issue #5456: [integration-tests]Enable messaging tests to integration tests.

2019-10-24 Thread GitBox
jerrypeng commented on issue #5456: [integration-tests]Enable messaging tests to integration tests. URL: https://github.com/apache/pulsar/pull/5456#issuecomment-546142477 @codelipenghui some of the PersistentTopicMessagingTest are failing

[GitHub] [pulsar] merlimat opened a new pull request #5458: [cpp-build] Do not install libsnappy at each build

2019-10-24 Thread GitBox
merlimat opened a new pull request #5458: [cpp-build] Do not install libsnappy at each build URL: https://github.com/apache/pulsar/pull/5458 ### Motivation The docker image containing libsnappy was already pushed to docker hub so there's no need to install the package in the

[GitHub] [pulsar] rdhabalia commented on issue #3594: [pulsar-broker] vip-status checker for tls-only mode broker

2019-10-24 Thread GitBox
rdhabalia commented on issue #3594: [pulsar-broker] vip-status checker for tls-only mode broker URL: https://github.com/apache/pulsar/pull/3594#issuecomment-546063820 retest this please This is an automated message from the

[GitHub] [pulsar] aahmed-se commented on issue #5459: macOS 10.15 support

2019-10-24 Thread GitBox
aahmed-se commented on issue #5459: macOS 10.15 support URL: https://github.com/apache/pulsar/issues/5459#issuecomment-546078221 the packages published are for mac os 10.14, will try to publish osx 10.15 packages next release.

[GitHub] [pulsar] aahmed-se commented on issue #5455: Update test docker scripts and library

2019-10-24 Thread GitBox
aahmed-se commented on issue #5455: Update test docker scripts and library URL: https://github.com/apache/pulsar/pull/5455#issuecomment-546078472 run integration tests This is an automated message from the Apache Git Service.

[GitHub] [pulsar] aahmed-se opened a new pull request #5461: (WIP) Add Unified native clients build step

2019-10-24 Thread GitBox
aahmed-se opened a new pull request #5461: (WIP) Add Unified native clients build step URL: https://github.com/apache/pulsar/pull/5461 Remove python2 from package. make python point to python3 make pip point to pip3

[GitHub] [pulsar] merlimat commented on issue #5458: [cpp-build] Do not install libsnappy at each build

2019-10-24 Thread GitBox
merlimat commented on issue #5458: [cpp-build] Do not install libsnappy at each build URL: https://github.com/apache/pulsar/pull/5458#issuecomment-546113583 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] aahmed-se commented on issue #5460: Handle tearDown exception for test stability

2019-10-24 Thread GitBox
aahmed-se commented on issue #5460: Handle tearDown exception for test stability URL: https://github.com/apache/pulsar/pull/5460#issuecomment-546115727 run integration tests This is an automated message from the Apache Git

[GitHub] [pulsar] rdhabalia commented on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx

2019-10-24 Thread GitBox
rdhabalia commented on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx URL: https://github.com/apache/pulsar/pull/3705#issuecomment-546115353 rerun java8 tests This is an automated message from

[GitHub] [pulsar] rdhabalia removed a comment on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx

2019-10-24 Thread GitBox
rdhabalia removed a comment on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx URL: https://github.com/apache/pulsar/pull/3705#issuecomment-546115353 rerun java8 tests This is an automated

[GitHub] [pulsar] rdhabalia commented on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx

2019-10-24 Thread GitBox
rdhabalia commented on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx URL: https://github.com/apache/pulsar/pull/3705#issuecomment-546132739 rerun java8 tests This is an automated message from

[GitHub] [pulsar] vicaya commented on issue #3671: pulsar-admin topics list public/default fails

2019-10-24 Thread GitBox
vicaya commented on issue #3671: pulsar-admin topics list public/default fails URL: https://github.com/apache/pulsar/issues/3671#issuecomment-546100467 See this on a 2.4.0 cluster as well. This is an automated message from

[pulsar] branch master updated (402ecec -> 7c3f3bd)

2019-10-24 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 402ecec [pulsar-client] support set read-position based on timestamp (#5075) add 7c3f3bd Expose metrics of

[GitHub] [pulsar] merlimat merged pull request #5320: Expose metrics of Caffeine caches

2019-10-24 Thread GitBox
merlimat merged pull request #5320: Expose metrics of Caffeine caches URL: https://github.com/apache/pulsar/pull/5320 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [pulsar] merlimat closed pull request #5331: Avoid rethrowing exception during test teardown

2019-10-24 Thread GitBox
merlimat closed pull request #5331: Avoid rethrowing exception during test teardown URL: https://github.com/apache/pulsar/pull/5331 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [pulsar] merlimat commented on issue #5359: Set default ensemble size to 2 in service conf, to match broker.conf

2019-10-24 Thread GitBox
merlimat commented on issue #5359: Set default ensemble size to 2 in service conf, to match broker.conf URL: https://github.com/apache/pulsar/pull/5359#issuecomment-546122189 run cpp tests run java8 tests This is an

[GitHub] [pulsar] rdhabalia removed a comment on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx

2019-10-24 Thread GitBox
rdhabalia removed a comment on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx URL: https://github.com/apache/pulsar/pull/3705#issuecomment-546126133 rerun java8 tests This is an automated

[GitHub] [pulsar] rdhabalia commented on issue #3986: [pulsar-broker] Introduce publish rate-limiting on topic

2019-10-24 Thread GitBox
rdhabalia commented on issue #3986: [pulsar-broker] Introduce publish rate-limiting on topic URL: https://github.com/apache/pulsar/pull/3986#issuecomment-546126280 rerun java8 tests This is an automated message from the

[GitHub] [pulsar] rdhabalia removed a comment on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx

2019-10-24 Thread GitBox
rdhabalia removed a comment on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx URL: https://github.com/apache/pulsar/pull/3705#issuecomment-546096081 retest this please This is an automated

[GitHub] [pulsar] rdhabalia commented on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx

2019-10-24 Thread GitBox
rdhabalia commented on issue #3705: [pulsar-broker] add support for connection expiry and renew-cnx URL: https://github.com/apache/pulsar/pull/3705#issuecomment-546126133 rerun java8 tests This is an automated message from

[GitHub] [pulsar] rdhabalia opened a new pull request #5462: [pulsar-broker] fix broken custom auth-provider that uses authenticationData

2019-10-24 Thread GitBox
rdhabalia opened a new pull request #5462: [pulsar-broker] fix broken custom auth-provider that uses authenticationData URL: https://github.com/apache/pulsar/pull/5462 ### Motivation with #3677, ServerCnx is not storing client's `authenticationData` which is used by custom

[GitHub] [pulsar] rdhabalia commented on issue #3986: [pulsar-broker] Introduce publish rate-limiting on topic

2019-10-24 Thread GitBox
rdhabalia commented on issue #3986: [pulsar-broker] Introduce publish rate-limiting on topic URL: https://github.com/apache/pulsar/pull/3986#issuecomment-546132176 rerun java8 tests This is an automated message from the

[GitHub] [pulsar] merlimat commented on issue #5328: Increase the retries count for PulsarFunctionStateTest

2019-10-24 Thread GitBox
merlimat commented on issue #5328: Increase the retries count for PulsarFunctionStateTest URL: https://github.com/apache/pulsar/pull/5328#issuecomment-546137897 run java8 tests This is an automated message from the

[GitHub] [pulsar] merlimat commented on issue #5358: PIP-45: Switch ManagedLedger to use MetadataStore interface

2019-10-24 Thread GitBox
merlimat commented on issue #5358: PIP-45: Switch ManagedLedger to use MetadataStore interface URL: https://github.com/apache/pulsar/pull/5358#issuecomment-546114978 run java8 tests run integration tests This is an

[GitHub] [pulsar] aahmed-se commented on issue #5458: [cpp-build] Do not install libsnappy at each build

2019-10-24 Thread GitBox
aahmed-se commented on issue #5458: [cpp-build] Do not install libsnappy at each build URL: https://github.com/apache/pulsar/pull/5458#issuecomment-546143954 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] merlimat commented on issue #5320: Expose metrics of Caffeine caches

2019-10-24 Thread GitBox
merlimat commented on issue #5320: Expose metrics of Caffeine caches URL: https://github.com/apache/pulsar/pull/5320#issuecomment-546023994 run integration tests This is an automated message from the Apache Git

[GitHub] [pulsar] rdhabalia commented on issue #4400: PIP 37: [pulsar-client] support large message size

2019-10-24 Thread GitBox
rdhabalia commented on issue #4400: PIP 37: [pulsar-client] support large message size URL: https://github.com/apache/pulsar/pull/4400#issuecomment-546039490 I think rebasing this PR takes lot of time as txn and other changes have changed a lot after this PR. @merlimat can you please

[GitHub] [pulsar] merlimat merged pull request #5439: Multiple unit tests improvements

2019-10-24 Thread GitBox
merlimat merged pull request #5439: Multiple unit tests improvements URL: https://github.com/apache/pulsar/pull/5439 This is an automated message from the Apache Git Service. To respond to the message, please log on to

  1   2   >