<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=ks_c_5601-1987">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
Both concerns have now been addressed in the PR and CSR.</div>
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<br>
</div>
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof ContentPasted0">
<a href="https://github.com/openjdk/jdk/pull/16420/commits/ec841740eb4cb6304243dd1b29a59acf9526bb79" id="LPlnkOWALinkPreview">https://github.com/openjdk/jdk/pull/16420/commits/ec841740eb4cb6304243dd1b29a59acf9526bb79</a><br>
</div>
<div class="_Entity _EType_OWALinkPreview _EId_OWALinkPreview _EReadonly_1">
<div id="LPBorder_GTaHR0cHM6Ly9naXRodWIuY29tL29wZW5qZGsvamRrL3B1bGwvMTY0MjAvY29tbWl0cy9lYzg0MTc0MGViNGNiNjMwNDI0M2RkMWIyOWE1OWFjZjk1MjZiYjc5" class="LPBorder959271" style="width: 100%; margin-top: 16px; margin-bottom: 16px; position: relative; max-width: 800px; min-width: 424px;">
<table id="LPContainer959271" role="presentation" style="padding: 12px 36px 12px 12px; width: 100%; border-width: 1px; border-style: solid; border-color: rgb(200, 200, 200); border-radius: 2px;">
<tbody>
<tr style="border-spacing: 0px;" valign="top">
<td>
<div id="LPImageContainer959271" style="position: relative; margin-right: 12px; height: 160px; overflow: hidden;">
<a target="_blank" id="LPImageAnchor959271" href="https://github.com/openjdk/jdk/pull/16420/commits/ec841740eb4cb6304243dd1b29a59acf9526bb79"><img id="LPThumbnailImageId959271" alt="" style="display: block;" width="160" height="160" src="https://avatars.githubusercontent.com/u/114235711?s=400&v=4"></a></div>
</td>
<td style="width: 100%;">
<div id="LPTitle959271" style="font-size: 21px; font-weight: 300; margin-right: 8px; font-family: "wf_segoe-ui_light", "Segoe UI Light", "Segoe WP Light", "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; margin-bottom: 12px;">
<a target="_blank" id="LPUrlAnchor959271" href="https://github.com/openjdk/jdk/pull/16420/commits/ec841740eb4cb6304243dd1b29a59acf9526bb79" style="text-decoration: none;">8319123: Implement JEP 461: Stream Gatherers (Preview) by viktorklang-ora ¡¤ Pull Request
#16420 ¡¤ openjdk/jdk</a></div>
<div id="LPDescription959271" style="font-size: 14px; max-height: 100px; font-family: "wf_segoe-ui_normal", "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; margin-bottom: 12px; margin-right: 8px; overflow: hidden; color: rgb(102, 102, 102);">
This Pull-Request implements JEP-461 Progress Change must be properly reviewed (1 review required, with at least 1 Reviewer) Change must not contain extraneous whitespace Commit message must ...</div>
<div id="LPMetadata959271" style="font-size: 14px; font-weight: 400; font-family: "wf_segoe-ui_normal", "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; color: rgb(166, 166, 166);">
github.com</div>
</td>
</tr>
</tbody>
</table>
</div>
</div>
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof">
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div id="Signature">
<div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Cheers,<br>
¡î</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<b><br>
</b></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<b>Viktor Klang</b></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Software Architect, Java Platform Group<br>
Oracle<br>
</div>
</div>
</div>
</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> jdk-dev <jdk-dev-retn@openjdk.org> on behalf of Viktor Klang <viktor.klang@oracle.com><br>
<b>Sent:</b> Tuesday, 21 November 2023 10:59<br>
<b>To:</b> Daniel Schmid <daniel@wwwmaster.at>; jdk-dev@openjdk.org <jdk-dev@openjdk.org>; Mark Reinhold <mark.reinhold@oracle.com><br>
<b>Subject:</b> Re: [External] : Re: JEP proposed to target JDK 22: 461: Stream Gatherers (Preview)</font>
<div> </div>
</div>
<style type="text/css" style="display:none">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div dir="ltr">
<div class="x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
No worries -- I agree that using initializer there seems more consistent, and it's already clear that it is a Supplier so repeating that seems unnecessary.<br>
</div>
<div class="x_elementToProof">
<div style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<div id="x_Signature">
<div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
Cheers,<br>
¡î</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<b><br>
</b></div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<b>Viktor Klang</b></div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
Software Architect, Java Platform Group<br>
Oracle<br>
</div>
</div>
</div>
</div>
<div>
<div id="x_appendonsend"></div>
<div><br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
<hr tabindex="-1" style="display:inline-block; width:98%">
<b>From:</b> Daniel Schmid<br>
<b>Sent:</b> Tuesday, November 21, 2023 10:41<br>
<b>To:</b> Viktor Klang; jdk-dev@openjdk.org; Mark Reinhold<br>
<b>Subject:</b> [External] : Re: JEP proposed to target JDK 22: 461: Stream Gatherers (Preview)
<div><br>
</div>
</div>
<div class="x_rps_63f6">
<div>
<p style="margin-top:0px; margin-bottom:0px">And another thing I realized after writing that E-Mail, the "initializer" parameter is called</p>
<p style="margin-top:0px; margin-bottom:0px">> the supplier function for the new gatherer</p>
<p style="margin-top:0px; margin-bottom:0px">I am unsure about the name "supplier function". After all, it is already named initializer so why not use the name initializer (function) instead of supplier function?<br>
</p>
<p style="margin-top:0px; margin-bottom:0px"><br>
</p>
<div class="x_x_moz-cite-prefix">Am 21.11.2023 um 10:35 schrieb Viktor Klang:<br>
</div>
<blockquote type="cite">
<div class="x_x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
Thanks for the feedback -- I agree that could be improved.</div>
<div class="x_x_elementToProof" style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
I'll address this in the PR/CSR shortly.<br>
</div>
<div class="x_x_elementToProof">
<div style="font-family:Aptos,Aptos_EmbeddedFont,Aptos_MSFontService,Calibri,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<div id="x_x_Signature">
<div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
Cheers,<br>
¡î</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<b><br>
</b></div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<b>Viktor Klang</b></div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
Software Architect, Java Platform Group<br>
Oracle<br>
</div>
</div>
</div>
</div>
<div>
<div><br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
<hr tabindex="-1" style="display:inline-block; width:98%">
<b>From:</b> jdk-dev on behalf of Daniel Schmid<br>
<b>Sent:</b> Tuesday, November 21, 2023 10:31<br>
<b>To:</b> <a href="mailto:jdk-dev@openjdk.org" class="x_x_moz-txt-link-abbreviated x_OWAAutoLink" id="OWA099a624b-1a1f-ecad-78cc-601ffc69ddfc" data-loopstyle="linkonly">
jdk-dev@openjdk.org</a>; Mark Reinhold<br>
<b>Subject:</b> Re: JEP proposed to target JDK 22: 461: Stream Gatherers (Preview)
<div><br>
</div>
</div>
<div class="x_x_BodyFragment"><font size="2"><span style="font-size:11pt">
<div class="x_x_PlainText">When reading the type parameters of the <br>
"Gatherer.of"/"Gatherer.ofSequential" methods, I realized that the type <br>
parameter for the intermediate state is documted as<br>
<br>
> the type of initializer for the new gatherer<br>
<br>
While not being wrong, it looks a bit confusing to me as it refers to <br>
the intermediate state which is also used by the integrator, that type <br>
parameter is not exclusive to the initializer.<br>
<br>
Is there a reason it is not called<br>
<br>
> the type of intermediate state for the new gatherer<br>
<br>
or something similar?<br>
<br>
Am 20.11.2023 um 17:17 schrieb Mark Reinhold:<br>
> The following JEP is proposed to target JDK 22:<br>
><br>
> 461: Stream Gatherers (Preview)<br>
> <a href="https://openjdk.org/jeps/461" target="_blank" rel="noopener noreferrer" data-auth="NotApplicable" id="OWAe4450ccd-5eae-7d44-4660-e46f70766eb3" class="x_x_OWAAutoLink x_x_moz-txt-link-freetext" data-loopstyle="linkonly">
https://openjdk.org/jeps/461</a><br>
><br>
> Summary: Enhance the Stream API to support custom intermediate<br>
> operations. This will allow stream pipelines to transform data in ways<br>
> that are not easily achievable with the existing built-in intermediate<br>
> operations. This is a preview API.<br>
><br>
> Feedback on this proposal from JDK Project Committers and Reviewers [1]<br>
> is more than welcome, as are reasoned objections. If no such objections<br>
> are raised by 20:00 UTC on Wednesday, 29 November, or if they¡¯re raised<br>
> and then satisfactorily answered, then per the JEP 2.0 process proposal<br>
> [2] I¡¯ll target this JEP to JDK 22.<br>
><br>
> - Mark<br>
><br>
><br>
> [1] <a href="https://openjdk.org/census#jdk" target="_blank" rel="noopener noreferrer" data-auth="NotApplicable" id="OWA4454c98a-edde-f8a7-3acd-1c0b10a2bb21" class="x_x_OWAAutoLink x_x_moz-txt-link-freetext" data-loopstyle="linkonly">
https://openjdk.org/census#jdk</a><br>
> [2] <a href="https://cr.openjdk.java.net/~mr/jep/jep-2.0-02.html" target="_blank" rel="noopener noreferrer" data-auth="NotApplicable" id="OWAb017981f-0b90-6c8b-f2d9-a8b1cca35ccc" class="x_x_OWAAutoLink x_x_moz-txt-link-freetext" data-loopstyle="linkonly">
https://cr.openjdk.java.net/~mr/jep/jep-2.0-02.html</a><br>
</div>
</span></font></div>
</div>
</blockquote>
</div>
</div>
</div>
</div>
</body>
</html>