[GitHub] [pulsar] BewareMyPower opened a new issue #6822: Segmentation fault of C++ client when close a producer if there're pending expired messages

2020-04-25 Thread GitBox
BewareMyPower opened a new issue #6822: URL: https://github.com/apache/pulsar/issues/6822 **Describe the bug** With C++ client, if a `Producer` call `close()` while there're some pending expired messages, after that, a segmentation fault would happen. **To Reproduce** Steps to

[GitHub] [pulsar-client-go] JoseFMP commented on issue #234: Cannot build the project

2020-04-25 Thread GitBox
JoseFMP commented on issue #234: URL: https://github.com/apache/pulsar-client-go/issues/234#issuecomment-619487431 @wolfstudy I installed the libraries. Seems correct but now I get another build error. I updated the bug description.

[GitHub] [pulsar-client-go] JoseFMP commented on issue #234: Cannot build the project

2020-04-25 Thread GitBox
JoseFMP commented on issue #234: URL: https://github.com/apache/pulsar-client-go/issues/234#issuecomment-619483566 Another thing I am thinking, wouldn't it be better to offer the Go client with those binaries already built?

[GitHub] [pulsar-client-go] JoseFMP commented on issue #234: Cannot build the project

2020-04-25 Thread GitBox
JoseFMP commented on issue #234: URL: https://github.com/apache/pulsar-client-go/issues/234#issuecomment-619483349 Thank you, I am going to look at it and see if I can build and that was the issue. Do you know other dependencies that I need to have before I can use the package?

[GitHub] [pulsar] hangc0276 commented on pull request #6776: [Docs] Fix the Create subscribtion swagger of PersistentTopic

2020-04-25 Thread GitBox
hangc0276 commented on pull request #6776: URL: https://github.com/apache/pulsar/pull/6776#issuecomment-619477045 > > But i am afraid your present implementation will compile failed due to lack of `MessageIdImpl` constructor for `@QueryParam("messageId") @DefaultValue("latest")

[GitHub] [pulsar] ltamber commented on pull request #6524: [Issue 6319][Pulsar client] connection leak fix

2020-04-25 Thread GitBox
ltamber commented on pull request #6524: URL: https://github.com/apache/pulsar/pull/6524#issuecomment-619475445 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] zplinuxlover commented on pull request #6775: change `org.apache.pulsar.broker.admin.impl.NamespacesBase` code

2020-04-25 Thread GitBox
zplinuxlover commented on pull request #6775: URL: https://github.com/apache/pulsar/pull/6775#issuecomment-619473059 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] zplinuxlover commented on pull request #6775: change `org.apache.pulsar.broker.admin.impl.NamespacesBase` code

2020-04-25 Thread GitBox
zplinuxlover commented on pull request #6775: URL: https://github.com/apache/pulsar/pull/6775#issuecomment-619467005 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] surendra-k opened a new issue #6821: Pulsar+SSL Java client issue (TLS encryption) via Nginx

2020-04-25 Thread GitBox
surendra-k opened a new issue #6821: URL: https://github.com/apache/pulsar/issues/6821 Expected behavior Should be able to connect to pulsar+ssl proxy and create producer/consumer Actual behavior Environment: Java8, pulsar-java-client, mac 10.14.6 Pulsar setup:

[GitHub] [pulsar] cckellogg commented on a change in pull request #6752: Support different docker images in Kubernetes runtime of Pulsar Functions

2020-04-25 Thread GitBox
cckellogg commented on a change in pull request #6752: URL: https://github.com/apache/pulsar/pull/6752#discussion_r415115606 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/kubernetes/KubernetesRuntime.java ## @@ -973,8 +982,19 @@

[GitHub] [pulsar] cckellogg commented on a change in pull request #6752: Support different docker images in Kubernetes runtime of Pulsar Functions

2020-04-25 Thread GitBox
cckellogg commented on a change in pull request #6752: URL: https://github.com/apache/pulsar/pull/6752#discussion_r415115606 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/kubernetes/KubernetesRuntime.java ## @@ -973,8 +982,19 @@

[GitHub] [pulsar-client-go] wolfstudy edited a comment on issue #223: Reader can not read new messages after subscribing with LatestMessageID

2020-04-25 Thread GitBox
wolfstudy edited a comment on issue #223: URL: https://github.com/apache/pulsar-client-go/issues/223#issuecomment-619392817 Thanks @cornelk feedback, @jiazhai maybe this is not a bug. In the first send, we can't read the message, because the reader `StartMessageID` option is

[GitHub] [pulsar-client-go] wolfstudy commented on issue #223: Reader can not read new messages after subscribing with LatestMessageID

