[GitHub] [pulsar-site] michaeljmarshall commented on pull request #138: Only replace correctly formed endpoints

2022-07-05 Thread GitBox
michaeljmarshall commented on PR #138: URL: https://github.com/apache/pulsar-site/pull/138#issuecomment-1175837630 Note that in the above comment, the replace script tries to create this link: https://pulsar.apache.org/api/client/2.1.0-incubating/org/apache/pulsar/client/api/ConsumerConfigu

[GitHub] [pulsar-site] michaeljmarshall commented on pull request #138: Only replace correctly formed endpoints

2022-07-05 Thread GitBox
michaeljmarshall commented on PR #138: URL: https://github.com/apache/pulsar-site/pull/138#issuecomment-1175833645 Oops, I included a couple of extra commits in there, I'll overwrite that in just a minute by pushing with force. -- This is an automated message from the Apache Git Service.

[GitHub] [pulsar-site] michaeljmarshall opened a new pull request, #138: Only replace correctly formed endpoints

2022-07-05 Thread GitBox
michaeljmarshall opened a new pull request, #138: URL: https://github.com/apache/pulsar-site/pull/138 The current replace.js script leads to malformed output because it converts proper links like `http://pulsar.apache.org/api/client/org/apache/pulsar/client/api/ConsumerConfiguration.html#se

[GitHub] [pulsar-site] michaeljmarshall commented on pull request #133: Only generate Javadocs for released versions of Pulsar

2022-07-05 Thread GitBox
michaeljmarshall commented on PR #133: URL: https://github.com/apache/pulsar-site/pull/133#issuecomment-1175786907 Note: no docs reference `pulsar.apache.org/api/admin/org/apache/pulsar/client/admin/internal/`, but plenty reference `pulsar.apache.org/api/admin/org/apache/pulsar/client/admin

[GitHub] [pulsar-site] michaeljmarshall commented on pull request #133: Only generate Javadocs for released versions of Pulsar

2022-07-05 Thread GitBox
michaeljmarshall commented on PR #133: URL: https://github.com/apache/pulsar-site/pull/133#issuecomment-1175784198 I just discovered that the Pulsar Admin API Javadocs changed their generation mechanism starting with 2.8.0 because of this change: https://github.com/apache/pulsar/pull/9246.

[GitHub] [pulsar-site] michaeljmarshall commented on pull request #133: Only generate Javadocs for released versions of Pulsar

2022-07-05 Thread GitBox
michaeljmarshall commented on PR #133: URL: https://github.com/apache/pulsar-site/pull/133#issuecomment-1175745654 I updated the PR to remove the `pulsar-broker` doc generation. I started a discussion on the dev@ mailing list to propose removing these docs from the website permanently. --

Re: [DISCUSS] Move all generated documentation to apache/pulsar-site repo

2022-07-05 Thread Michael Marshall
Thanks for the feedback, and thank you for sharing those docs Yu. I will probably move forward on this task next week. First, I plan to fix the doc generation in the pulsar-site repo. Thanks, Michael On Fri, Jul 1, 2022 at 4:19 AM Yu wrote: > > Hi Michael, > Thanks for your awesome idea! > You

[DISCUSS] Remove Pulsar Broker Javadocs from Website

