[GitHub] [pulsar] candlerb closed pull request #5609: [Doc] Remove duplicate pages

2019-11-26 Thread GitBox
candlerb closed pull request #5609: [Doc] Remove duplicate pages URL: https://github.com/apache/pulsar/pull/5609 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar] candlerb commented on issue #5609: [Doc] Remove duplicate pages

2019-11-26 Thread GitBox
candlerb commented on issue #5609: [Doc] Remove duplicate pages URL: https://github.com/apache/pulsar/pull/5609#issuecomment-558973958 I withdraw this PR. This is an automated message from the Apache Git Service. To respond to

[GitHub] [pulsar] aahmed-se commented on issue #5642: Add Github workflow for gated checkin

2019-11-26 Thread GitBox
aahmed-se commented on issue #5642: Add Github workflow for gated checkin URL: https://github.com/apache/pulsar/pull/5642#issuecomment-558972726 run java8 tests This is an automated message from the Apache Git Service. To resp

[pulsar] branch master updated (eee4d89 -> e372cc9)

2019-11-26 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 eee4d89 init broker publish limitor at broker start time (#5754) add e372cc9 Fix documentation: how to start se

[GitHub] [pulsar] sijie merged pull request #5549: Fix documentation: how to start server for cpp tests

2019-11-26 Thread GitBox
sijie merged pull request #5549: Fix documentation: how to start server for cpp tests URL: https://github.com/apache/pulsar/pull/5549 This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [pulsar] Anonymitaet edited a comment on issue #5753: [Doc] link for TLS Authentication on Proxies is dead

2019-11-26 Thread GitBox
Anonymitaet edited a comment on issue #5753: [Doc] link for TLS Authentication on Proxies is dead URL: https://github.com/apache/pulsar/issues/5753#issuecomment-558924764 @Jennifer88huang not yet. This is an automated messag

[GitHub] [pulsar] sijie commented on a change in pull request #5376: [Issue 3009][docs] Add presto sql REST APIs doc

2019-11-26 Thread GitBox
sijie commented on a change in pull request #5376: [Issue 3009][docs] Add presto sql REST APIs doc URL: https://github.com/apache/pulsar/pull/5376#discussion_r351123077 ## File path: site2/docs/sql-rest-api.md ## @@ -0,0 +1,196 @@ +--- +id: sql-rest-api +title: Pulsar SQL

[GitHub] [pulsar] wolfstudy commented on issue #5734: [Reopen][Issue 5597]retry when getPartitionedTopicMetadata failed

2019-11-26 Thread GitBox
wolfstudy commented on issue #5734: [Reopen][Issue 5597]retry when getPartitionedTopicMetadata failed URL: https://github.com/apache/pulsar/pull/5734#issuecomment-558953397 run java8 tests This is an automated message from th

[pulsar-manager.wiki] branch master updated: Updated Authentication and Authorization in Pulsar Manager (markdown)

2019-11-26 Thread guangning
This is an automated email from the ASF dual-hosted git repository. guangning pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-manager.wiki.git The following commit(s) were added to refs/heads/master by this push: new a96a56f Updated Authentication

[pulsar-manager.wiki] branch master updated: Updated Authentication and Authorization in Pulsar Manager (markdown)

2019-11-26 Thread guangning
This is an automated email from the ASF dual-hosted git repository. guangning pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-manager.wiki.git The following commit(s) were added to refs/heads/master by this push: new 41b0375 Updated Authentication

[pulsar-manager.wiki] branch master updated: Updated Authentication and Authorization in Pulsar Manager (markdown)

2019-11-26 Thread guangning
This is an automated email from the ASF dual-hosted git repository. guangning pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-manager.wiki.git The following commit(s) were added to refs/heads/master by this push: new b5aa39c Updated Authentication

[GitHub] [pulsar] aahmed-se commented on issue #5642: Add Github workflow for gated checkin

2019-11-26 Thread GitBox
aahmed-se commented on issue #5642: Add Github workflow for gated checkin URL: https://github.com/apache/pulsar/pull/5642#issuecomment-558941579 run integration tests This is an automated message from the Apache Git Service. T

[GitHub] [pulsar] Anonymitaet commented on issue #5753: [Doc] link for TLS Authentication on Proxies is dead

2019-11-26 Thread GitBox
Anonymitaet commented on issue #5753: [Doc] link for TLS Authentication on Proxies is dead URL: https://github.com/apache/pulsar/issues/5753#issuecomment-558924764 @Jennifer88huang not yet, I'll fix when I'm available. This

[GitHub] [pulsar] wolfstudy commented on issue #5669: Pulsar Broker restart fail owing to create functions/assignments topic fail

2019-11-26 Thread GitBox
wolfstudy commented on issue #5669: Pulsar Broker restart fail owing to create functions/assignments topic fail URL: https://github.com/apache/pulsar/issues/5669#issuecomment-558922166 @laxpio If you enabled `Deduplication` feature. If possible, please provide more detailed information, th

[GitHub] [pulsar] wolfstudy commented on issue #5652: Server wedged after deleting and recreating functions

2019-11-26 Thread GitBox
wolfstudy commented on issue #5652: Server wedged after deleting and recreating functions URL: https://github.com/apache/pulsar/issues/5652#issuecomment-558920748 > Would you please help to take a look if this is reproduce-able? In fact, after I tried many times in my locally, i can'

[GitHub] [pulsar] Jennifer88huang commented on issue #5609: [Doc] Remove duplicate pages

2019-11-26 Thread GitBox
Jennifer88huang commented on issue #5609: [Doc] Remove duplicate pages URL: https://github.com/apache/pulsar/pull/5609#issuecomment-558919202 @candlerb could you please update the suggestion in the `deploy-bare-metal.md` file, and resolve conflicting files, and then we can go on with the n

[GitHub] [pulsar] jiazhai commented on issue #5747: Release ByteBuf failed when send a over size message in message batch container

2019-11-26 Thread GitBox
jiazhai commented on issue #5747: Release ByteBuf failed when send a over size message in message batch container URL: https://github.com/apache/pulsar/issues/5747#issuecomment-558919034 Thanks @codelipenghui for the issue and PR. --

[GitHub] [pulsar] jiazhai commented on issue #5749: Bookie does not start because of UnknownHostException

2019-11-26 Thread GitBox
jiazhai commented on issue #5749: Bookie does not start because of UnknownHostException URL: https://github.com/apache/pulsar/issues/5749#issuecomment-558918815 @vishalananth07 regarding this issue: ``` [main] ERROR org.apache.bookkeeper.server.Main - Failed to build bookie server

[GitHub] [pulsar] aahmed-se commented on issue #5642: Add Github workflow for gated checkin

2019-11-26 Thread GitBox
aahmed-se commented on issue #5642: Add Github workflow for gated checkin URL: https://github.com/apache/pulsar/pull/5642#issuecomment-558918009 run java8 tests This is an automated message from the Apache Git Service. To resp

[GitHub] [pulsar] wolfstudy commented on issue #5695: Go: Segfault on messageId serialize after getting it from producer

2019-11-26 Thread GitBox
wolfstudy commented on issue #5695: Go: Segfault on messageId serialize after getting it from producer URL: https://github.com/apache/pulsar/issues/5695#issuecomment-558917988 @lspgn Could you please provide your msg struct content? In here, there is an example about sendWithMsgID, please

[GitHub] [pulsar] jiazhai commented on issue #5751: [DISCUSS] Avoid broker OOM

2019-11-26 Thread GitBox
jiazhai commented on issue #5751: [DISCUSS] Avoid broker OOM URL: https://github.com/apache/pulsar/issues/5751#issuecomment-558917892 Thanks @codelipenghui for the discussion. right, This is more directly for the OOM issue. add @sijie @rdhabalia @merlimat to the discuss. -

[GitHub] [pulsar] jiazhai commented on issue #5752: Expose key shared consumer selector range in API

2019-11-26 Thread GitBox
jiazhai commented on issue #5752: Expose key shared consumer selector range in API URL: https://github.com/apache/pulsar/issues/5752#issuecomment-558917183 Thanks @cbornet for watching on this. will put this in our backlog. It would be great if you or any other would like to contribute to

[GitHub] [pulsar] Jennifer88huang edited a comment on issue #5753: [Doc] link for TLS Authentication on Proxies is dead

2019-11-26 Thread GitBox
Jennifer88huang edited a comment on issue #5753: [Doc] link for TLS Authentication on Proxies is dead URL: https://github.com/apache/pulsar/issues/5753#issuecomment-558916272 @Anonymitaet @wangyunpeng666 Have you solved the issue? The link does not work currently. ---

[GitHub] [pulsar] kaykillyou commented on issue #5668: bookie server's log error: Failed to restore rocksdb 000000000000000000/000000000000000001/000000000000000000

2019-11-26 Thread GitBox
kaykillyou commented on issue #5668: bookie server's log error: Failed to restore rocksdb 00/01/00 URL: https://github.com/apache/pulsar/issues/5668#issuecomment-558916765 @jiazhai Thanks for your reply! today my bookie server found this is

[GitHub] [pulsar] Jennifer88huang commented on issue #5753: [Doc] link for TLS Authentication on Proxies is dead

2019-11-26 Thread GitBox
Jennifer88huang commented on issue #5753: [Doc] link for TLS Authentication on Proxies is dead URL: https://github.com/apache/pulsar/issues/5753#issuecomment-558916272 @Anonymitaet @wangyunpeng666 Have you solved the issue?

[GitHub] [pulsar] jiazhai commented on issue #5746: NPE occur when send an over size message.

2019-11-26 Thread GitBox
jiazhai commented on issue #5746: NPE occur when send an over size message. URL: https://github.com/apache/pulsar/issues/5746#issuecomment-558915882 Thanks @codelipenghui for the fix. This is an automated message from the Apac

[GitHub] [pulsar] massakam commented on issue #5638: Bump jackson-databind from 2.9.9.3 to 2.10.1

2019-11-26 Thread GitBox
massakam commented on issue #5638: Bump jackson-databind from 2.9.9.3 to 2.10.1 URL: https://github.com/apache/pulsar/pull/5638#issuecomment-558915019 retest this please This is an automated message from the Apache Git Service

[GitHub] [pulsar] zymap commented on a change in pull request #5376: [Issue 3009][docs] Add presto sql REST APIs doc

2019-11-26 Thread GitBox
zymap commented on a change in pull request #5376: [Issue 3009][docs] Add presto sql REST APIs doc URL: https://github.com/apache/pulsar/pull/5376#discussion_r351082663 ## File path: site2/docs/sql-rest-api.md ## @@ -0,0 +1,196 @@ +--- +id: sql-rest-api +title: Pulsar SQL

[GitHub] [pulsar] zymap commented on a change in pull request #5376: [Issue 3009][docs] Add presto sql REST APIs doc

2019-11-26 Thread GitBox
zymap commented on a change in pull request #5376: [Issue 3009][docs] Add presto sql REST APIs doc URL: https://github.com/apache/pulsar/pull/5376#discussion_r351082437 ## File path: site2/docs/sql-rest-api.md ## @@ -0,0 +1,196 @@ +--- +id: sql-rest-api +title: Pulsar SQL

[GitHub] [pulsar] zymap commented on a change in pull request #5376: [Issue 3009][docs] Add presto sql REST APIs doc

2019-11-26 Thread GitBox
zymap commented on a change in pull request #5376: [Issue 3009][docs] Add presto sql REST APIs doc URL: https://github.com/apache/pulsar/pull/5376#discussion_r351082071 ## File path: site2/docs/sql-rest-api.md ## @@ -0,0 +1,196 @@ +--- +id: sql-rest-api +title: Pulsar SQL

[pulsar-client-node] branch master updated: Add negative acknowledgement support for consumer (#59)

2019-11-26 Thread nkurihar
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/pulsar-client-node.git The following commit(s) were added to refs/heads/master by this push: new ae91394 Add negative acknowledgem

[GitHub] [pulsar] Anonymitaet commented on issue #5745: Add document for sticky consumer

2019-11-26 Thread GitBox
Anonymitaet commented on issue #5745: Add document for sticky consumer URL: https://github.com/apache/pulsar/pull/5745#issuecomment-558896596 @codelipenghui could you please update the doc? Thank you This is an automated mess

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #5745: Add document for sticky consumer

2019-11-26 Thread GitBox
Anonymitaet commented on a change in pull request #5745: Add document for sticky consumer URL: https://github.com/apache/pulsar/pull/5745#discussion_r350597425 ## File path: site2/docs/client-libraries-java.md ## @@ -607,6 +607,19 @@ consumer 2 will receive: ("key-4", "m

[GitHub] [pulsar] codelipenghui commented on issue #5572: Fix create consumer on partitioned topic while disable topic auto creation.

2019-11-26 Thread GitBox
codelipenghui commented on issue #5572: Fix create consumer on partitioned topic while disable topic auto creation. URL: https://github.com/apache/pulsar/pull/5572#issuecomment-558895168 it can move to 2.5.1 if 2.5.0 branch will start cut soon. Sijie Guo 于2019年11月26日 周二04:57写道:

[GitHub] [pulsar] aahmed-se commented on issue #5642: Add Github workflow for gated checkin

2019-11-26 Thread GitBox
aahmed-se commented on issue #5642: Add Github workflow for gated checkin URL: https://github.com/apache/pulsar/pull/5642#issuecomment-558878990 run java8 tests run integration tests This is an automated message from the Ap

[GitHub] [pulsar] zymap commented on issue #5714: Add more info in the error messages

2019-11-26 Thread GitBox
zymap commented on issue #5714: Add more info in the error messages URL: https://github.com/apache/pulsar/pull/5714#issuecomment-558874357 Yes. There are many exceptions thrown by `ClientCnx`. In `ClientCnx`, it using a map to save `requestId` and CompleableFuture. And the `requestId` is g

[GitHub] [pulsar] Ma3yTa commented on issue #3032: Add gRPC proxy

2019-11-26 Thread GitBox
Ma3yTa commented on issue #3032: Add gRPC proxy URL: https://github.com/apache/pulsar/pull/3032#issuecomment-558867931 Guys, please keep it moving forward. This is an automated message from the Apache Git Service. To respond t

[GitHub] [pulsar-client-go] merlimat merged pull request #106: [CONTRIBUTING.md] Fix repository link

2019-11-26 Thread GitBox
merlimat merged pull request #106: [CONTRIBUTING.md] Fix repository link URL: https://github.com/apache/pulsar-client-go/pull/106 This is an automated message from the Apache Git Service. To respond to the message, please log

[pulsar-client-go] branch master updated: Fix repository link (#106)

2019-11-26 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/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new ae057bd Fix repository link (#106) ae

[GitHub] [pulsar] sijie commented on issue #5718: [Issue 5712][broker] Auth provider superuser

2019-11-26 Thread GitBox
sijie commented on issue #5718: [Issue 5712][broker] Auth provider superuser URL: https://github.com/apache/pulsar/pull/5718#issuecomment-558844171 run cpp tests This is an automated message from the Apache Git Service. To res

[GitHub] [pulsar] sijie commented on issue #5549: Fix documentation: how to start server for cpp tests

2019-11-26 Thread GitBox
sijie commented on issue #5549: Fix documentation: how to start server for cpp tests URL: https://github.com/apache/pulsar/pull/5549#issuecomment-558843690 run java8 tests This is an automated message from the Apache Git Serv

[GitHub] [pulsar] sijie commented on issue #5549: Fix documentation: how to start server for cpp tests

2019-11-26 Thread GitBox
sijie commented on issue #5549: Fix documentation: how to start server for cpp tests URL: https://github.com/apache/pulsar/pull/5549#issuecomment-558843646 run cpp tests This is an automated message from the Apache Git Servic

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

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

[GitHub] [pulsar] sijie commented on issue #5147: Deprecate getstatus subcommand for source command

2019-11-26 Thread GitBox
sijie commented on issue #5147: Deprecate getstatus subcommand for source command URL: https://github.com/apache/pulsar/pull/5147#issuecomment-558843006 run java8 tests This is an automated message from the Apache Git Service

[GitHub] [pulsar] sijie commented on issue #5702: [pulsar-sql]Expose configurations of managed ledger and bookkeeper client.

2019-11-26 Thread GitBox
sijie commented on issue #5702: [pulsar-sql]Expose configurations of managed ledger and bookkeeper client. URL: https://github.com/apache/pulsar/pull/5702#issuecomment-558842594 run integration tests This is an automated mess

[GitHub] [pulsar] sijie commented on issue #5734: [Reopen][Issue 5597]retry when getPartitionedTopicMetadata failed

2019-11-26 Thread GitBox
sijie commented on issue #5734: [Reopen][Issue 5597]retry when getPartitionedTopicMetadata failed URL: https://github.com/apache/pulsar/pull/5734#issuecomment-558842466 run java8 tests This is an automated message from the Ap

[GitHub] [pulsar] sijie commented on issue #5741: [website]Add content of pulsar-manager in website(include release notes, download link)

2019-11-26 Thread GitBox
sijie commented on issue #5741: [website]Add content of pulsar-manager in website(include release notes, download link) URL: https://github.com/apache/pulsar/pull/5741#issuecomment-558842040 @Anonymitaet can you review it again? -

[GitHub] [pulsar] sijie commented on issue #5742: [pulsar-broker] add support to configure max pending publish request per connection

2019-11-26 Thread GitBox
sijie commented on issue #5742: [pulsar-broker] add support to configure max pending publish request per connection URL: https://github.com/apache/pulsar/pull/5742#issuecomment-558841826 run integration tests This is an autom

[GitHub] [pulsar] sijie merged pull request #5754: Init broker publish limiter at broker start time

2019-11-26 Thread GitBox
sijie merged pull request #5754: Init broker publish limiter at broker start time URL: https://github.com/apache/pulsar/pull/5754 This is an automated message from the Apache Git Service. To respond to the message, please lo

[pulsar] branch master updated (a88300c -> eee4d89)

2019-11-26 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 a88300c Adding Spark example to Write Pojo classes to Stream (#5470) add eee4d89 init broker publish limitor a

[GitHub] [pulsar] sijie commented on a change in pull request #5716: [Issues 5709]remove the namespace checking

2019-11-26 Thread GitBox
sijie commented on a change in pull request #5716: [Issues 5709]remove the namespace checking URL: https://github.com/apache/pulsar/pull/5716#discussion_r350956612 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/MultiTopicsConsumerImplTest.java ##

[GitHub] [pulsar] sijie commented on a change in pull request #5716: [Issues 5709]remove the namespace checking

2019-11-26 Thread GitBox
sijie commented on a change in pull request #5716: [Issues 5709]remove the namespace checking URL: https://github.com/apache/pulsar/pull/5716#discussion_r351007391 ## File path: pulsar-client/src/test/java/org/apache/pulsar/client/impl/MultiTopicsConsumerImplTest.java ##

[GitHub] [pulsar-client-go] reugn opened a new pull request #106: [CONTRIBUTING.md] Fix repository link

2019-11-26 Thread GitBox
reugn opened a new pull request #106: [CONTRIBUTING.md] Fix repository link URL: https://github.com/apache/pulsar-client-go/pull/106 ### Motivation Fix repository link ### Verifying this change - [ ] Make sure that the change passes the CI checks. --

[GitHub] [pulsar] sijie edited a comment on issue #5714: Add more info in the error messages

2019-11-26 Thread GitBox
sijie edited a comment on issue #5714: Add more info in the error messages URL: https://github.com/apache/pulsar/pull/5714#issuecomment-558719279 In the first case, you need to find where the exception was first raised and fix it, no? ---

[GitHub] [pulsar] aahmed-se commented on issue #5642: Add Github workflow for gated checkin

2019-11-26 Thread GitBox
aahmed-se commented on issue #5642: Add Github workflow for gated checkin URL: https://github.com/apache/pulsar/pull/5642#issuecomment-558794702 run integration tests This is an automated message from the Apache Git Service. T

[GitHub] [pulsar] aahmed-se commented on issue #5642: Add Github workflow for gated checkin

2019-11-26 Thread GitBox
aahmed-se commented on issue #5642: Add Github workflow for gated checkin URL: https://github.com/apache/pulsar/pull/5642#issuecomment-558794615 run java8 tests This is an automated message from the Apache Git Service. To resp

[GitHub] [pulsar] aahmed-se commented on issue #5642: Add Github workflow for gated checkin

2019-11-26 Thread GitBox
aahmed-se commented on issue #5642: Add Github workflow for gated checkin URL: https://github.com/apache/pulsar/pull/5642#issuecomment-558794523 run java8 tests This is an automated message from the Apache Git Service. To resp

[GitHub] [pulsar] sijie commented on issue #5722: [Issue 5287][docs] Add documentation on how to configure multiple broker services

2019-11-26 Thread GitBox
sijie commented on issue #5722: [Issue 5287][docs] Add documentation on how to configure multiple broker services URL: https://github.com/apache/pulsar/pull/5722#issuecomment-558793705 @tooptoop4 yes. Can you comment about your setup, configuration and the exception you are seeing? Happy t

[pulsar-client-go] branch master updated: Set error only if not nil (#105)

2019-11-26 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/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 9620e75 Set error only if not nil (#1

[GitHub] [pulsar-client-go] merlimat merged pull request #105: [Producer] Set error only if not nil

2019-11-26 Thread GitBox
merlimat merged pull request #105: [Producer] Set error only if not nil URL: https://github.com/apache/pulsar-client-go/pull/105 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [pulsar] tooptoop4 commented on issue #5722: [Issue 5287][docs] Add documentation on how to configure multiple broker services

2019-11-26 Thread GitBox
tooptoop4 commented on issue #5722: [Issue 5287][docs] Add documentation on how to configure multiple broker services URL: https://github.com/apache/pulsar/pull/5722#issuecomment-558781959 did u try it @sijie This is an auto

[GitHub] [pulsar-client-go] reugn opened a new pull request #105: [Producer] Set error only if not nil

2019-11-26 Thread GitBox
reugn opened a new pull request #105: [Producer] Set error only if not nil URL: https://github.com/apache/pulsar-client-go/pull/105 ### Motivation Fix error set bug in impl_producer.go ### Modifications ```go if pe.err != nil { err = pe.err } ``` ###

[GitHub] [pulsar] codelipenghui commented on issue #5680: [transaction-coordinator] Implementation of transaction coordinator client.

2019-11-26 Thread GitBox
codelipenghui commented on issue #5680: [transaction-coordinator] Implementation of transaction coordinator client. URL: https://github.com/apache/pulsar/pull/5680#issuecomment-558727630 run java8 tests run cpp tests This

[pulsar] branch master updated (14f3eee -> a88300c)

2019-11-26 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 14f3eee [Issue 5287][docs] Add documentation on how to configure multiple broker services (#5722) add a88300c

[GitHub] [pulsar] sijie merged pull request #5470: Adding Spark example to Write Pojo classes to Stream

2019-11-26 Thread GitBox
sijie merged pull request #5470: Adding Spark example to Write Pojo classes to Stream URL: https://github.com/apache/pulsar/pull/5470 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [pulsar] sijie commented on a change in pull request #5713: avoid unsafe split when validate hostname which might be ipv6 address

2019-11-26 Thread GitBox
sijie commented on a change in pull request #5713: avoid unsafe split when validate hostname which might be ipv6 address URL: https://github.com/apache/pulsar/pull/5713#discussion_r350864806 ## File path: pulsar-common/src/test/java/org/apache/pulsar/common/net/ServiceURITest.java

[pulsar] branch master updated (9e7b938 -> f83793e)

2019-11-26 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 9e7b938 Add publish rate limit for each broker to avoid OOM (#5710) add f83793e fix potential deadlock in pulsa

[pulsar] branch master updated: [Issue 5287][docs] Add documentation on how to configure multiple broker services (#5722)

2019-11-26 Thread sijie
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/pulsar.git The following commit(s) were added to refs/heads/master by this push: new 14f3eee [Issue 5287][docs] Add documentation on

[GitHub] [pulsar] sijie merged pull request #5722: [Issue 5287][docs] Add documentation on how to configure multiple broker services

2019-11-26 Thread GitBox
sijie merged pull request #5722: [Issue 5287][docs] Add documentation on how to configure multiple broker services URL: https://github.com/apache/pulsar/pull/5722 This is an automated message from the Apache Git Service. To

[GitHub] [pulsar] sijie closed issue #5287: presto connector - how to specify cluster (multi broker/zk) connection

2019-11-26 Thread GitBox
sijie closed issue #5287: presto connector - how to specify cluster (multi broker/zk) connection URL: https://github.com/apache/pulsar/issues/5287 This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [pulsar] sijie commented on issue #5714: Add more info in the error messages

2019-11-26 Thread GitBox
sijie commented on issue #5714: Add more info in the error messages URL: https://github.com/apache/pulsar/pull/5714#issuecomment-558719279 In the first case, you need to first where the exception was first raised and fix it, no? -

[GitHub] [pulsar] sijie merged pull request #5731: fix potential deadlock in pulsar client close

2019-11-26 Thread GitBox
sijie merged pull request #5731: fix potential deadlock in pulsar client close URL: https://github.com/apache/pulsar/pull/5731 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [pulsar] zymap commented on issue #5714: Add more info in the error messages

2019-11-26 Thread GitBox
zymap commented on issue #5714: Add more info in the error messages URL: https://github.com/apache/pulsar/pull/5714#issuecomment-558706669 @sijie There are two ways when we threw an Exception. One is we get an exception and throw it out. ``` future.exceptionally(e -> {

[GitHub] [pulsar] tuteng commented on issue #5741: [website]Add content of pulsar-manager in website(include release notes, download link)

2019-11-26 Thread GitBox
tuteng commented on issue #5741: [website]Add content of pulsar-manager in website(include release notes, download link) URL: https://github.com/apache/pulsar/pull/5741#issuecomment-558652972 Thank you for your comments. This

[GitHub] [pulsar] tuteng commented on a change in pull request #5741: [website]Add content of pulsar-manager in website(include release notes, download link)

2019-11-26 Thread GitBox
tuteng commented on a change in pull request #5741: [website]Add content of pulsar-manager in website(include release notes, download link) URL: https://github.com/apache/pulsar/pull/5741#discussion_r350771298 ## File path: site2/docs/administration-pulsar-manager.md ## @@

[GitHub] [pulsar] jiazhai opened a new pull request #5754: Init broker publish limiter at broker start time

2019-11-26 Thread GitBox
jiazhai opened a new pull request #5754: Init broker publish limiter at broker start time URL: https://github.com/apache/pulsar/pull/5754 Broker publish limiter was not start at broker start time, and only init when dynamic system config happens. This change init broker publish limiter

[GitHub] [pulsar] wangyunpeng666 commented on issue #5753: [Doc] link for TLS Authentication on Proxies is dead

2019-11-26 Thread GitBox
wangyunpeng666 commented on issue #5753: [Doc] link for TLS Authentication on Proxies is dead URL: https://github.com/apache/pulsar/issues/5753#issuecomment-558621286 @Anonymitaet Thank you for solving my problem This is an

[GitHub] [pulsar] wangyunpeng666 opened a new issue #5753: [Doc] link for TLS Authentication on Proxies is dead

2019-11-26 Thread GitBox
wangyunpeng666 opened a new issue #5753: [Doc] link for TLS Authentication on Proxies is dead URL: https://github.com/apache/pulsar/issues/5753 Expected behavior Show a page / Enabling TLS Authentication on Proxies Actual behavior 404 Steps to reproduce 1、Open https://pulsa

[GitHub] [pulsar] cbornet opened a new issue #5752: Expose key shared consumer selector range in API

2019-11-26 Thread GitBox
cbornet opened a new issue #5752: Expose key shared consumer selector range in API URL: https://github.com/apache/pulsar/issues/5752 **Is your feature request related to a problem? Please describe.** When using Key-Shared subscription, there's currently no way to know which consumer wil

[GitHub] [pulsar] vivekpd15 commented on issue #4718: failed to get Partitioned metadata : Policies not found for mytenant/mynamespace namespace

2019-11-26 Thread GitBox
vivekpd15 commented on issue #4718: failed to get Partitioned metadata : Policies not found for mytenant/mynamespace namespace URL: https://github.com/apache/pulsar/issues/4718#issuecomment-558618213 I am facing the same issue. @f-ld were you able to resolve this? -

[GitHub] [pulsar] lxkaka commented on issue #5711: K8s: through pulsar proxy visit api return 502

2019-11-26 Thread GitBox
lxkaka commented on issue #5711: K8s: through pulsar proxy visit api return 502 URL: https://github.com/apache/pulsar/issues/5711#issuecomment-558614853 Other components work well, sometimes proxy return 200, but more proable return 520. so service of proxy is configured correctly. ```

[GitHub] [pulsar] Anonymitaet commented on issue #5549: Fix documentation: how to start server for cpp tests

2019-11-26 Thread GitBox
Anonymitaet commented on issue #5549: Fix documentation: how to start server for cpp tests URL: https://github.com/apache/pulsar/pull/5549#issuecomment-558603655 run java8 tests This is an automated message from the Apache Gi

[GitHub] [pulsar] codelipenghui commented on issue #5748: Fix NPE and release already released ByteBuf in batch message container

2019-11-26 Thread GitBox
codelipenghui commented on issue #5748: Fix NPE and release already released ByteBuf in batch message container URL: https://github.com/apache/pulsar/pull/5748#issuecomment-558598022 run cpp tests run java8 tests This is a

[GitHub] [pulsar] codelipenghui opened a new issue #5751: [DISCUSS] Avoid broker OOM

2019-11-26 Thread GitBox
codelipenghui opened a new issue #5751: [DISCUSS] Avoid broker OOM URL: https://github.com/apache/pulsar/issues/5751 **Is your feature request related to a problem? Please describe.** Currently, we have do some performance tests, if broker with a smaller direct memory(e.g. 2G) and run pu

[GitHub] [pulsar] wolfstudy commented on issue #5147: Deprecate getstatus subcommand for source command

2019-11-26 Thread GitBox
wolfstudy commented on issue #5147: Deprecate getstatus subcommand for source command URL: https://github.com/apache/pulsar/pull/5147#issuecomment-558588490 run cpp tests run java8 tests This is an automated message from t

[GitHub] [pulsar] wolfstudy commented on issue #5549: Fix documentation: how to start server for cpp tests

2019-11-26 Thread GitBox
wolfstudy commented on issue #5549: Fix documentation: how to start server for cpp tests URL: https://github.com/apache/pulsar/pull/5549#issuecomment-558588303 run java8 tests This is an automated message from the Apache Git

[GitHub] [pulsar] wolfstudy commented on issue #4753: [docs] Add documentation for Functions develop: encryption

2019-11-26 Thread GitBox
wolfstudy commented on issue #4753: [docs] Add documentation for Functions develop: encryption URL: https://github.com/apache/pulsar/issues/4753#issuecomment-558587705 > can you add the functions documentation for 2.5.0? It is better to make documentation as completed as possible in 2.5.0.

[GitHub] [pulsar] Jennifer88huang opened a new issue #4554: [docs] Improve the structure of Pulsar Functions

2019-11-26 Thread GitBox
Jennifer88huang opened a new issue #4554: [docs] Improve the structure of Pulsar Functions URL: https://github.com/apache/pulsar/issues/4554 **Is your feature request related to a problem? Please describe.** I'm a little confused when searching information on Pulsar Functions. The conte

[GitHub] [pulsar] Jennifer88huang opened a new pull request #5750: [WIP][Issue 4752][docs] Add language tabs and description for Go

2019-11-26 Thread GitBox
Jennifer88huang opened a new pull request #5750: [WIP][Issue 4752][docs] Add language tabs and description for Go URL: https://github.com/apache/pulsar/pull/5750 Fixes #4752 ### Motivation Currently, the following features are not available in Go Functions: SerDe, metrics, state

[GitHub] [pulsar] jiazhai edited a comment on issue #5711: K8s: through pulsar proxy visit api return 502

2019-11-26 Thread GitBox
jiazhai edited a comment on issue #5711: K8s: through pulsar proxy visit api return 502 URL: https://github.com/apache/pulsar/issues/5711#issuecomment-558495847 Seems local deploy works well, This may related to K8s configs. @lxkaka Would you mind share a little details of how you deployed

[GitHub] [pulsar] zymap commented on issue #5001: [Pulsar-sql]Using pulsar SQL query messages will appear `NoSuchLedger` when zk root directory changed

2019-11-26 Thread GitBox
zymap commented on issue #5001: [Pulsar-sql]Using pulsar SQL query messages will appear `NoSuchLedger` when zk root directory changed URL: https://github.com/apache/pulsar/pull/5001#issuecomment-558585093 @sijie @jiazhai PTAL. ---

[GitHub] [pulsar] KannarFr commented on issue #4739: offload bucket per namespace

2019-11-26 Thread GitBox
KannarFr commented on issue #4739: offload bucket per namespace URL: https://github.com/apache/pulsar/pull/4739#issuecomment-558579498 @cdbartholomew Hi, do you still want to work on it? :) This is an automated message from th

[GitHub] [pulsar] KannarFr commented on issue #5723: Quota, retention (macro management) on tenant level.

2019-11-26 Thread GitBox
KannarFr commented on issue #5723: Quota, retention (macro management) on tenant level. URL: https://github.com/apache/pulsar/issues/5723#issuecomment-558579187 @jiazhai will write one. This is an automated message from the A

[GitHub] [pulsar] huangdx0726 commented on issue #5737: [Issue 5729]Picture and text are not match

2019-11-26 Thread GitBox
huangdx0726 commented on issue #5737: [Issue 5729]Picture and text are not match URL: https://github.com/apache/pulsar/pull/5737#issuecomment-558572043 @jiazhai I have fixed it,PTAL. This is an automated message from the Apa

[GitHub] [pulsar] PierreZ commented on issue #5735: Provide raw counters metrics

2019-11-26 Thread GitBox
PierreZ commented on issue #5735: Provide raw counters metrics URL: https://github.com/apache/pulsar/issues/5735#issuecomment-558568942 I like `pulsar_in_bytes_total`! This is an automated message from the Apache Git Service.

[GitHub] [pulsar] codelipenghui commented on issue #5748: Fix NPE and release already released ByteBuf in batch message container

2019-11-26 Thread GitBox
codelipenghui commented on issue #5748: Fix NPE and release already released ByteBuf in batch message container URL: https://github.com/apache/pulsar/pull/5748#issuecomment-558560776 run cpp tests run java8 tests run integration tests -

[GitHub] [pulsar] codelipenghui commented on issue #5745: Add document for sticky consumer

2019-11-26 Thread GitBox
codelipenghui commented on issue #5745: Add document for sticky consumer URL: https://github.com/apache/pulsar/pull/5745#issuecomment-558560495 > Looking forward we have sticky consumer docs for other client libraries, such as Go, Python, C++, and so on. I'm glad to help as well. Th

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #5745: Add document for sticky consumer

2019-11-26 Thread GitBox
Anonymitaet commented on a change in pull request #5745: Add document for sticky consumer URL: https://github.com/apache/pulsar/pull/5745#discussion_r350643393 ## File path: site2/docs/client-libraries-java.md ## @@ -607,6 +607,19 @@ consumer 2 will receive: ("key-4", "m

[GitHub] [pulsar] Anonymitaet commented on a change in pull request #5745: Add document for sticky consumer

2019-11-26 Thread GitBox
Anonymitaet commented on a change in pull request #5745: Add document for sticky consumer URL: https://github.com/apache/pulsar/pull/5745#discussion_r350621322 ## File path: site2/docs/client-libraries-java.md ## @@ -607,6 +607,19 @@ consumer 2 will receive: ("key-4", "m

[GitHub] [pulsar] vishalananth07 opened a new issue #5749: Bookie does not start because of UnknownHostException

2019-11-26 Thread GitBox
vishalananth07 opened a new issue #5749: Bookie does not start because of UnknownHostException URL: https://github.com/apache/pulsar/issues/5749 We are trying to set up Pulsar using 3 systems each with 8GB RAM and AMD A8 processors. We are planning on running bookkeeper, zookeeper, broker

  1   2   >