2020-04-25 Thread GitBox
wolfstudy commented on issue #223: URL: https://github.com/apache/pulsar-client-go/issues/223#issuecomment-619392817 Thanks @cornelk feedback, @jiazhai maybe this is not a bug. In the first send, we can't read the message, because the reader `StartMessageID` option is

[GitHub] [pulsar-client-go] wolfstudy edited a comment on issue #234: Cannot build the project

2020-04-25 Thread GitBox
wolfstudy edited a comment on issue #234: URL: https://github.com/apache/pulsar-client-go/issues/234#issuecomment-619390621 Thanks @JoseFMP feedback. Currently, you are building a [cgo version of pulsar-client-go](https://github.com/apache/pulsar/tree/master/pulsar-client-go), this repo

[GitHub] [pulsar-client-go] wolfstudy commented on issue #234: Cannot build the project

2020-04-25 Thread GitBox
wolfstudy commented on issue #234: URL: https://github.com/apache/pulsar-client-go/issues/234#issuecomment-619390908 > Actual behavior > An error appears when trying to build the client: > > ``` > ./c_go_pulsar.h:22:10: fatal error: pulsar/c/client.h: No such file or

[GitHub] [pulsar-client-go] wolfstudy edited a comment on issue #234: Cannot build the project

2020-04-25 Thread GitBox
wolfstudy edited a comment on issue #234: URL: https://github.com/apache/pulsar-client-go/issues/234#issuecomment-619390621 Thanks @JoseFMP feedback. Currently, you are building a cgo version of pulsar-client-go, this repo is a native go client. And the repo

[GitHub] [pulsar-client-go] wolfstudy commented on issue #234: Cannot build the project

2020-04-25 Thread GitBox
wolfstudy commented on issue #234: URL: https://github.com/apache/pulsar-client-go/issues/234#issuecomment-619390621 Thanks @JoseFMP feedback. Currently, you are building a cgo version of pulsar-client-go, this repo is a native go client. And the repo [Goal](Currently, you are building a

[GitHub] [pulsar-client-go] wolfstudy commented on issue #233: Reader cannot read first message produced to new topic

2020-04-25 Thread GitBox
wolfstudy commented on issue #233: URL: https://github.com/apache/pulsar-client-go/issues/233#issuecomment-619390303 @frankjkelly Thanks feedback. Maybe we can try the following methods: ``` consumer, err := writerClient.Subscribe(pulsar.ConsumerOptions{

[GitHub] [pulsar] wolfstudy commented on pull request #6776: [Docs] Fix the Create subscribtion swagger of PersistentTopic

2020-04-25 Thread GitBox
wolfstudy commented on pull request #6776: URL: https://github.com/apache/pulsar/pull/6776#issuecomment-619388651 > But i am afraid your present implementation will compile failed due to lack of `MessageIdImpl` constructor for `@QueryParam("messageId") @DefaultValue("latest")

[GitHub] [pulsar-client-go] JoseFMP opened a new issue #234: Cannot build the project

2020-04-25 Thread GitBox
JoseFMP opened a new issue #234: URL: https://github.com/apache/pulsar-client-go/issues/234 Expected behavior The client in this repository should build. Actual behavior An error appears when trying to build the client: ``` ./c_go_pulsar.h:22:10: fatal

[GitHub] [pulsar] nitinmuteja opened a new issue #6820: Persistent topic put request doesn't return 409 error if topic exists

2020-04-25 Thread GitBox
nitinmuteja opened a new issue #6820: URL: https://github.com/apache/pulsar/issues/6820 **Describe the bug** A clear and concise description of what the bug is. **To Reproduce** Steps to reproduce the behavior: 1. Post a put request to

[GitHub] [pulsar-client-go] wolfstudy commented on pull request #215: [Issue 213] Fix data race when accessing partition producer state

2020-04-25 Thread GitBox
wolfstudy commented on pull request #215: URL: https://github.com/apache/pulsar-client-go/pull/215#issuecomment-619386514 ping @merlimat @cornelk can you help to check whether this problem also exists on the consumer side?

[GitHub] [pulsar-client-go] wolfstudy commented on issue #218: Seek function is missing in Reader

2020-04-25 Thread GitBox
wolfstudy commented on issue #218: URL: https://github.com/apache/pulsar-client-go/issues/218#issuecomment-619385227 > `Seek()` and `SeekByTime()` are available in `Consumer` but not `Reader`. The C++ Client has the function in both classes. Yes, in the next version, we can support

