[GitHub] [pulsar-helm-chart] sijie merged pull request #13: Use regular 2-2-2 BK client settings by default

2020-05-21 Thread GitBox
sijie merged pull request #13: URL: https://github.com/apache/pulsar-helm-chart/pull/13 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [DISCUSS] PIP-65: Adapting Pulsar IO Sources to support Batch Sources

2020-05-21 Thread Devin Bost
I apologize for not fully understanding the context here, but is the concern about using the existing function architecture the complexity of needing two sequential operations in a function flow to be synchronous with respect to transactions, such as to avoid race conditions and issues with paralle

[GitHub] [pulsar-client-node] equanz commented on pull request #88: Adds NodeJS version to the readme file

2020-05-21 Thread GitBox
equanz commented on pull request #88: URL: https://github.com/apache/pulsar-client-node/pull/88#issuecomment-632436457 My environment is macOS 10.15.4, but it seems that be able to install pulsar-client. ``` $ sudo n 10.16.3 installed : v10.16.3 (with npm 6.9.0) $ sw_

[GitHub] [pulsar-client-node] jiazhai closed issue #90: Does pulsar-client-node support JWT authentication?

2020-05-21 Thread GitBox
jiazhai closed issue #90: URL: https://github.com/apache/pulsar-client-node/issues/90 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go t

[GitHub] [pulsar-client-node] jiazhai commented on issue #90: Does pulsar-client-node support JWT authentication?

2020-05-21 Thread GitBox
jiazhai commented on issue #90: URL: https://github.com/apache/pulsar-client-node/issues/90#issuecomment-632433269 @alphara would like to close this issue. This is an automated message from the Apache Git Service. To respond

[GitHub] [pulsar-client-node] massakam commented on pull request #88: Adds NodeJS version to the readme file

2020-05-21 Thread GitBox
massakam commented on pull request #88: URL: https://github.com/apache/pulsar-client-node/pull/88#issuecomment-632432680 Yes, my environment is CentOS 7. This is an automated message from the Apache Git Service. To respond to

Re: ReadOnly Topic Ownership Support

2020-05-21 Thread Jason O'Gray
I'm curious what the current work around is: 1) a partitioned topic to help spread the load or 2) reading directly from the bookie? Best. On Thu, May 21, 2020, 2:29 PM Joe F wrote: > Very useful feature. > > I would like the proposers to think just beyond scaling consumers. If done > right, th

[GitHub] [pulsar-client-node] pikitgb commented on pull request #88: Adds NodeJS version to the readme file

2020-05-21 Thread GitBox
pikitgb commented on pull request #88: URL: https://github.com/apache/pulsar-client-node/pull/88#issuecomment-632409993 uhm based on your folder structure it doesn't look like you are running Catalina 10.15/Mac. This is an

[GitHub] [pulsar-helm-chart] merlimat opened a new pull request #13: Use regular 2-2-2 BK client settings by default

2020-05-21 Thread GitBox
merlimat opened a new pull request #13: URL: https://github.com/apache/pulsar-helm-chart/pull/13 ### Motivation Using write=3 and ack=2 leads to unbound memory usage in BK client when one bookie is slow or failing, so we should avoid it by default. -

Re: ReadOnly Topic Ownership Support

