[GitHub] [pulsar-client-node] nkurihar merged pull request #147: Bump the master version to 1.4.0

2021-03-16 Thread GitBox
nkurihar merged pull request #147: URL: https://github.com/apache/pulsar-client-node/pull/147 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

Re: Problems with Avro upgrade from 1.9 to 1.10

2021-03-16 Thread Enrico Olivelli
Sijie, Thanks for your explanation. Now it is clear to me that it won't be a problem for users. I will fix all of the tests on my PR Enrico Il Mar 16 Mar 2021, 19:15 Sijie Guo ha scritto: > I would be surprised that it would break compatibility just because it > writes the schema definition

Re: Problems with Avro upgrade from 1.9 to 1.10

2021-03-16 Thread Sijie Guo
I would be surprised that it would break compatibility just because it writes the schema definition in a different order. The test validating how schema definition was written should be improved. I don't see that breaks the compatibility. The test case you have is problematic. If you want to veri

Re: Problems with Avro upgrade from 1.9 to 1.10

2021-03-16 Thread Enrico Olivelli
Matteo, actually it breaks @Test public void helloCompatibility() throws Exception { // sampled data created using Pulsar with Avro 1.9 String schemaAvro19 = "{\"type\":\"record\",\"name\":\"MyPojo\",\"namespace\":\"SenderTest\",\"fields\":[{\"name\":\"foo\",\"type\":[\"null\",\"string\"],\"

Pulsar Community Weekly Update] 2021-03-08 ~ 2021-03-14

2021-03-16 Thread Huanli Meng
Dear Pulsar enthusiast, This is the Pulsar community weekly update for 2021-03-08 ~ 2021-03-14, with updates on Pulsar client, broker, and so on. This Pulsar community weekly update is also available at https://streamnative.io/weekly/2021/2021-03/2021-03-15-pulsar-weekly. *All Pulsar community w

Re: Problems with Avro upgrade from 1.9 to 1.10

2021-03-16 Thread Matteo Merli
A good test to add would be to include the object serialized by avro 1.9 and ensure we can read it with avro 1.10. On Tue, Mar 16, 2021 at 8:13 AM Enrico Olivelli wrote: > Sijie. > let me write an example. > > if you have this Pojo class > class Pojo { > String foo; > String bar; > } >

Pulsar Flaky Test report 2021-03-16

2021-03-16 Thread Lari Hotari
Dear Pulsar community members, Here's a report of the flaky tests in Pulsar CI during the observation period of 2021-02-23 - 2021-03-16 https://docs.google.com/spreadsheets/d/106QKMFfdLa-VSFGlggs35eMw-ZnjMUjMhA3KVBjzHLI/edit?usp=sharing subset of the report, 20 most flaky tests (some are already

Re: Problems with Avro upgrade from 1.9 to 1.10

