[GitHub] [pulsar-client-node] Aries-hash opened a new issue, #225: Can't serialize message of type "pulsar.proto.MessageMetadata" because it is missing required fields: (cannot determine missing field

2022-07-11 Thread GitBox
Aries-hash opened a new issue, #225: URL: https://github.com/apache/pulsar-client-node/issues/225 nodejs 16 sdk version 1.6.2 cluster version 2.7 When I restart the producer, I get the following error: 2022-07-12 12:54:07.697 INFO [140305944786688] HTTPLookupService:295

Re: CI - remove cmd line test retries?

2022-07-11 Thread Michael Marshall
+1 I support removing the retries for the many reasons already mentioned. - Michael On Tue, Jul 12, 2022 at 12:35 AM Matteo Merli wrote: > > +1 > > The retry that repeats the whole maven job is often hiding the real > test failures that were in the 1st failed job. > -- > Matteo Merli > > > On M

Re: CI - remove cmd line test retries?

2022-07-11 Thread Matteo Merli
+1 The retry that repeats the whole maven job is often hiding the real test failures that were in the 1st failed job. -- Matteo Merli On Mon, Jul 11, 2022 at 9:06 PM Anon Hxy wrote: > > Hi Boschi > > I support this plan and I think we need take some effort to make the Pulsar > CI more stable. >

Re: CI - remove cmd line test retries?

2022-07-11 Thread Anon Hxy
Hi Boschi I support this plan and I think we need take some effort to make the Pulsar CI more stable. Thanks, Xiaoyu Hou Nicolò Boschi 于2022年7月11日周一 22:14写道: > Hi all, > > I'd like to start a discussion about the stability of Pulsar CI. > > It is common that some tests suite in our CI times ou

RE: [VOTE] [PIP-179] Support the admin API to check unknown request parameters

2022-07-11 Thread Zixuan Liu
+1 (non-binding) On 2022/06/29 16:40:42 Yubiao Feng wrote: > Hi Pulsar Community > > I would like to start a VOTE on "Support the admin API to check unknown > request parameters" (PIP-179). > > The proposal can be read at https://github.com/apache/pulsar/issues/16135 > > and the discussion thre

Re: CI - remove cmd line test retries?

2022-07-11 Thread Zixuan Liu
Hi Boschi, That sounds good that removing retry reduces testing time and helps us find problems in testing quickly. Thanks, Zixuan Nicolò Boschi 于2022年7月11日周一 22:14写道: > Hi all, > > I'd like to start a discussion about the stability of Pulsar CI. > > It is common that some tests suite in our C

[GitHub] [pulsar-client-node] k2la merged pull request #219: Add null check for consumer in MessageListenerProxy

2022-07-11 Thread GitBox
k2la merged PR #219: URL: https://github.com/apache/pulsar-client-node/pull/219 -- 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 the specific comment. To unsubscribe, e-mail: dev-unsubscr...@pulsar.a

Re: [VOTE] [PIP-182] Provide new load balance placement strategy implementation for ModularLoadManagerStrategy

2022-07-11 Thread Lin Lin
+1 binding Thanks, Lin Lin

CI - remove cmd line test retries?

2022-07-11 Thread Nicolò Boschi
Hi all, I'd like to start a discussion about the stability of Pulsar CI. It is common that some tests suite in our CI times out. This is because when a test fails the entire suite is retried from the beginning (max 3 times). (example: https://github.com/apache/pulsar/runs/7281063499?check_suite_f

Re: [DISCUSS] PIP-180: Shadow Topic, an alternative way to support readonly topic ownership.

2022-07-11 Thread Asaf Mesika
On Thu, Jun 23, 2022 at 6:26 AM Haiting Jiang wrote: > Hi Asaf, > > > I did a quick reading and I couldn't understand the gist of this change: > > The shadow topic doesn't really have it's own messages, or it's own > ledgers > > right? When it reads messages, it reads from the original topic ledg

Re: How to Show Info about Community?

2022-07-11 Thread Yu
Hi team, Now we have a fresh "Community" page [1], which shows the Pulsar PMC list, contributors over time, monthly active contributors, etc. Thanks Wenqing for making this happen! [1] https://pulsar.apache.org/community#section-community Yu On Tue, Jul 5, 2022 at 9:15 PM Yu wrote: > Hi tiso

Re: [DISCUSS] PIP-184: Topic specific consumer priorityLevel

2022-07-11 Thread Zike Yang
Hi, Dave Thanks for your proposals. Overall looks good to me. Just a minor comment: What about using the Pattern type to store the compiled Pattern in the TopicConsumerConfigurationData? Like below: ``` public class TopicConsumerConfigurationData implements Serializable { private static final

Re: [DISCUSS] PIP-181: Pulsar Shell

2022-07-11 Thread Nicolò Boschi
Hi Asaf, Actually it's in the "Future works" of the PIP. I already implemented a PoC of it and you can see it here: https://github.com/nicoloboschi/pulsar/commit/f227be324c96daeaf89566260a63eb558357f34f There are still some issues with some runtime config since GraalVM requires to declare most of

Re: [DISCUSS] PIP-181: Pulsar Shell

2022-07-11 Thread tison
Hi Asaf, Why not compile the related package with graal and submit a report on the master issue[1] :) AFAIK Pulsar didn't optimize for graal compilation so that you MAY meet some issues, and pulsar-shell actually depends on other core module so that it's not an orphan issue to resolve. Best, tis

Re: [DISCUSS] PIP-181: Pulsar Shell

2022-07-11 Thread Asaf Mesika
Looks awesome. I was wondering if you have thought about another direction which is supplying a GraalVM compiled binary. This should take care of load time. So in effect, you can have a single command, which boots in microseconds, and has shell auto-complete? On Mon, Jul 4, 2022 at 4:34 PM Hang Ch

[GitHub] [pulsar-site] Anonymitaet merged pull request #132: Fix inaccurate community info

2022-07-11 Thread GitBox
Anonymitaet merged PR #132: URL: https://github.com/apache/pulsar-site/pull/132 -- 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 the specific comment. To unsubscribe, e-mail: dev-unsubscr...@pulsar.a

[GitHub] [pulsar-site] urfreespace commented on pull request #132: Fix inaccurate community info

2022-07-11 Thread GitBox
urfreespace commented on PR #132: URL: https://github.com/apache/pulsar-site/pull/132#issuecomment-1180081984 > @urfreespace could you please review this PR from the technical perspective? Thank you! LGTM -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [pulsar-site] Anonymitaet commented on pull request #132: Fix inaccurate community info

2022-07-11 Thread GitBox
Anonymitaet commented on PR #132: URL: https://github.com/apache/pulsar-site/pull/132#issuecomment-1180079703 @urfreespace could you please review this PR from the technical perspective? Thank you! -- This is an automated message from the Apache Git Service. To respond to the message,