Re: Updates on PIP-62

2020-11-12 Thread Sijie Guo
I have cloned the changes to pulsar-connectors, pulsar-adapters, and pulsar-presto. They can be built independently. However, there is one challenge that we release the `pulsar-all` image to include all connectors, offloaders, and presto. And we also use the image for integration tests. It is hard

[GitHub] [pulsar-helm-chart] sijie commented on pull request #69: Allow use of existing secret for pulsar manager credentials

2020-11-12 Thread GitBox
sijie commented on pull request #69: URL: https://github.com/apache/pulsar-helm-chart/pull/69#issuecomment-726488182 @pinkavaj Can you rebase your pull request to the latest master? This is an automated message from the Apach

[GitHub] [pulsar-helm-chart] sijie commented on pull request #73: Updating pods on configmap change

2020-11-12 Thread GitBox
sijie commented on pull request #73: URL: https://github.com/apache/pulsar-helm-chart/pull/73#issuecomment-726487830 @milos-matijasevic You can rebase your pull request to the latest master and push to your branch. It then will trigger the CI to run again.

[GitHub] [pulsar-helm-chart] sijie commented on pull request #74: Allow appending dnsNames to self-signed certs

2020-11-12 Thread GitBox
sijie commented on pull request #74: URL: https://github.com/apache/pulsar-helm-chart/pull/74#issuecomment-726487539 @volgorean Can you rebase your change to the latest master? This is an automated message from the Apache Git

[GitHub] [pulsar-helm-chart] sijie commented on pull request #80: Use `.Release.Namespace` by default to handle namespaces

2020-11-12 Thread GitBox
sijie commented on pull request #80: URL: https://github.com/apache/pulsar-helm-chart/pull/80#issuecomment-726487226 @jeantil Can you rebase this pull request to the latest master? This is an automated message from the Apache

[GitHub] [pulsar-helm-chart] sijie commented on pull request #77: Bumping version

2020-11-12 Thread GitBox
sijie commented on pull request #77: URL: https://github.com/apache/pulsar-helm-chart/pull/77#issuecomment-726487009 @naveen1100 the CI failure is due to #76. We have fixed it. We also released a new version in #81 This is

[GitHub] [pulsar-helm-chart] sijie closed pull request #77: Bumping version

2020-11-12 Thread GitBox
sijie closed pull request #77: URL: https://github.com/apache/pulsar-helm-chart/pull/77 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

[GitHub] [pulsar-helm-chart] sijie merged pull request #75: Local mode for kubernetes object generators

2020-11-12 Thread GitBox
sijie merged pull request #75: URL: https://github.com/apache/pulsar-helm-chart/pull/75 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

[GitHub] [pulsar-helm-chart] sijie merged pull request #81: Bump the image version to 2.6.2

2020-11-12 Thread GitBox
sijie merged pull request #81: URL: https://github.com/apache/pulsar-helm-chart/pull/81 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

[GitHub] [pulsar-helm-chart] wolfstudy merged pull request #83: Upgrade chart-testing-action to 2.0.0

2020-11-12 Thread GitBox
wolfstudy merged pull request #83: URL: https://github.com/apache/pulsar-helm-chart/pull/83 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 t

[GitHub] [pulsar-client-node] coryvirok opened a new issue #133: How to clean up consumers/producers on SIGINT

2020-11-12 Thread GitBox
coryvirok opened a new issue #133: URL: https://github.com/apache/pulsar-client-node/issues/133 First off, thanks for Pulsar and all of these client libs! I'm excited to vet Pulsar for my company and see if it solves some of Kafka's shortcomings. I have a gRPC service that imple

[GitHub] [pulsar-helm-chart] devinbost commented on issue #84: TLS Authentication in Kubernetes, Pulsar 2.6.1 - Broker crash loop on startup due to 401 in WorkerService.start(..)

2020-11-12 Thread GitBox
devinbost commented on issue #84: URL: https://github.com/apache/pulsar-helm-chart/issues/84#issuecomment-726350229 After I swapped out the brokerClient auth to use token auth instead of TLS auth in the broker-configmap.yaml and proxy-configmap.yaml files, the cluster started just fine. S

[GitHub] [pulsar-helm-chart] devinbost opened a new issue #84: TLS Authentication in Kubernetes, Pulsar 2.6.1 - Broker crash loop on startup due to 401 in WorkerService.start(..)

2020-11-12 Thread GitBox
devinbost opened a new issue #84: URL: https://github.com/apache/pulsar-helm-chart/issues/84 Copying from the Apache/Pulsar Github issue (https://github.com/apache/pulsar/issues/8536): **Describe the bug** After configuring TLS Authentication in Pulsar 2.6.1 with this helm chart:

Re: Proposal for Consumer Filtering in Pulsar brokers

2020-11-12 Thread Sijie Guo
Hi Andre, I didn't see the attached writeup. Can you write a PIP for this feature? Given it is a big feature, it would be good to discuss it through a PIP. - Sijie On Thu, Nov 12, 2020 at 6:17 AM Kramer, Andre wrote: > Hello everyone, > > > > We at Software AG have prototyped adding filtering

[GitHub] [pulsar-helm-chart] milos-matijasevic removed a comment on pull request #73: Updating pods on configmap change

2020-11-12 Thread GitBox
milos-matijasevic removed a comment on pull request #73: URL: https://github.com/apache/pulsar-helm-chart/pull/73#issuecomment-726184646 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service.

[GitHub] [pulsar-helm-chart] milos-matijasevic commented on pull request #73: Updating pods on configmap change