2021-03-16 Thread Enrico Olivelli
Sijie. let me write an example. if you have this Pojo class class Pojo { String foo; String bar; } if you use Schema.AVRO(Pojo.class) with Avro 1.9 you will probably see a Schema with the fields in this order: foo, bar with Avro 1.10 you will see exactly the schema with bar, foo. (in alp

[ANNOUNCE] Introducing Apache Pulsar Hackathon 2021!!!

2021-03-16 Thread Sijie Guo
Hi, Pulsar and BookKeeper communities, The adoption of Apache Pulsar and BookKeeper is accelerating as organizations around the world pursue cloud-native technologies. To build on this momentum, we are excited to announce the first-ever Apache Pulsar Hackathon 2021, *taking place on May 6-7th*. Th

[GitHub] [pulsar-dotpulsar] blankensteiner commented on issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
blankensteiner commented on issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69#issuecomment-800181546 Cool, thanks for requesting and testing it! :-) I'll make this part of the next release (later this week I hope). In regards to multi-URLs: Yes, that would be han

[GitHub] [pulsar-dotpulsar] blankensteiner closed issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
blankensteiner closed issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69 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

[GitHub] [pulsar-dotpulsar] darkwatchuk commented on issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
darkwatchuk commented on issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69#issuecomment-800167858 Hi, the listenerName seems to work as expected, I can see positive and negative response messages coming back depending on the configuration so that all looks good to me th

[GitHub] [pulsar-dotpulsar] blankensteiner commented on issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
blankensteiner commented on issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69#issuecomment-800147046 Great. I have just committed support for setting the listener name. Let me know if that work as expected :-) In regards to multiple URLs. Well... in my experience, this

[GitHub] [pulsar-dotpulsar] darkwatchuk commented on issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
darkwatchuk commented on issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69#issuecomment-800142548 Yes, should be able to. Is it also possible to set multiple endpoints for the service url for availability purposes? Again, other clients support a comma separated s

[GitHub] [pulsar-dotpulsar] blankensteiner commented on issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
blankensteiner commented on issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69#issuecomment-800109646 Thanks! :-) If it's just a matter of allowing the user to set the listener name on the client and set it (AdvertisedListenerName) on the topic lookup command, then I c

[GitHub] [pulsar-dotpulsar] darkwatchuk commented on issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
darkwatchuk commented on issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69#issuecomment-800102657 Yep, sorry. Do these help? https://pulsar.apache.org/docs/en/concepts-multiple-advertised-listeners/ https://github.com/apache/pulsar/blob/master/pulsar-

[GitHub] [pulsar-dotpulsar] blankensteiner commented on issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
blankensteiner commented on issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69#issuecomment-800097110 Hi @darkwatchuk Do you have a link for the wanted feature? This is an automated message from the Apac

[GitHub] [pulsar-dotpulsar] darkwatchuk opened a new issue #69: How to set listenerName for PulsarClient

2021-03-16 Thread GitBox
darkwatchuk opened a new issue #69: URL: https://github.com/apache/pulsar-dotpulsar/issues/69 Hi, The Java version of the client and other .Net clients allow the ability to set the listenerName. Can this be supported on here too? Many thanks --

[GitHub] [pulsar-helm-chart] jeantil commented on issue #85: Change Pulsar Manager docker image version

2021-03-16 Thread GitBox
jeantil commented on issue #85: URL: https://github.com/apache/pulsar-helm-chart/issues/85#issuecomment-800071262 I found this issue while looking into #31 and #108 Since the tag version is in values.yml you can use `--set images.pulsar_manager.tag=v0.2.0` to upgrade the version li

Re: [Discuss] draft PIP for "Changes to GitHub Actions based Pulsar CI"

2021-03-16 Thread Lari Hotari
On Tue, Mar 16, 2021 at 3:11 AM Sijie Guo wrote: > > The prototype has demonstrated about 60% reduction in > resource consumption. > > It is hard to quantify. Merging them into one large workflow can result in > more failures. Re-running those failures can consume resources as well. > Yes, you a

[GitHub] [pulsar-helm-chart] jeantil commented on issue #92: Helm chart repo not tracking latest commits

2021-03-16 Thread GitBox
jeantil commented on issue #92: URL: https://github.com/apache/pulsar-helm-chart/issues/92#issuecomment-800049506 @nickelozz a new version has just been released which includes the requested commits (it also upgrades puslar to 2.7.1) --

[GitHub] [pulsar-helm-chart] jeantil commented on issue #108: Pulsar manager always uses default user/password

2021-03-16 Thread GitBox
jeantil commented on issue #108: URL: https://github.com/apache/pulsar-helm-chart/issues/108#issuecomment-800047276 hi @sloan-dog , this report looks a lot like #31 to me :) This is an automated message from the Apache Git

[GitHub] [pulsar-helm-chart] jeantil commented on pull request #107: makes cert-manager apiVersion configurable

2021-03-16 Thread GitBox
jeantil commented on pull request #107: URL: https://github.com/apache/pulsar-helm-chart/pull/107#issuecomment-800036245 @sijie thank you ! This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar-helm-chart] sijie merged pull request #107: makes cert-manager apiVersion configurable

2021-03-16 Thread GitBox
sijie merged pull request #107: URL: https://github.com/apache/pulsar-helm-chart/pull/107 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

[GitHub] [pulsar-helm-chart] sijie closed issue #68: Cert-manager resources api-version is hardcoded

2021-03-16 Thread GitBox
sijie closed issue #68: URL: https://github.com/apache/pulsar-helm-chart/issues/68 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 t

[GitHub] [pulsar-helm-chart] sijie merged pull request #109: Bump pulsar version to 2.7.1

2021-03-16 Thread GitBox
sijie merged pull request #109: URL: https://github.com/apache/pulsar-helm-chart/pull/109 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