[GitHub] merlimat closed pull request #2095: Remove unused test annotation

merlimat closed pull request #2095: Remove unused test annotation URL: https://github.com/apache/incubator-pulsar/pull/2095 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: Remove unused test annotation (#2095)

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 86f0a0d Remove unused test

[GitHub] merlimat commented on a change in pull request #2095: Remove unused test annotation

merlimat commented on a change in pull request #2095: Remove unused test annotation URL: https://github.com/apache/incubator-pulsar/pull/2095#discussion_r200802291 ## File path:

[GitHub] jerrypeng commented on issue #2098: Add context to source and sink

jerrypeng commented on issue #2098: Add context to source and sink URL: https://github.com/apache/incubator-pulsar/pull/2098#issuecomment-403183103 retest this please This is an automated message from the Apache Git Service.

[GitHub] jerrypeng opened a new pull request #2107: patching CmdSink and adding extensive unit tests

jerrypeng opened a new pull request #2107: patching CmdSink and adding extensive unit tests URL: https://github.com/apache/incubator-pulsar/pull/2107 This is an automated message from the Apache Git Service. To respond

[GitHub] aahmed-se commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar

aahmed-se commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar URL: https://github.com/apache/incubator-pulsar/pull/2080#issuecomment-403182287 retest this please This is an automated

[GitHub] aahmed-se commented on issue #2093: Disable empty methods in SequenceIdWithErrorTest

aahmed-se commented on issue #2093: Disable empty methods in SequenceIdWithErrorTest URL: https://github.com/apache/incubator-pulsar/pull/2093#issuecomment-403180091 Done This is an automated message from the Apache Git

[GitHub] merlimat opened a new pull request #2106: Provide API and CLI to access list of connectors supported by system

merlimat opened a new pull request #2106: Provide API and CLI to access list of connectors supported by system URL: https://github.com/apache/incubator-pulsar/pull/2106 ### Motivation Added API and CLI command for clients to discover list of supported connectors ###

[GitHub] sijie commented on issue #2093: Fix SequenceIdWithErrorTest

sijie commented on issue #2093: Fix SequenceIdWithErrorTest URL: https://github.com/apache/incubator-pulsar/pull/2093#issuecomment-403179765 @aahmed-se can you update the caption and description? This is an automated message

[GitHub] sijie commented on a change in pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer

sijie commented on a change in pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer URL: https://github.com/apache/incubator-pulsar/pull/2103#discussion_r200798912 ## File path:

[GitHub] aahmed-se commented on issue #1995: [WIP] Migrate to test containers Part 5

aahmed-se commented on issue #1995: [WIP] Migrate to test containers Part 5 URL: https://github.com/apache/incubator-pulsar/pull/1995#issuecomment-403175899 this has been replaced by individual test migration pr's This is an

[GitHub] aahmed-se commented on issue #2045: [WIP] Add Embedded Pulsar Class

aahmed-se commented on issue #2045: [WIP] Add Embedded Pulsar Class URL: https://github.com/apache/incubator-pulsar/pull/2045#issuecomment-403175794 this is replaced by https://github.com/apache/incubator-pulsar/pull/2080

[GitHub] aahmed-se commented on issue #2095: Remove unused test annotation

aahmed-se commented on issue #2095: Remove unused test annotation URL: https://github.com/apache/incubator-pulsar/pull/2095#issuecomment-403175839 retest this please This is an automated message from the Apache Git Service.

[GitHub] aahmed-se closed pull request #2045: [WIP] Add Embedded Pulsar Class

aahmed-se closed pull request #2045: [WIP] Add Embedded Pulsar Class URL: https://github.com/apache/incubator-pulsar/pull/2045 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

[GitHub] merlimat commented on a change in pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer

merlimat commented on a change in pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer URL: https://github.com/apache/incubator-pulsar/pull/2103#discussion_r200796299 ## File path:

[GitHub] jerrypeng commented on issue #2098: Add context to source and sink

jerrypeng commented on issue #2098: Add context to source and sink URL: https://github.com/apache/incubator-pulsar/pull/2098#issuecomment-403172092 retest this please This is an automated message from the Apache Git Service.

[GitHub] sijie commented on issue #2102: Load connectors from NAR archives