[GitHub] [pulsar-client-go] wolfstudy commented on issue #231: Race in partitionProducer Send on err access

2020-04-25 Thread GitBox
wolfstudy commented on issue #231: URL: https://github.com/apache/pulsar-client-go/issues/231#issuecomment-619385017 @cornelk Thanks feedback, If possible, can you help to send a new pull request to fix this? This is an

[GitHub] [pulsar-client-go] wolfstudy commented on issue #217: Support ack timeout

2020-04-25 Thread GitBox
wolfstudy commented on issue #217: URL: https://github.com/apache/pulsar-client-go/issues/217#issuecomment-619384804 > @hrsakai Put this into our backlog. @jiazhai The ack timeout was intentionally left out from the Go client. Maybe we don't put this into our backlog

[GitHub] [pulsar-client-go] wolfstudy edited a comment on issue #217: Support ack timeout

2020-04-25 Thread GitBox
wolfstudy edited a comment on issue #217: URL: https://github.com/apache/pulsar-client-go/issues/217#issuecomment-619384804 > @hrsakai Put this into our backlog. @jiazhai The ack timeout was intentionally left out from the Go client. Maybe we don't put this into our backlog,

[GitHub] [pulsar-client-go] wolfstudy commented on issue #217: Support ack timeout

2020-04-25 Thread GitBox
wolfstudy commented on issue #217: URL: https://github.com/apache/pulsar-client-go/issues/217#issuecomment-619384577 > We are currently considering using go client. Welcome This is an automated message from the

[GitHub] [pulsar-client-go] wolfstudy commented on issue #217: Support ack timeout

2020-04-25 Thread GitBox
wolfstudy commented on issue #217: URL: https://github.com/apache/pulsar-client-go/issues/217#issuecomment-619384515 > We want to know the detailed reason. AckTimeout is actually a legacy problem, we can use nack instead. In many cases, it is hard for the application to handle the

[GitHub] [pulsar-client-go] wolfstudy commented on issue #232: Pulsar Message Backlog:consumer can't consume messages!!!!

2020-04-25 Thread GitBox
wolfstudy commented on issue #232: URL: https://github.com/apache/pulsar-client-go/issues/232#issuecomment-619381655 > OK. I used the pulsar-admin topic stats, it displays like this the subscriptions of the topic message backlog. I find the error msg in the broker log like this: "

