PriorityQueue PR requiring review
David Holmes
david.holmes at oracle.com
Fri Mar 4 01:14:00 UTC 2022
On 3/03/2022 11:02 pm, Julian Waters wrote:
> I understand, I'll notify the author about this. I'm not sure if they'll
> be alright with discussing on the mailing lists though, since they have
> expressed that they prefer discussing it on the PR itself
That's not the way OpenJDK works, and anything in the PR (once properly
setup) goes to the mailing list anyway.
https://openjdk.java.net/contribute/
Cheers,
David
> best regards,
> Julian Waters
>
> On Thu, Mar 3, 2022 at 8:57 PM David Holmes <david.holmes at oracle.com
> <mailto:david.holmes at oracle.com>> wrote:
>
> On 3/03/2022 10:47 pm, Julian Waters wrote:
> > Hi David,
> >
> > I did not create the PR, I'm instead asking for others to review it
> > before I help the author create the issue on the JBS. Should I
> just go
> > ahead and create the issue for them?
>
> The best thing would be for the PR creator to discuss the proposed API
> addition on the mailing list first. I have a fairly good idea what the
> outcome of that discussion will be but ... :)
>
> Cheers,
> David
>
> > best regards,
> > Julian
> >
> > On Thu, Mar 3, 2022 at 8:45 PM David Holmes
> <david.holmes at oracle.com <mailto:david.holmes at oracle.com>
> > <mailto:david.holmes at oracle.com
> <mailto:david.holmes at oracle.com>>> wrote:
> >
> > Hi Julian,
> >
> > On 3/03/2022 10:33 pm, Jules W. wrote:
> > > Hi all,
> > >
> > > A new PR that adds methods to PriorityQueue was created
> some time
> > ago at
> > > https://github.com/openjdk/jdk/pull/6938
> <https://github.com/openjdk/jdk/pull/6938>
> > <https://github.com/openjdk/jdk/pull/6938
> <https://github.com/openjdk/jdk/pull/6938>> but has no corresponding
> > issue. As
> > > I'm not too familiar with this part of the JDK I'm
> querying this
> > mailing
> > > list for anyone to properly review the PR before I create an
> > issue for it
> > > in the JBS
> >
> > First you need an issue, then you review a PR. Your PR would
> not be
> > seen
> > by anyone unless they go looking for it, as without the
> association
> > with
> > an issue, the system will not send the email to the mailing
> lists.
> >
> > I see someone is helping you create the issue so things
> should progress
> > in that sense. But note the bar for adding to a public API is set
> > very high.
> >
> > Cheers,
> > David
> >
> > > best regards,
> > > Julian Waters
> >
>
More information about the core-libs-dev
mailing list