sijie commented on issue #2102: Load connectors from NAR archives URL: https://github.com/apache/incubator-pulsar/pull/2102#issuecomment-403168125 @jerrypeng @srkukarni @rdhabalia please spend some time on reviewing this.

[GitHub] sijie commented on a change in pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer

sijie commented on a change in pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer URL: https://github.com/apache/incubator-pulsar/pull/2103#discussion_r200787191 ## File path:

[GitHub] aahmed-se commented on issue #2045: [WIP] Add Embedded Pulsar Class

aahmed-se commented on issue #2045: [WIP] Add Embedded Pulsar Class URL: https://github.com/apache/incubator-pulsar/pull/2045#issuecomment-403162182 retest this please This is an automated message from the Apache Git Service.

[GitHub] aahmed-se commented on issue #2095: Remove unused test annotation

aahmed-se commented on issue #2095: Remove unused test annotation URL: https://github.com/apache/incubator-pulsar/pull/2095#issuecomment-403161959 retest this please This is an automated message from the Apache Git Service.

[GitHub] aahmed-se commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar

aahmed-se commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar URL: https://github.com/apache/incubator-pulsar/pull/2080#issuecomment-403158397 retest this please This is an automated

[incubator-pulsar] branch master updated: Fixed path for Postgres in dashboard docker image (#2088)

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 1ebe334 Fixed path for Postgres in

[GitHub] merlimat closed pull request #2088: Fixed path for Postgres in dashboard docker image

merlimat closed pull request #2088: Fixed path for Postgres in dashboard docker image URL: https://github.com/apache/incubator-pulsar/pull/2088 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

[incubator-pulsar] branch master updated: Remove dependencies exclusions for org.apache.httpcomponents:httpclient (#2104)

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 7bfc343 Remove dependencies

[GitHub] merlimat closed pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient

merlimat closed pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient URL: https://github.com/apache/incubator-pulsar/pull/2104 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] rdhabalia commented on a change in pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient

rdhabalia commented on a change in pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient URL: https://github.com/apache/incubator-pulsar/pull/2104#discussion_r200752973 ## File path: pom.xml ## @@ -188,12 +188,6 @@ flexible messaging

[GitHub] merlimat commented on a change in pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient

merlimat commented on a change in pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient URL: https://github.com/apache/incubator-pulsar/pull/2104#discussion_r200750911 ## File path: pom.xml ## @@ -188,12 +188,6 @@ flexible messaging

[GitHub] merlimat commented on a change in pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient

merlimat commented on a change in pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient URL: https://github.com/apache/incubator-pulsar/pull/2104#discussion_r200750766 ## File path: pom.xml ## @@ -188,12 +188,6 @@ flexible messaging

[GitHub] rdhabalia commented on a change in pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient

rdhabalia commented on a change in pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient URL: https://github.com/apache/incubator-pulsar/pull/2104#discussion_r200750533 ## File path: pom.xml ## @@ -188,12 +188,6 @@ flexible messaging

[GitHub] merlimat opened a new pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient

merlimat opened a new pull request #2104: Remove dependencies exclusions for org.apache.httpcomponents:httpclient URL: https://github.com/apache/incubator-pulsar/pull/2104 ### Motivation In `pulsar-client` we have dependency on `org.apache.httpcomponents:httpclient` and we had

[GitHub] sijie commented on issue #2051: Move Security related documentation to a top-level category

sijie commented on issue #2051: Move Security related documentation to a top-level category URL: https://github.com/apache/incubator-pulsar/pull/2051#issuecomment-403122140 @merlimat I addressed your comment. This is an

[incubator-pulsar] branch master updated: Kinesis sink publish full json message (#2079)

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 da8981c Kinesis sink publish full

[GitHub] rdhabalia closed pull request #2079: Kinesis sink publish full json message

rdhabalia closed pull request #2079: Kinesis sink publish full json message URL: https://github.com/apache/incubator-pulsar/pull/2079 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

[GitHub] merlimat commented on a change in pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer

merlimat commented on a change in pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer URL: https://github.com/apache/incubator-pulsar/pull/2103#discussion_r200742150 ## File path:

[GitHub] sijie commented on issue #2088: Fixed path for Postgres in dashboard docker image

sijie commented on issue #2088: Fixed path for Postgres in dashboard docker image URL: https://github.com/apache/incubator-pulsar/pull/2088#issuecomment-403118834 retest this please This is an automated message from the

[GitHub] sijie opened a new pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer

sijie opened a new pull request #2103: Issue 1433: Expose batch flushAsync() and flush() methods in Producer URL: https://github.com/apache/incubator-pulsar/pull/2103 *Motivation* Currently when you're using batches server will wait until timeout to send messages. To archive

[GitHub] aahmed-se commented on issue #2093: Fix SequenceIdWithErrorTest

aahmed-se commented on issue #2093: Fix SequenceIdWithErrorTest URL: https://github.com/apache/incubator-pulsar/pull/2093#issuecomment-403117550 @rdhabalia will need to take a deeper look at the handleProducer code , this change is mostly to disable empty test methods to remove them from

[GitHub] merlimat opened a new pull request #2102: Load connectors from NAR archives

merlimat opened a new pull request #2102: Load connectors from NAR archives URL: https://github.com/apache/incubator-pulsar/pull/2102 ### Motivation Allow the connectors to be submitted as NAR bundles. Each NAR bundle contains the classes for the connector code plus all of its

[GitHub] sijie commented on issue #2092: Issue 2090: Links to connector source code are broken

sijie commented on issue #2092: Issue 2090: Links to connector source code are broken URL: https://github.com/apache/incubator-pulsar/pull/2092#issuecomment-403070345 retest this please This is an automated message from the

[GitHub] sijie commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar

sijie commented on issue #2080: Extract out common code from standalone and build Embedded Pulsar URL: https://github.com/apache/incubator-pulsar/pull/2080#issuecomment-403070170 retest this please This is an automated

[GitHub] sijie closed pull request #2096: Issue 1252: Enable bookie autorecovery daemon by default

sijie closed pull request #2096: Issue 1252: Enable bookie autorecovery daemon by default URL: https://github.com/apache/incubator-pulsar/pull/2096 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:

[incubator-pulsar] branch master updated: Issue 1252: Enable bookie autorecovery daemon by default (#2096)

This is an automated email from the ASF dual-hosted git repository. sijie 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 3e9bac6 Issue 1252: Enable bookie

[GitHub] sijie closed issue #1252: Enable auto-recovery by default in bookie configuration

sijie closed issue #1252: Enable auto-recovery by default in bookie configuration URL: https://github.com/apache/incubator-pulsar/issues/1252 This is an automated message from the Apache Git Service. To respond to the

[GitHub] sijie closed pull request #2099: Issue 1455: MessageID has always batch index 0 when sending messages in a batch

sijie closed pull request #2099: Issue 1455: MessageID has always batch index 0 when sending messages in a batch URL: https://github.com/apache/incubator-pulsar/pull/2099 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[incubator-pulsar] branch master updated: Issue 1455: MessageID has always batch index 0 when sending messages in a batch (#2099)

This is an automated email from the ASF dual-hosted git repository. sijie 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 83e3157 Issue 1455: MessageID has

[GitHub] sijie closed issue #1455: MessageID has always batch index 0 when sending messages in a batch

sijie closed issue #1455: MessageID has always batch index 0 when sending messages in a batch URL: https://github.com/apache/incubator-pulsar/issues/1455 This is an automated message from the Apache Git Service. To respond

[incubator-pulsar] branch master updated: Fix wget argument (#2085)

This is an automated email from the ASF dual-hosted git repository. nkurihar 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 0420e59 Fix wget argument (#2085)

[GitHub] nkurihar closed pull request #2085: Fix wget argument

nkurihar closed pull request #2085: Fix wget argument URL: https://github.com/apache/incubator-pulsar/pull/2085 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 pull request

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

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 6db2ee8 Updated site at revision

[GitHub] aahmed-se commented on issue #2093: Fix SequenceIdWithErrorTest

aahmed-se commented on issue #2093: Fix SequenceIdWithErrorTest URL: https://github.com/apache/incubator-pulsar/pull/2093#issuecomment-402960417 retest this please This is an automated message from the Apache Git Service. To

[GitHub] aahmed-se commented on issue #2095: Remove unused test annotation

aahmed-se commented on issue #2095: Remove unused test annotation URL: https://github.com/apache/incubator-pulsar/pull/2095#issuecomment-402960382 retest this please This is an automated message from the Apache Git Service.