2022-07-05 Thread Michael Marshall
Hi Pulsar Community, We currently host the generated Pulsar Broker Javadocs on our website. You can find them here [0]. I cannot find any references to these docs in the actual pulsar documentation, which indicates to me that these are not used. (Searching for `api/pulsar-broker` returns nothing.

[GitHub] [pulsar-site] SignorMercurio commented on pull request #137: Highlight toc when scrolling

2022-07-05 Thread GitBox
SignorMercurio commented on PR #137: URL: https://github.com/apache/pulsar-site/pull/137#issuecomment-1175740427 @Anonymitaet PTAL -- 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 com

[GitHub] [pulsar-site] SignorMercurio opened a new pull request, #137: Highlight toc when scrolling

2022-07-05 Thread GitBox
SignorMercurio opened a new pull request, #137: URL: https://github.com/apache/pulsar-site/pull/137 Fix apache/pulsar#16174. -- 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.

[GitHub] [pulsar-site] michaeljmarshall commented on pull request #133: Only generate Javadocs for released versions of Pulsar

2022-07-05 Thread GitBox
michaeljmarshall commented on PR #133: URL: https://github.com/apache/pulsar-site/pull/133#issuecomment-1175722892 > I see you've generated API docs for major and minor versions, so for future releases, we need to generate API docs for each version (align with the Pulsar release), correct?

Re: [VOTE] PIP-181: Pulsar Shell

2022-07-05 Thread Michael Marshall
+1 (non-binding) Thanks, Michael On Tue, Jul 5, 2022 at 1:28 AM Hang Chen wrote: > > +1 (binding) > > Thanks, > Hang > > Haiting Jiang 于2022年7月5日周二 14:26写道: > > > > +1 (non-binding) > > > > Thanks, > > Haiting > > > > On 2022/07/04 14:45:28 Nicolò Boschi wrote: > > > Dear Community, > > > > > >

Re: Trademark listing?

2022-07-05 Thread Dave Fisher
Hi Patrick, Pulsar is a trademark of the ASF.. It is not registered. Please use the ™ as appropriate. All The Best, Dave > On Jul 4, 2022, at 10:24 AM, Patrick McFadin wrote: > > Hi all, > > I was working with my publisher on an upcoming book project I'm working on > and found something I co

Re: [VOTE] Pulsar Release 2.9.3 Candidate 1

2022-07-05 Thread Massimiliano Mirelli
Thank you for the rc! +1 (non-binding) * verify sha512 checksums * verify gpg signatures * build pulsar and pulsar-all docker images (massimilianomirelli/pulsar{-all}:2.9.3-rc-1) * execute Fallout distributed system tests on multi-node k8s environment: * Pulsar functions, data-generator-source

Re: [VOTE] Pulsar Release 2.9.3 Candidate 1

2022-07-05 Thread mattison chao
>> There are bad files, pom.xml.versionsBackup, inside the source tarball >> >> I believe that this is a problem >> How did you build the release candidate ? >> >> the source tarball should not contain temporary Maven files >> Is this problem present in other Pulsar releases ? Yes, It’s a proble

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

2022-07-05 Thread Enrico Olivelli
-1 (binding) I left other comments on the discussion. I believe that this feature is very powerful and I generally support it. But we must clarify how it integrates with many other Pulsar features Thanks for driving this improvement Enrico Il giorno mar 5 lug 2022 alle ore 08:29 Haiting Jiang ha

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

2022-07-05 Thread Enrico Olivelli
I have a couple of additional questions. 1. Security What about security permissions about the shadow topic ? We are reading from another topic. I think we must clarify the decisions in the PIP 2. Truncation and deletion What happens when you truncate or delete the source topic ? please add a par

[GitHub] [pulsar-client-node] nkurihar merged pull request #224: Update async to 3.2.4

2022-07-05 Thread GitBox
nkurihar merged PR #224: URL: https://github.com/apache/pulsar-client-node/pull/224 -- 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...@puls

[GitHub] [pulsar-client-node] nkurihar closed pull request #81: Support seting topic schema when creating producers/consumers

2022-07-05 Thread GitBox
nkurihar closed pull request #81: Support seting topic schema when creating producers/consumers URL: https://github.com/apache/pulsar-client-node/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-client-node] nkurihar commented on pull request #81: Support seting topic schema when creating producers/consumers

2022-07-05 Thread GitBox
nkurihar commented on PR #81: URL: https://github.com/apache/pulsar-client-node/pull/81#issuecomment-1174693067 This change is merged as a new PR with fixing some compliation errors: https://github.com/apache/pulsar-client-node/pull/223 @lcartercondon Thank you for your contributi