2020-05-21 Thread Joe F
Very useful feature. I would like the proposers to think just beyond scaling consumers. If done right, this has the potential to open up a lot of use cases in ML, where you need to reprocess old/archived data. Being able to spin up a read-only broker, ( dedicated brokers that read from tiered s

[GitHub] [pulsar-manager] KannarFr commented on issue #221: Use environment variables

2020-05-21 Thread GitBox
KannarFr commented on issue #221: URL: https://github.com/apache/pulsar-manager/issues/221#issuecomment-632178504 I didn't work on it sorry, no time yet, maybe in the future. This is an automated message from the Apache Git S

[GitHub] [pulsar-client-node] alphara commented on issue #90: Does pulsar-client-node support JWT authentication?

2020-05-21 Thread GitBox
alphara commented on issue #90: URL: https://github.com/apache/pulsar-client-node/issues/90#issuecomment-632166107 We found this in code: https://github.com/apache/pulsar-client-node/blob/master/src/AuthenticationToken.js ---

[GitHub] [pulsar-client-node] alphara opened a new issue #90: Does pulsar-client-node support JWT authentication?

2020-05-21 Thread GitBox
alphara opened a new issue #90: URL: https://github.com/apache/pulsar-client-node/issues/90 We want our service that uses node.js pulsar client to authenticate using JWT. I don't see [here](http://pulsar.apache.org/docs/en/security-jwt/) that Node.js client supports authentication using to

[GitHub] [pulsar-manager] tuteng commented on issue #299: Unable to get anything about a partition: TypeError: Cannot read property 'status' of undefined

2020-05-21 Thread GitBox
tuteng commented on issue #299: URL: https://github.com/apache/pulsar-manager/issues/299#issuecomment-632120570 This option `pagehelper.helperDialect=postgresql` is specified in this file https://github.com/streamnative/charts/blob/master/charts/pulsar/templates/pulsar-manager/pulsar-manage

[GitHub] [pulsar-manager] tuteng commented on issue #299: Unable to get anything about a partition: TypeError: Cannot read property 'status' of undefined

2020-05-21 Thread GitBox
tuteng commented on issue #299: URL: https://github.com/apache/pulsar-manager/issues/299#issuecomment-632118685 I am very sorry for the late reply. I think this error is caused by using the wrong paging plug-in, I will look at this problem, but at present, you can try to use this YAML h

Build failed in Jenkins: pulsar_release_nightly_snapshot #820

2020-05-21 Thread Apache Jenkins Server
See Changes: [github] Add Docs for c# client (#7003) -- [...truncated 7.98 MB...] 2020-05-21\T\13:38:14.896 [INFO] Nothing to compile - all classes are up t

Re: [Discuss] Disable "Require branches to be up-to-date before merging" for github

2020-05-21 Thread Jia Zhai
Would like to do a quick summary for this discussion: 3 binding +1 votes, 0 -1 votes. 3 +1 bindings are from: PengHui Li Sijie Guo Jia Zhai On Thu, May 21, 2020 at 6:04 PM Jia Zhai wrote: > Thanks. Add my own +1. > > Best Regards. > > > Jia Zhai > > Beijing, China > > Mobile: +86 15810491983

Build failed in Jenkins: pulsar-website-build #1002

2020-05-21 Thread Apache Jenkins Server
See Changes: [github] [release][website] change version order for 2.5.2 (#6999) -- [...truncated 5.45 MB...] [WARNING] /pulsar/pulsar-client-tools-test/src/test/java

Re: [Discuss] Disable "Require branches to be up-to-date before merging" for github

2020-05-21 Thread Jia Zhai
Thanks. Add my own +1. Best Regards. Jia Zhai Beijing, China Mobile: +86 15810491983 On Thu, May 21, 2020 at 5:46 PM PengHui Li wrote: > +1 > On May 21, 2020, 3:31 PM +0800, Sijie Guo , wrote: > > +1 > > > > On Wed, May 20, 2020 at 11:51 PM Jia Zhai wrote: > > > > > Hi Guys, > > > > > >

Re: [Discuss] Disable "Require branches to be up-to-date before merging" for github

2020-05-21 Thread PengHui Li
+1 On May 21, 2020, 3:31 PM +0800, Sijie Guo , wrote: > +1 > > On Wed, May 20, 2020 at 11:51 PM Jia Zhai wrote: > > > Hi Guys, > > > > Recently we enabled feature "Require branches to be up-to-date before > > merging" in github by help from ASF infra team, as this has been discussed > > several ti

Slack digest for #dev - 2020-05-21

2020-05-21 Thread Apache Pulsar Slack
2020-05-20 09:22:49 UTC - jia zhai: ```webSocketSessionIdleTimeoutMillis``` You need this config above, try set and add it into your conf/broker.conf, or conf/websocket.conf 2020-05-20 09:23:25 UTC - jia zhai: opened an issue to ASF infa f

[GitHub] [pulsar-manager] Anonymitaet commented on issue #221: Use environment variables

2020-05-21 Thread GitBox
Anonymitaet commented on issue #221: URL: https://github.com/apache/pulsar-manager/issues/221#issuecomment-631950979 @KannarFr any update for this doc issue? This is an automated message from the Apache Git Service. To respon

Re: [Discuss] Disable "Require branches to be up-to-date before merging" for github

2020-05-21 Thread Sijie Guo
+1 On Wed, May 20, 2020 at 11:51 PM Jia Zhai wrote: > Hi Guys, > > Recently we enabled feature "Require branches to be up-to-date before > merging" in github by help from ASF infra team, as this has been discussed > several times to protect merge conflict. > > But seems it is not worked as expec