2020-11-12 Thread GitBox
milos-matijasevic commented on pull request #73: URL: https://github.com/apache/pulsar-helm-chart/pull/73#issuecomment-726184646 /pulsarbot run-failure-checks This is an automated message from the Apache Git Service. To respo

[GitHub] [pulsar-helm-chart] milos-matijasevic commented on pull request #73: Updating pods on configmap change

2020-11-12 Thread GitBox
milos-matijasevic commented on pull request #73: URL: https://github.com/apache/pulsar-helm-chart/pull/73#issuecomment-726168894 hello, @sijie after #76 is merged i want to rerun CI tests, is there way to do it ? This is an

[Pulsar story]: Powering Federated Learning at Tencent with Apache Pulsar

2020-11-12 Thread Jinfeng Huang
Dear Pulsar community, We're excited to know that *Tencent adopted Apache Pulsar * as the data synchronization solution in *Federated Learning(FL)*. And now they'd love to share the story

[GitHub] [pulsar-helm-chart] wolfstudy commented on pull request #80: Use `.Release.Namespace` by default to handle namespaces

2020-11-12 Thread GitBox
wolfstudy commented on pull request #80: URL: https://github.com/apache/pulsar-helm-chart/pull/80#issuecomment-726104988 @jeantil After https://github.com/apache/pulsar-helm-chart/pull/83 is merged, please merge the latest master code to ensure that ci passes. Thanks.

[GitHub] [pulsar-helm-chart] wolfstudy edited a comment on pull request #83: Upgrade chart-testing-action to 2.0.0

2020-11-12 Thread GitBox
wolfstudy edited a comment on pull request #83: URL: https://github.com/apache/pulsar-helm-chart/pull/83#issuecomment-726103521 Through the verification of https://github.com/apache/pulsar-helm-chart/pull/81, it seems feasible to upgrade the version of chart-testing-action. @sijie

[GitHub] [pulsar-helm-chart] wolfstudy commented on pull request #83: Upgrade chart-testing-action to 2.0.0

2020-11-12 Thread GitBox
wolfstudy commented on pull request #83: URL: https://github.com/apache/pulsar-helm-chart/pull/83#issuecomment-726103521 Through the verification of https://github.com/apache/pulsar-helm-chart/pull/81, it seems feasible to upgrade the version of chart-testing-action -

[GitHub] [pulsar-helm-chart] wolfstudy opened a new pull request #83: Upgrade chart-testing-action to 2.0.0

2020-11-12 Thread GitBox
wolfstudy opened a new pull request #83: URL: https://github.com/apache/pulsar-helm-chart/pull/83 Signed-off-by: xiaolong.ran ### Motivation The lint ci error as follows: ``` Linting chart 'pulsar => (version: "2.6.2-1", path: "charts/pulsar")' Checking chart 'pul

Proposal for Consumer Filtering in Pulsar brokers

2020-11-12 Thread Kramer, Andre
Hello everyone, We at Software AG have prototyped adding filtering on Consumer subscriptions in the Pulsar broker and are submitting our changes for consideration under Apache 2.0 license. Please see pull request [Consumer Filtering #8544 https://github.com/apache/pulsar/pull/8544] and attached

[GitHub] [pulsar-helm-chart] wolfstudy commented on pull request #81: Bump the image version to 2.6.2

2020-11-12 Thread GitBox
wolfstudy commented on pull request #81: URL: https://github.com/apache/pulsar-helm-chart/pull/81#issuecomment-726064732 > @wolfstudy Can you rebase the pull request to latest master to include the fix in #76 ? Sure, will process it.

[GitHub] [pulsar-helm-chart] Imunhatep commented on issue #66: Helm charts are managing namespace

2020-11-12 Thread GitBox
Imunhatep commented on issue #66: URL: https://github.com/apache/pulsar-helm-chart/issues/66#issuecomment-726040127 Hey, thank you for follow up. Here I see chart is managing NS and in fact will try to create/delete it: [https://github.com/apache/pulsar-helm-chart/blob/master/charts

[GitHub] [pulsar-connectors] sijie opened a new pull request #2: Move connectors to the top structure

2020-11-12 Thread GitBox
sijie opened a new pull request #2: URL: https://github.com/apache/pulsar-connectors/pull/2 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 t

[GitHub] [pulsar-presto] jiazhai merged pull request #2: Add license header file and fix checkstyle

2020-11-12 Thread GitBox
jiazhai merged pull request #2: URL: https://github.com/apache/pulsar-presto/pull/2 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 to

[GitHub] [pulsar-presto] sijie opened a new pull request #2: Add license header file and fix checkstyle

2020-11-12 Thread GitBox
sijie opened a new pull request #2: URL: https://github.com/apache/pulsar-presto/pull/2 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

[GitHub] [pulsar-helm-chart] jeantil commented on a change in pull request #80: Use `.Release.Namespace` by default to handle namespaces

2020-11-12 Thread GitBox
jeantil commented on a change in pull request #80: URL: https://github.com/apache/pulsar-helm-chart/pull/80#discussion_r521904247 ## File path: charts/pulsar/Chart.yaml ## @@ -21,11 +21,11 @@ apiVersion: v1 appVersion: "2.6.1" description: Apache Pulsar Helm chart for Kuberne

[ANNOUNCE] Apache Pulsar 2.6.2 released

2020-11-12 Thread xiaolong ran
The Apache Pulsar team is proud to announce Apache Pulsar version 2.6.2. Pulsar is a highly scalable, low latency messaging platform running on commodity hardware. It provides simple pub-sub semantics over topics, guaranteed at-least-once delivery of messages, automatic cursor management for subsc