[GitHub] massakam merged pull request #3534: Revert "Replaces boost::regex with std::regex (#3512)"

massakam merged pull request #3534: Revert "Replaces boost::regex with std::regex (#3512)" URL: https://github.com/apache/pulsar/pull/3534 This is an automated message from the Apache Git Service. To respond to the message,

[pulsar] branch master updated: Revert "Replaces boost::regex with std::regex (#3512)" (#3534)

This is an automated email from the ASF dual-hosted git repository. massakam 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 ec923cc Revert "Replaces boost::regex with st

[GitHub] massakam commented on issue #3530: Fix command name in ns-isolation-policy help

massakam commented on issue #3530: Fix command name in ns-isolation-policy help URL: https://github.com/apache/pulsar/pull/3530#issuecomment-461323908 rerun integration tests This is an automated message from the Apache Git Se

[GitHub] ivankelly commented on issue #1016: Support bytes message key

ivankelly commented on issue #1016: Support bytes message key URL: https://github.com/apache/pulsar/issues/1016#issuecomment-461338855 This is done in #2612 This is an automated message from the Apache Git Service. To respond

[GitHub] ivankelly commented on issue #1016: Support bytes message key

ivankelly commented on issue #1016: Support bytes message key URL: https://github.com/apache/pulsar/issues/1016#issuecomment-461338812 This is done in #2612 This is an automated message from the Apache Git Service. To respond

[GitHub] ivankelly closed issue #1016: Support bytes message key

ivankelly closed issue #1016: Support bytes message key URL: https://github.com/apache/pulsar/issues/1016 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the U

[GitHub] ivankelly closed issue #1334: Experimental settings to achieve 5 ms latency

ivankelly closed issue #1334: Experimental settings to achieve 5 ms latency URL: https://github.com/apache/pulsar/issues/1334 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] ivankelly commented on issue #1334: Experimental settings to achieve 5 ms latency

ivankelly commented on issue #1334: Experimental settings to achieve 5 ms latency URL: https://github.com/apache/pulsar/issues/1334#issuecomment-461341973 Closing for inactivity This is an automated message from the Apache Gi

[GitHub] ivankelly commented on a change in pull request #3450: Adding configuration to disable auto-topic creation

ivankelly commented on a change in pull request #3450: Adding configuration to disable auto-topic creation URL: https://github.com/apache/pulsar/pull/3450#discussion_r254601605 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java ##

[GitHub] ivankelly commented on a change in pull request #3450: Adding configuration to disable auto-topic creation

ivankelly commented on a change in pull request #3450: Adding configuration to disable auto-topic creation URL: https://github.com/apache/pulsar/pull/3450#discussion_r254599753 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java ##

[GitHub] ivankelly commented on a change in pull request #3450: Adding configuration to disable auto-topic creation

ivankelly commented on a change in pull request #3450: Adding configuration to disable auto-topic creation URL: https://github.com/apache/pulsar/pull/3450#discussion_r254599678 ## File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java ##

[GitHub] ivankelly commented on issue #3446: Support Time Index to support seeking based on time in reader interface.

ivankelly commented on issue #3446: Support Time Index to support seeking based on time in reader interface. URL: https://github.com/apache/pulsar/issues/3446#issuecomment-461347919 One problem here would be that timestamps on messages aren't necessarily monotonically increasing, but ther

[GitHub] ivankelly commented on issue #3454: [Feature Request] Dynamically refresh client after updating the number of partitions when the routing mode is not key hash

ivankelly commented on issue #3454: [Feature Request] Dynamically refresh client after updating the number of partitions when the routing mode is not key hash URL: https://github.com/apache/pulsar/issues/3454#issuecomment-461348550 Marking this as a dupe of #3230 , so we can track this in

[GitHub] ivankelly closed issue #3454: [Feature Request] Dynamically refresh client after updating the number of partitions when the routing mode is not key hash

ivankelly closed issue #3454: [Feature Request] Dynamically refresh client after updating the number of partitions when the routing mode is not key hash URL: https://github.com/apache/pulsar/issues/3454 This is an automated

[GitHub] massakam merged pull request #3530: Fix command name in ns-isolation-policy help

massakam merged pull request #3530: Fix command name in ns-isolation-policy help URL: https://github.com/apache/pulsar/pull/3530 This is an automated message from the Apache Git Service. To respond to the message, please log

[pulsar] branch master updated: fix command name in help (#3530)

This is an automated email from the ASF dual-hosted git repository. massakam 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 dd3e7df fix command name in help (#3530) dd3e

[GitHub] ivankelly commented on issue #3490: Fail to reset-cursor on replicator subscription

ivankelly commented on issue #3490: Fail to reset-cursor on replicator subscription URL: https://github.com/apache/pulsar/issues/3490#issuecomment-461351221 @tuan6956 how many clusters do you have? Did you run this at the source cluster or the destination cluster? ---

[GitHub] ivankelly commented on issue #3493: Security Vulnerabilities - Black Duck Scan

ivankelly commented on issue #3493: Security Vulnerabilities - Black Duck Scan URL: https://github.com/apache/pulsar/issues/3493#issuecomment-461352124 jetty was upgraded to 9.4 a while back to get around thread pooling issues. Looks like jquery is the last thing that needs upgrade. @

[GitHub] sijie commented on issue #3521: Use testDupConsumersOnSharedModeNotThrowsExcOnUnsubscribe to avoid interference with other tests running in parallel

sijie commented on issue #3521: Use testDupConsumersOnSharedModeNotThrowsExcOnUnsubscribe to avoid interference with other tests running in parallel URL: https://github.com/apache/pulsar/pull/3521#issuecomment-461375110 run integration tests ---

[GitHub] merlimat commented on issue #3523: fix: RuntimeInfo doesn't get set when transferring ownership of function from another worker to this worker

merlimat commented on issue #3523: fix: RuntimeInfo doesn't get set when transferring ownership of function from another worker to this worker URL: https://github.com/apache/pulsar/pull/3523#issuecomment-461428237 run integrations tests -

[GitHub] merlimat commented on issue #3517: Improved SimpleProducerConsumerTest to avoid receive timeout

merlimat commented on issue #3517: Improved SimpleProducerConsumerTest to avoid receive timeout URL: https://github.com/apache/pulsar/pull/3517#issuecomment-461464057 run integration tests run cpp tests This is an automate

[GitHub] merlimat commented on issue #3462: Added suppport for Kafka partitioner and explicit setting of partition on record

merlimat commented on issue #3462: Added suppport for Kafka partitioner and explicit setting of partition on record URL: https://github.com/apache/pulsar/pull/3462#issuecomment-461464254 run cpp tests run java8 tests This

[GitHub] merlimat commented on issue #3462: Added suppport for Kafka partitioner and explicit setting of partition on record

merlimat commented on issue #3462: Added suppport for Kafka partitioner and explicit setting of partition on record URL: https://github.com/apache/pulsar/pull/3462#issuecomment-461464338 run integration tests This is an autom

[GitHub] merlimat commented on issue #3527: Removed usages of boost::filesystem

merlimat commented on issue #3527: Removed usages of boost::filesystem URL: https://github.com/apache/pulsar/pull/3527#issuecomment-461464787 run integration tests This is an automated message from the Apache Git Service. To r

[GitHub] merlimat commented on issue #3523: fix: RuntimeInfo doesn't get set when transferring ownership of function from another worker to this worker

merlimat commented on issue #3523: fix: RuntimeInfo doesn't get set when transferring ownership of function from another worker to this worker URL: https://github.com/apache/pulsar/pull/3523#issuecomment-461465046 run integration tests --

[GitHub] merlimat closed issue #3481: Intermittent test failure in `BasicEndToEndTest.testDupConsumersOnSharedModeNotThrowsExcOnUnsubscribe`

merlimat closed issue #3481: Intermittent test failure in `BasicEndToEndTest.testDupConsumersOnSharedModeNotThrowsExcOnUnsubscribe` URL: https://github.com/apache/pulsar/issues/3481 This is an automated message from the Apa

[GitHub] merlimat merged pull request #3521: Use testDupConsumersOnSharedModeNotThrowsExcOnUnsubscribe to avoid interference with other tests running in parallel

merlimat merged pull request #3521: Use testDupConsumersOnSharedModeNotThrowsExcOnUnsubscribe to avoid interference with other tests running in parallel URL: https://github.com/apache/pulsar/pull/3521 This is an automated m

[pulsar] branch master updated: Use testDupConsumersOnSharedModeNotThrowsExcOnUnsubscribe to avoid interference with other tests running in parallel (#3521)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new d59a6ed Use testDupConsumersOnSharedModeNotThro

[GitHub] merlimat merged pull request #3523: fix: RuntimeInfo doesn't get set when transferring ownership of function from another worker to this worker

merlimat merged pull request #3523: fix: RuntimeInfo doesn't get set when transferring ownership of function from another worker to this worker URL: https://github.com/apache/pulsar/pull/3523 This is an automated message fro

[pulsar] branch master updated: fix: RuntimeInfo doesn't get set when transferring ownership of function from another worker to this worker (#3523)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new c55bd95 fix: RuntimeInfo doesn't get set when t

[GitHub] merlimat opened a new issue #3539: Intermittent test failure in AuthenticatedProducerConsumerTest.testAnonymousSyncProducerAndConsumer

merlimat opened a new issue #3539: Intermittent test failure in AuthenticatedProducerConsumerTest.testAnonymousSyncProducerAndConsumer URL: https://github.com/apache/pulsar/issues/3539 ``` 2019-02-07\T\15:46:31.485 [INFO] Running org.apache.pulsar.client.api.AuthenticatedProducerCon

[GitHub] merlimat commented on issue #3527: Removed usages of boost::filesystem

merlimat commented on issue #3527: Removed usages of boost::filesystem URL: https://github.com/apache/pulsar/pull/3527#issuecomment-461515965 run cpp tests run java8 tests This is an automated message from the Apache Git Se

[GitHub] merlimat opened a new pull request #3540: Use synchronized when accessing consumers identity map

merlimat opened a new pull request #3540: Use synchronized when accessing consumers identity map URL: https://github.com/apache/pulsar/pull/3540 ### Motivation Fixes #3539. As reported in the issue, there's a concurrent modification exception that was recently introduced in 231db03

[GitHub] merlimat commented on issue #3520: Minor cleanup of Functions code to boost code uniformity

merlimat commented on issue #3520: Minor cleanup of Functions code to boost code uniformity URL: https://github.com/apache/pulsar/pull/3520#issuecomment-461528116 run integration tests This is an automated message from the Ap

[GitHub] merlimat commented on issue #3531: fix some LGTM alerts

merlimat commented on issue #3531: fix some LGTM alerts URL: https://github.com/apache/pulsar/pull/3531#issuecomment-461535065 retest this please This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] lovelle commented on issue #3540: Use synchronized when accessing consumers identity map

lovelle commented on issue #3540: Use synchronized when accessing consumers identity map URL: https://github.com/apache/pulsar/pull/3540#issuecomment-461538621 Ouch, another fault on me :disappointed:, it sound pretty obvious now!, next time feel free to assign any issue :+1: just

[GitHub] merlimat opened a new issue #3541: Intermittent test failure in PulsarFunctionE2ETest

merlimat opened a new issue #3541: Intermittent test failure in PulsarFunctionE2ETest URL: https://github.com/apache/pulsar/issues/3541 ``` 2019-02-07\T\17:22:21.662 [INFO] Running org.apache.pulsar.io.PulsarFunctionE2ETest 2019-02-07\T\17:24:42.450 [ERROR] Tests run: 11, Failures: 1

[GitHub] merlimat opened a new issue #3542: Intermittent test failure in C++ test AuthPluginTest.testTlsDetectHttpsWithHostNameValidation

merlimat opened a new issue #3542: Intermittent test failure in C++ test AuthPluginTest.testTlsDetectHttpsWithHostNameValidation URL: https://github.com/apache/pulsar/issues/3542 ``` [5/147] AuthPluginTest.testTlsDetectHttpsWithHostNameValidation (32 ms) Note: Google Test filte

[GitHub] merlimat commented on issue #3540: Use synchronized when accessing consumers identity map

merlimat commented on issue #3540: Use synchronized when accessing consumers identity map URL: https://github.com/apache/pulsar/pull/3540#issuecomment-461545359 No worries :) This ones was actually very easy to reproduce and then fix. I think to remember that the Identity map here wa

[GitHub] merlimat commented on issue #3513: Issue 3230: auto update topic partitions extend for consumer and producer

merlimat commented on issue #3513: Issue 3230: auto update topic partitions extend for consumer and producer URL: https://github.com/apache/pulsar/pull/3513#issuecomment-461547078 rerun integration tests This is an automated

[pulsar] branch master updated: Removed usages of boost::filesystem (#3527)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new a5b17f5 Removed usages of boost::filesystem (#3

[GitHub] merlimat merged pull request #3527: Removed usages of boost::filesystem

merlimat merged pull request #3527: Removed usages of boost::filesystem URL: https://github.com/apache/pulsar/pull/3527 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] merlimat merged pull request #3520: Minor cleanup of Functions code to boost code uniformity

merlimat merged pull request #3520: Minor cleanup of Functions code to boost code uniformity URL: https://github.com/apache/pulsar/pull/3520 This is an automated message from the Apache Git Service. To respond to the message

[pulsar] branch master updated: Cleaned up the code to not return Response objects (#3520)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new b833bce Cleaned up the code to not return Respo

[GitHub] merlimat commented on issue #3526: In Python schema, allow to declare required fields

merlimat commented on issue #3526: In Python schema, allow to declare required fields URL: https://github.com/apache/pulsar/pull/3526#issuecomment-461551782 run cpp tests This is an automated message from the Apache Git Servi

[GitHub] jerrypeng commented on issue #3532: fix: Exception thrown in get list connectors

jerrypeng commented on issue #3532: fix: Exception thrown in get list connectors URL: https://github.com/apache/pulsar/pull/3532#issuecomment-461554691 rerun integrations tests rerun cpp tests rerun java8 tests This is

[GitHub] jerrypeng commented on issue #3536: Add minimum amount of resources to run setting for functions

jerrypeng commented on issue #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#issuecomment-461554796 rerun integration tests This is an automated message from

[GitHub] merlimat commented on issue #3529: For integration tests, copy function logs out of container on shutdown for debugging purposes

merlimat commented on issue #3529: For integration tests, copy function logs out of container on shutdown for debugging purposes URL: https://github.com/apache/pulsar/pull/3529#issuecomment-461555805 Scratching that, the integration tests were failing because of the python client segfaulti

[GitHub] jerrypeng commented on issue #3529: For integration tests, copy function logs out of container on shutdown for debugging purposes

jerrypeng commented on issue #3529: For integration tests, copy function logs out of container on shutdown for debugging purposes URL: https://github.com/apache/pulsar/pull/3529#issuecomment-461555697 rerun integration tests rerun java8 tests

[GitHub] merlimat commented on issue #3532: fix: Exception thrown in get list connectors

merlimat commented on issue #3532: fix: Exception thrown in get list connectors URL: https://github.com/apache/pulsar/pull/3532#issuecomment-461535400 retest this please This is an automated message from the Apache Git Servic

[GitHub] merlimat closed issue #3518: Python intermittent test failure on test_publish_compact_and_consume

merlimat closed issue #3518: Python intermittent test failure on test_publish_compact_and_consume URL: https://github.com/apache/pulsar/issues/3518 This is an automated message from the Apache Git Service. To respond to the

[GitHub] merlimat commented on issue #3518: Python intermittent test failure on test_publish_compact_and_consume

merlimat commented on issue #3518: Python intermittent test failure on test_publish_compact_and_consume URL: https://github.com/apache/pulsar/issues/3518#issuecomment-461566521 Dup of #2321 This is an automated message from t

[GitHub] merlimat commented on issue #2321: Flaky python unit test test_publish_compact_and_consume

merlimat commented on issue #2321: Flaky python unit test test_publish_compact_and_consume URL: https://github.com/apache/pulsar/issues/2321#issuecomment-461567168 Still happening in https://builds.apache.org/job/pulsar_precommit_cpp/5783/console --

[GitHub] rdhabalia commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional

rdhabalia commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional URL: https://github.com/apache/pulsar/pull/3501#discussion_r254848234 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java

[GitHub] merlimat commented on issue #3540: Use synchronized when accessing consumers identity map

merlimat commented on issue #3540: Use synchronized when accessing consumers identity map URL: https://github.com/apache/pulsar/pull/3540#issuecomment-461580106 run java8 tests This is an automated message from the Apache Git

[GitHub] merlimat commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional

merlimat commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional URL: https://github.com/apache/pulsar/pull/3501#discussion_r254852800 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java

[GitHub] lovelle commented on issue #3540: Use synchronized when accessing consumers identity map

lovelle commented on issue #3540: Use synchronized when accessing consumers identity map URL: https://github.com/apache/pulsar/pull/3540#issuecomment-461581505 :+1: This is an automated message from the Apache Git Service. T

[GitHub] rdhabalia commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional

rdhabalia commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional URL: https://github.com/apache/pulsar/pull/3501#discussion_r254855764 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java

[GitHub] rdhabalia commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional

rdhabalia commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional URL: https://github.com/apache/pulsar/pull/3501#discussion_r254855764 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java

[GitHub] merlimat closed issue #3230: expect not restart application after more partitions created

merlimat closed issue #3230: expect not restart application after more partitions created URL: https://github.com/apache/pulsar/issues/3230 This is an automated message from the Apache Git Service. To respond to the message

[GitHub] merlimat merged pull request #3513: Issue 3230: auto update topic partitions extend for consumer and producer

merlimat merged pull request #3513: Issue 3230: auto update topic partitions extend for consumer and producer URL: https://github.com/apache/pulsar/pull/3513 This is an automated message from the Apache Git Service. To resp

[pulsar] branch master updated: Issue 3230: auto update topic partitions extend for consumer and producer (#3513)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 3a5df1a Issue 3230: auto update topic partitio

[GitHub] jerrypeng commented on issue #3536: Add minimum amount of resources to run setting for functions

jerrypeng commented on issue #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#issuecomment-461613874 rerun cpp tests This is an automated message from the Apa

[pulsar] branch master updated: Use synchronized when accessing consumers identity map (#3540)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 1e531e0 Use synchronized when accessing consume

[GitHub] merlimat closed issue #3539: Intermittent test failure in AuthenticatedProducerConsumerTest.testAnonymousSyncProducerAndConsumer

merlimat closed issue #3539: Intermittent test failure in AuthenticatedProducerConsumerTest.testAnonymousSyncProducerAndConsumer URL: https://github.com/apache/pulsar/issues/3539 This is an automated message from the Apache

[GitHub] merlimat closed issue #2842: pulsar go client static build error

merlimat closed issue #2842: pulsar go client static build error URL: https://github.com/apache/pulsar/issues/2842 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[pulsar] branch master updated: Build static lib with bundled dependencies (#3488)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 3b1dd8c Build static lib with bundled dependenc

[GitHub] merlimat merged pull request #3509: Upgrade AsyncHTTP client to 2.7.0

merlimat merged pull request #3509: Upgrade AsyncHTTP client to 2.7.0 URL: https://github.com/apache/pulsar/pull/3509 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub a

[GitHub] merlimat merged pull request #3540: Use synchronized when accessing consumers identity map

merlimat merged pull request #3540: Use synchronized when accessing consumers identity map URL: https://github.com/apache/pulsar/pull/3540 This is an automated message from the Apache Git Service. To respond to the message,

[pulsar] branch master updated: Upgrade AsyncHTTP client to 2.7.0 (#3509)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 7ef26dd Upgrade AsyncHTTP client to 2.7.0 (#350

[GitHub] merlimat merged pull request #3488: Build static lib with bundled dependencies

merlimat merged pull request #3488: Build static lib with bundled dependencies URL: https://github.com/apache/pulsar/pull/3488 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] merlimat commented on issue #3472: [pulsar-client] Modify CmdProduce to avoid closing resource manually

merlimat commented on issue #3472: [pulsar-client] Modify CmdProduce to avoid closing resource manually URL: https://github.com/apache/pulsar/pull/3472#issuecomment-461619210 run integration tests This is an automated message

[pulsar] branch master updated: For integration tests, copy function logs out of container on shutdown for debugging purposes (#3529)

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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 2bc5c5b For integration tests, copy function lo

[GitHub] merlimat merged pull request #3529: For integration tests, copy function logs out of container on shutdown for debugging purposes

merlimat merged pull request #3529: For integration tests, copy function logs out of container on shutdown for debugging purposes URL: https://github.com/apache/pulsar/pull/3529 This is an automated message from the Apache G

[GitHub] rdhabalia opened a new pull request #3543: [pulsar-common] add option that field can be empty

rdhabalia opened a new pull request #3543: [pulsar-common] add option that field can be empty URL: https://github.com/apache/pulsar/pull/3543 ### Motivation sometimes eg: #3501, user want to initialize few service config null/empty by defining empty value into config property file. ---

[GitHub] rdhabalia commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional

rdhabalia commented on a change in pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional URL: https://github.com/apache/pulsar/pull/3501#discussion_r254897624 ## File path: pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java

[GitHub] merlimat commented on issue #3528: Replace JsonCPP lib with boost::property_tree

merlimat commented on issue #3528: Replace JsonCPP lib with boost::property_tree URL: https://github.com/apache/pulsar/pull/3528#issuecomment-461624123 run cpp tests This is an automated message from the Apache Git Service. To

[GitHub] merlimat opened a new pull request #3544: In functions log topic appender, don't set producer name

merlimat opened a new pull request #3544: In functions log topic appender, don't set producer name URL: https://github.com/apache/pulsar/pull/3544 ### Motivation When enabling log topic in functions, we shouldn't set the producer name, otherwise multiple function instances will fail

[GitHub] merlimat commented on a change in pull request #3543: [pulsar-common] add option that field can be empty

merlimat commented on a change in pull request #3543: [pulsar-common] add option that field can be empty URL: https://github.com/apache/pulsar/pull/3543#discussion_r254903092 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/FieldContext.java ## @@ -

[GitHub] merlimat opened a new pull request #3545: Fixed BrokerClientIntegrationTest.testUnsupportedBatchMessageConsumer

merlimat opened a new pull request #3545: Fixed BrokerClientIntegrationTest.testUnsupportedBatchMessageConsumer URL: https://github.com/apache/pulsar/pull/3545 ### Motivation Fixes #3538. The second consumer was being created with `Exclusive` type and was actually failing to connect

[GitHub] merlimat commented on issue #3511: Added docs for using schema in Python client lib

merlimat commented on issue #3511: Added docs for using schema in Python client lib URL: https://github.com/apache/pulsar/pull/3511#issuecomment-461632847 run java8 tests This is an automated message from the Apache Git Servi

[GitHub] merlimat opened a new pull request #3546: Only require linking with Boost program_options for CLI tools

merlimat opened a new pull request #3546: Only require linking with Boost program_options for CLI tools URL: https://github.com/apache/pulsar/pull/3546 ### Motivation We don't need to link the pulsar client lib with Boost program-options. That's only need for C++ CLI tools. ---

[GitHub] sijie commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions

sijie commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#discussion_r254918308 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/WorkerConfig.

[GitHub] merlimat commented on issue #3511: Added docs for using schema in Python client lib

merlimat commented on issue #3511: Added docs for using schema in Python client lib URL: https://github.com/apache/pulsar/pull/3511#issuecomment-461646730 run java8 tests This is an automated message from the Apache Git Servi

[GitHub] jerrypeng commented on issue #3532: fix: Exception thrown in get list connectors

jerrypeng commented on issue #3532: fix: Exception thrown in get list connectors URL: https://github.com/apache/pulsar/pull/3532#issuecomment-461646974 rerun integration tests This is an automated message from the Apache

[GitHub] jerrypeng opened a new pull request #3547: fix a couple race conditions in function worker

jerrypeng opened a new pull request #3547: fix a couple race conditions in function worker URL: https://github.com/apache/pulsar/pull/3547 ### Motivation The two potential race conditions: 1. RuntimeSpawner - make sure to interrupt the processLivenessCheckTimer so that we tha

[GitHub] jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions

jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#discussion_r254920228 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/WorkerCon

[GitHub] jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions

jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#discussion_r254920679 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/WorkerCon

[GitHub] jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions

jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#discussion_r254920228 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/WorkerCon

[GitHub] sijie commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions

sijie commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#discussion_r25491 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/WorkerConfig.

[GitHub] merlimat commented on issue #3538: Intermittent test failure in BrokerClientIntegrationTest.testUnsupportedBatchMessageConsumer

merlimat commented on issue #3538: Intermittent test failure in BrokerClientIntegrationTest.testUnsupportedBatchMessageConsumer URL: https://github.com/apache/pulsar/issues/3538#issuecomment-461652185 Removed "bug" label since the issue here was in the test code. --

[GitHub] jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions

jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#discussion_r254928455 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/WorkerCon

[GitHub] jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions

jerrypeng commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#discussion_r254930047 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/WorkerCon

[GitHub] jerrypeng commented on issue #3547: fix a couple race conditions in function worker

jerrypeng commented on issue #3547: fix a couple race conditions in function worker URL: https://github.com/apache/pulsar/pull/3547#issuecomment-461664893 rerun integration tests This is an automated message from the Apache G

[pulsar] branch master updated: fix: get list connectors return status (#3532)

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 d9e1370 fix: get list connectors return stat

[GitHub] jerrypeng merged pull request #3532: fix: Exception thrown in get list connectors

jerrypeng merged pull request #3532: fix: Exception thrown in get list connectors URL: https://github.com/apache/pulsar/pull/3532 This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] nkurihar commented on issue #3535: Broker stopped with a lot of LedgerFencedException

nkurihar commented on issue #3535: Broker stopped with a lot of LedgerFencedException URL: https://github.com/apache/pulsar/issues/3535#issuecomment-461667425 We found that producing from a particular topic causes Broker's frequent GCs, and finally ZK session will be disconnected. C

[GitHub] nkurihar edited a comment on issue #3535: Broker stopped with a lot of LedgerFencedException

nkurihar edited a comment on issue #3535: Broker stopped with a lot of LedgerFencedException URL: https://github.com/apache/pulsar/issues/3535#issuecomment-461667425 We found that producing from a particular topic causes Broker's frequent GCs, and finally ZK session will be disconnected.

[GitHub] merlimat opened a new issue #3548: Intermittent test failure in CreateSubscriptionTest

merlimat opened a new issue #3548: Intermittent test failure in CreateSubscriptionTest URL: https://github.com/apache/pulsar/issues/3548 ``` 2019-02-08\T\00:00:26.815 [INFO] Running org.apache.pulsar.broker.admin.CreateSubscriptionTest 2019-02-08\T\00:02:06.165 [ERROR] Tests run:

[GitHub] sijie commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions

sijie commented on a change in pull request #3536: Add minimum amount of resources to run setting for functions URL: https://github.com/apache/pulsar/pull/3536#discussion_r254941789 ## File path: pulsar-functions/worker/src/main/java/org/apache/pulsar/functions/worker/WorkerConfig.

  1   2   >