[GitHub] [pulsar-dotpulsar] blankensteiner commented on pull request #110: modified the "hello world" to fix first run issue

2022-09-29 Thread GitBox
blankensteiner commented on PR #110: URL: https://github.com/apache/pulsar-dotpulsar/pull/110#issuecomment-1263140651 Hi @aintJoshinya Thanks for the PR! I agree with the minimal optional configuration, but setting the initial position is the right solution and one I find people missi

[GitHub] [pulsar] Anonymitaet added a comment to the discussion: [Design] Pulsar All Releases Page

2022-09-29 Thread GitBox
GitHub user Anonymitaet added a comment to the discussion: [Design] Pulsar All Releases Page Hi @eolivelli, We have an issue with 2.9.0 and 2.9.1 and find that you're the release manager for them. Could you please give us a hand? Thank you! 🔹🔹🔹 ### Context We (@urfreespace, @SignorMercur

[GitHub] [pulsar] raunakagrawal47 added a comment to the discussion: Need max redelivery count at message level.

2022-09-29 Thread GitBox
GitHub user raunakagrawal47 added a comment to the discussion: Need max redelivery count at message level. We are using broker (server) version `2.8` and pulsar-client version on the java application `2.10.0 ` GitHub link: https://github.com/apache/pulsar/discussions/17626#discussioncomment-

Re: [DISCUSS] Use AUTO_CONSUME schema with Avro logical type conversion problem

2022-09-29 Thread Enrico Olivelli
Bo, Il giorno lun 26 set 2022 alle ore 15:39 丛搏 ha scritto: > > Hello, Pulsar community: > > Now, we use Avro Schema with Logical type, and the client > automatically adds logical type conversion. You can see PR > https://github.com/apache/pulsar/pull/6704. > > But AUTO_CONSUME Schema with Avro

Re: [VOTE] Pulsar Client Go Release 0.9.0 Candidate 2

2022-09-29 Thread PengHui Li
+1 (binding) - build from the source code - test the produce and consume Penghui On Thu, Sep 29, 2022 at 2:43 PM Guangning E wrote: > +1(non-binding) > - Check sha 512 value for source code > - Check go build producer example > > Thanks, > Guangning > > r...@apache.org 于2022年9月29日周四 14:35写道:

Re: [DISCUSS] Changed pulsar-go-client minimum Go SDK version from 1.15 to 1.16

2022-09-29 Thread Paul Gier
+1 Passing the git ID and other version information via ldflags would be better in my opinion than embedding a file. It can be automated in the Makefile, and the values can go directly into a variable, so you don't have to do any string parsing. The feature has been around a long time, so we woul

Re: [DISCUSS] Changed pulsar-go-client minimum Go SDK version from 1.15 to 1.16

2022-09-29 Thread Ming Luo
Have you thought about using ldflags to pass git commit or git tag which can be the actual version. This will tie into a build system automatically without additional files. We don't have to maintain the version information in git and a separate file. Here is an example. https://stackoverflow.com/q

Re: Pulsar CI slowness continues

2022-09-29 Thread Enrico Olivelli
Il giorno mer 28 set 2022 alle ore 07:50 Lari Hotari ha scritto: > > There was an incident with GitHub Actions yesterday, > https://www.githubstatus.com/incidents/nbhb2lxyh9bb . > > It looks like Pulsar CI is back in action today. > > I finally got a helpful response from GitHub Support. They are

Re: Master branch is broken due to #15914

2022-09-29 Thread Lari Hotari
> I think that we should be able (committers, or at least PMC members) > to push changes without passing for CI. Otherwise it may be sometimes > impossible to fix or revert. I guess that in this case, we don't need to by-pass CI. If we ever have to do that, it is possible by first editing .asf.yam

[GitHub] [pulsar] Jason918 added a comment to the discussion: Need max redelivery count at message level.

2022-09-29 Thread GitBox
GitHub user Jason918 added a comment to the discussion: Need max redelivery count at message level. This issue is not verified on the latest pulsar version. Which pulsar version are you using? GitHub link: https://github.com/apache/pulsar/discussions/17626#discussioncomment-3761863 Thi

Master branch is broken due to #15914

2022-09-29 Thread Enrico Olivelli
Hello, I have to revert this patch https://github.com/apache/pulsar/pull/15914 because now many tests fail This is the revert PR https://github.com/apache/pulsar/pull/17889 I think that we should be able (committers, or at least PMC members) to push changes without passing for CI. Otherwise it m

[GitHub] [pulsar] raunakagrawal47 added a comment to the discussion: Need max redelivery count at message level.

2022-09-29 Thread GitBox
GitHub user raunakagrawal47 added a comment to the discussion: Need max redelivery count at message level. @Jason918 @michaeljmarshall facing below issue: https://github.com/apache/pulsar/issues/12021 I am trying to produce 10 messages at a time. And all these 10 messages should be getting r

Re: [VOTE] Pulsar Release 2.10.2 Candidate 1

2022-09-29 Thread houxiaoyu
+1 (non-binding) - Checksum and signatures - Build on Mac, using JDK17 - Run Pulsar standalone and produce/consume case Thanks, Xiaoyu Hou Enrico Olivelli 于2022年9月29日周四 16:31写道: > +1 (binding) > - Apache RAT > - Signatures/Digests > - Built on Mac, using JDK 11 Temurin > - Verified LICENSE fil

Re: [VOTE] Pulsar Release 2.10.2 Candidate 1

2022-09-29 Thread Enrico Olivelli
+1 (binding) - Apache RAT - Signatures/Digests - Built on Mac, using JDK 11 Temurin - Verified LICENSE files (using the script), both on the staged tarball and on the tarbal produced by building from source - Did a couple of smoke tests on Pulsar standalone, from the binary package Thanks for driv

Re: [VOTE] Pulsar Release 2.10.2 Candidate 1

2022-09-29 Thread r...@apache.org
+1 (non-binding) - Checksum and signatures - Run Go SDK produce/consume - Run Pulsar standalone and produce/consume case -- Thanks Xiaolong Ran guo jiwei 于2022年9月29日周四 11:30写道: > +1 (binding) > > - Checksum and signatures > - Apache Rat check passes > - Compile from source with JDK11 > - Valid