[pulsar-client-go] branch master updated: Add flag to disable forced topic creation. (#226)

2020-04-25 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 6edc8f4 Add flag to disable forced

[pulsar-client-go] branch master updated: [auth] Add Athenz authentication provider (#227)

2020-04-25 Thread rxl
This is an automated email from the ASF dual-hosted git repository. rxl pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar-client-go.git The following commit(s) were added to refs/heads/master by this push: new 0f822ef [auth] Add Athenz

[GitHub] [pulsar] hangc0276 commented on pull request #6776: [Docs] Fix the Create subscribtion swagger of PersistentTopic

2020-04-25 Thread GitBox
hangc0276 commented on pull request #6776: URL: https://github.com/apache/pulsar/pull/6776#issuecomment-619380515 > > > @hangc0276 Good catch! @wolfstudy could please help take a look the last comment? maybe you need to move the `messageId` to the end of params list. > > > > > >

[GitHub] [pulsar] wolfstudy commented on a change in pull request #6752: Support different docker images in Kubernetes runtime of Pulsar Functions

2020-04-25 Thread GitBox
wolfstudy commented on a change in pull request #6752: URL: https://github.com/apache/pulsar/pull/6752#discussion_r415055733 ## File path: pulsar-functions/runtime/src/main/java/org/apache/pulsar/functions/runtime/kubernetes/KubernetesRuntime.java ## @@ -973,8 +982,19 @@

[GitHub] [pulsar] wolfstudy commented on pull request #6776: [Docs] Fix the Create subscribtion swagger of PersistentTopic

2020-04-25 Thread GitBox
wolfstudy commented on pull request #6776: URL: https://github.com/apache/pulsar/pull/6776#issuecomment-619373170 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] wolfstudy commented on pull request #6776: [Docs] Fix the Create subscribtion swagger of PersistentTopic

2020-04-25 Thread GitBox
wolfstudy commented on pull request #6776: URL: https://github.com/apache/pulsar/pull/6776#issuecomment-619373142 > > @hangc0276 Good catch! @wolfstudy could please help take a look the last comment? maybe you need to move the `messageId` to the end of params list. > > Thanks for

[GitHub] [pulsar] lookhua opened a new issue #6819: k8s deploy values.yaml metadataPrefix: "/configuration-store"

2020-04-25 Thread GitBox
lookhua opened a new issue #6819: URL: https://github.com/apache/pulsar/issues/6819 in k8s helm deploy mode. when i set metadataPrefix: "/configuration-store" and set components: zookeeper: true bookkeeper: true broker: true in the values.yaml then run

[GitHub] [pulsar] guyv opened a new issue #6818: pulsar-client vulnerability CVE-2015-5237 (shaded protobuf-java:2.4.)

2020-04-25 Thread GitBox
guyv opened a new issue #6818: URL: https://github.com/apache/pulsar/issues/6818 com.google.protobuf:protobuf-java:2.4.1 has a vulnerability issue: https://nvd.nist.gov/vuln/detail/CVE-2015-5237 This jar is shaded for the pulsar java client. **Possible solution** upgrade

[GitHub] [pulsar] equanz commented on pull request #6794: [Issue 6793][broker] Add config of maxNumPartitionsPerPartitionedTopic

2020-04-25 Thread GitBox
equanz commented on pull request #6794: URL: https://github.com/apache/pulsar/pull/6794#issuecomment-619361227 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] hangc0276 commented on pull request #6814: Expose managed ledger bookie client metric to prometheus

2020-04-25 Thread GitBox
hangc0276 commented on pull request #6814: URL: https://github.com/apache/pulsar/pull/6814#issuecomment-619358233 > The change looks good. Could you please help add the description of the new metric into https://pulsar.apache.org/docs/en/reference-metrics/#broker? This is very helpful for

[GitHub] [pulsar] hangc0276 commented on a change in pull request #6814: Expose managed ledger bookie client metric to prometheus

2020-04-25 Thread GitBox
hangc0276 commented on a change in pull request #6814: URL: https://github.com/apache/pulsar/pull/6814#discussion_r415034605 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/stats/prometheus/PrometheusTextFormatUtil.java ## @@ -0,0 +1,152 @@ +/** + *

[pulsar] branch master updated (fc69a62 -> 1f852ee)

2020-04-25 Thread zhaijia
This is an automated email from the ASF dual-hosted git repository. zhaijia pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git. from fc69a62 [C++] Fix message id is always the default value in send callback (#6812) add 1f852ee Adjust

[GitHub] [pulsar] vzhikserg commented on pull request #6770: [Grafana] Add tags to all grafana dashboards

2020-04-25 Thread GitBox
vzhikserg commented on pull request #6770: URL: https://github.com/apache/pulsar/pull/6770#issuecomment-619352086 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] tuteng opened a new pull request #6817: [website]Adjust version order

2020-04-25 Thread GitBox
tuteng opened a new pull request #6817: URL: https://github.com/apache/pulsar/pull/6817 ### Motivation * After the search is available, set 2.5.1 to the top ### Modifications * After the search is available, set 2.5.1 to the top

[GitHub] [pulsar] hangc0276 commented on pull request #6816: update topicName document to avoid confusion

2020-04-25 Thread GitBox
hangc0276 commented on pull request #6816: URL: https://github.com/apache/pulsar/pull/6816#issuecomment-619342090 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar] wonko commented on issue #6496: ReceiverQueueSize: -1 prefetches one message

2020-04-25 Thread GitBox
wonko commented on issue #6496: URL: https://github.com/apache/pulsar/issues/6496#issuecomment-61980 A gentle nudge here... What can we do to move this forward, or can someone acknowledge if this is intended or a bug?

[pulsar] branch master updated: [C++] Fix message id is always the default value in send callback (#6812)

2020-04-25 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.git The following commit(s) were added to refs/heads/master by this push: new fc69a62 [C++] Fix message id is always the

[pulsar] branch master updated: Make messageReceiveTimeoutMs in the PulsarConsumerSource configurable (#6783)

2020-04-25 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 47b4dd0 Make messageReceiveTimeoutMs in the

[pulsar] branch master updated: expose managedLedgerCache, managedLedger, loadBalance metrics to prometheus (#6705)

2020-04-25 Thread penghui
This is an automated email from the ASF dual-hosted git repository. penghui 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 a9ed984 expose managedLedgerCache,

[GitHub] [pulsar] codelipenghui commented on a change in pull request #6814: Expose managed ledger bookie client metric to prometheus

2020-04-25 Thread GitBox
codelipenghui commented on a change in pull request #6814: URL: https://github.com/apache/pulsar/pull/6814#discussion_r414996603 ## File path: managed-ledger/src/main/java/org/apache/bookkeeper/mledger/stats/prometheus/PrometheusMetricsProvider.java ## @@ -0,0 +1,154 @@ +/**