Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cu-dev] Concurrency 3.0 pulls ready for review: #38, #40, #46, #100, #101, #35, #99
  • From: "Steve Millidge (Payara)" <steve.millidge@xxxxxxxxxxx>
  • Date: Mon, 13 Sep 2021 11:25:32 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=payara.fish; dmarc=pass action=none header.from=payara.fish; dkim=pass header.d=payara.fish; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=INMs09Vh7fXdU09s6CSf2k8t50pjQcPhY1uN9lhfCgc=; b=ONfieqPJCy6UQWPsrHsMF3NG3115EAOnqi/4Vrpwg4l9pQP9g8VCCqXp4qyBwML6S9K8RLz+Mf9kLgwNusebIZeWQTSx0K1dlR/SRhYGm6Li3SM14zB8rfoRyf4gAA+Sfzt58MW+d/rUo70VnWX2UubYZ1/pKC1HtKcn1oc4t02PTbrpwcc+sWphJ8xWeX/0rGuFxosdZPPrtFCAAmcR/usZ8ggXyvbgV9UH8enKjvLSVOo51WeDhW7+97p5DcOuEYTWBg0i+ZlcZAhziSue3zaln6BJLbmzjBlnYabTSD929Y3CsOr2waEAwvXni/SvmvA+0jWBSTIlWCKZlM6c3Q==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=TDNZUiGNMjMn5Lc80eut+sBYD78kmrTD/1O5/Il8w9XDYvJ2Wp3g6glWYZKJpxErltpO4pYPow85076XYsaIE+WMLqvs2Vi6r5CXRfTjd+7OcDa8gYA6Z7sjYf775el35A9lisUgGzmx+Qv2MGQpmWgOxOmvq71FPwol8AJ/HajuQ/KhksSJtOGbLOjYVMw+hVWnGe0rMEXTiRIYdus9R03cZ5BtYzIg4wXYLQQTZ1Igyco+dfi2rR5+JCZ8emGxEKl4sNnCbmsd0a/QH3MQNtZxfyUOl+HNzocNVbWv/gFci70uQM/G8fZ2io7OGuj3+ycCNrl5TcqOiSRhZvvCvw==
  • Delivered-to: cu-dev@xxxxxxxxxxx
  • List-archive: <https://www.eclipse.org/mailman/private/cu-dev/>
  • List-help: <mailto:cu-dev-request@eclipse.org?subject=help>
  • List-subscribe: <https://www.eclipse.org/mailman/listinfo/cu-dev>, <mailto:cu-dev-request@eclipse.org?subject=subscribe>
  • List-unsubscribe: <https://www.eclipse.org/mailman/options/cu-dev>, <mailto:cu-dev-request@eclipse.org?subject=unsubscribe>
  • Thread-index: AQHXqJIQt/UgIU9/tk+dhCnpkNYBUA==
  • Thread-topic: [cu-dev] Concurrency 3.0 pulls ready for review: #38, #40, #46, #100, #101, #35, #99

Hi Nathan,

 

Go ahead and merge.

 

Apologies of being a bit slow in responding recently I am in the middle of a house move and sale.

 

Steve

 

From: cu-dev <cu-dev-bounces@xxxxxxxxxxx> On Behalf Of Nathan Rauh
Sent: 10 September 2021 23:12
To: cu developer discussions <cu-dev@xxxxxxxxxxx>
Subject: Re: [cu-dev] Concurrency 3.0 pulls ready for review: #38, #40, #46, #100, #101, #35, #99

 

I'd like to start merging the 3 pulls from the list below that consist of copying interfaces over from MicroProfile.
Let me know if anyone is still in the process of reviewing these and would like more time, otherwise I'll go ahead and merge them in 5 days or so

Issue #40Completion stages backed by ManagedExecutorService
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/104

Issue #100ContextService methods for contextualizing individual dependent stage actions
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/138

Issue #101Completion stages backed by a ContextService
pull (also contains the commit for #100 which is a prereq due to references made in JavaDoc): https://github.com/eclipse-ee4j/concurrency-api/pull/139
commit that is unique to this pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/139/commits/ba9c817de5e47c4a32ee40c6bd46d97dcb3eea0e




From:        Nathan Rauh/Rochester/IBM
To:        "cu developer discussions" <cu-dev@xxxxxxxxxxx>
Date:        08/27/2021 01:26 PM
Subject:        Re: [EXTERNAL] Re: [cu-dev] Concurrency 3.0 pulls ready for review: #38, #40, #46, #100, #101, #143, #144



Adding 2 more pulls to the list needing review,

Issue #40Completion stages backed by ManagedExecutorService
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/104

Issue #100ContextService methods for contextualizing individual dependent stage actions
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/138

Issue #101Completion stages backed by a ContextService
pull (also contains the commit for #100 which is a prereq due to references made in JavaDoc): https://github.com/eclipse-ee4j/concurrency-api/pull/139
commit that is unique to this pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/139/commits/ba9c817de5e47c4a32ee40c6bd46d97dcb3eea0e

Issue #38Define Concurrency Resources Portably
This pull is the first step - getting the annotations defined with the most basic attributes.
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/141

Issue #46Parallel Stream Support
The most I can see doing for this one is to have ManagedThreadFactory also implement ForkJoinWorkerThreadFactory.  Alternatively, we could do nothing and leave it to the user to implement something similar based on ContextService.
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/140

Issue #35Java SE 8 Date and Time Support
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/143

Issue #99Cron-based Trigger
This builds on the Java SE Date/Time based trigger (#35 above) so it also includes the commit for it.
commit that is unique to this pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/144/commits/8cec1222b66ea6ff68a730d1923fc7355592efe2






From:        "Steve Millidge (Payara)" <steve.millidge@xxxxxxxxxxx>
To:        "cu developer discussions" <cu-dev@xxxxxxxxxxx>
Date:        08/10/2021 06:52 AM
Subject:        [EXTERNAL] Re: [cu-dev] Concurrency 3.0 pulls ready for review: #38, #40, #46, #100, #101
Sent by:        "cu-dev" <cu-dev-bounces@xxxxxxxxxxx>

 

Hi,

 

I will get my team to review. I’m also trying to push to get more feedback and involvement from others through a number of JUG talks and Jakarta tech talks. I’d like to get together a team that is interested in driving this forward in the Jakarta EE 10 timeframe.

 

I think it would be good to also have a co-project lead so I don’t feel the burden to drive the specification falls to me. @Nathan would you be interested in being a co-project lead?

 

Steve

 

From:cu-dev <cu-dev-bounces@xxxxxxxxxxx> On Behalf Of Nathan Rauh
Sent: 09 August 2021 22:31
To: cu developer discussions <cu-dev@xxxxxxxxxxx>
Subject: [cu-dev] Concurrency 3.0 pulls ready for review: #38, #40, #46, #100, #101

 

Now that we are at a point where we can start making updates for Jakarta EE 10, could specification participants review the 5 pulls for the Concurrency 3.0 planned items below (whichever you are comfortable with reviewing) and make comments in them regarding any changes that are needed or otherwise approve them for merging? Thanks!

Issue #40Completion stages backed by ManagedExecutorService
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/104

Issue #100ContextService methods for contextualizing individual dependent stage actions
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/138

Issue #101Completion stages backed by a ContextService
pull (also contains the commit for #100 which is a prereq due to references made in JavaDoc): https://github.com/eclipse-ee4j/concurrency-api/pull/139
commit that is unique to this pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/139/commits/ba9c817de5e47c4a32ee40c6bd46d97dcb3eea0e

Issue #38Define Concurrency Resources Portably
This pull is the first step - getting the annotations defined with the most basic attributes.
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/141

Issue #46Parallel Stream Support
The most I can see doing for this one is to have ManagedThreadFactory also implement ForkJoinWorkerThreadFactory.  Alternatively, we could do nothing and leave it to the user to implement something similar based on ContextService.
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/140




From:        Nathan Rauh/Rochester/IBM
To:        cu developer discussions <cu-dev@xxxxxxxxxxx>
Date:        05/10/2021 04:29 PM
Subject:        Ready for review: #38, #40, #46, #100, #101



In addition to the 3 pulls that I mentioned previously (forwarded) for bringing over MicroProfile API,
the following 2 pulls are also ready for review:


Issue #38Define Concurrency Resources Portably
This pull is the first step - getting the annotations defined with the most basic attributes.
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/141


Issue #46Parallel Stream Support
The most I can see doing for this one is to have ManagedThreadFactory also implement ForkJoinWorkerThreadFactory.  Alternatively, we could do nothing and leave it to the user to implement something similar based on ContextService.
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/140


see below for 3 additional pulls to copy over MicroProfile API,

----- Forwarded by Nathan Rauh/Rochester/IBM on 05/10/2021 04:11 PM -----

From:        Nathan Rauh/Rochester/IBM
To:        cu developer discussions <cu-dev@xxxxxxxxxxx>
Date:        03/31/2021 10:11 AM
Subject:        Ready for review: #40, #100, #101 for transferring over MicroProfile API to Jakarta EE Concurrency 3.0



Given the amount of proposed content for Concurrency in Jakarta 10, I've started putting together pulls for the MicroProfile-related enhancements currently in the Milestone 3 list, starting with the ones for copying over API methods around CompletionStage/CompletableFuture.  The following 3 are ready to start reviewing now (I won't actually be merging anything until after the release plan is official, and of course pending approval from the community)

When reviewing these, it will be helpful to note that MicroProfile class ManagedExecutor is the equivalent of our ManagedExecutorService,
and MicroProfile class ThreadContext is the equivalent of our ContextService.

The method signatures themselves are an exact copy (which will be great for compatibility), except for the one place where MicroProfile's ManagedExecutor has a getThreadContext method that returns a ThreadContext, which is switched to ContextService. The intent behind the MicroProfile method here is so that threadContext/contextService.withContextCapture(completionStage/completableFuture) can create CompletionStage/CompletableFuture proxies that are backed by the ManagedExecutor.  But managedExecutor[Service].copy(completionStage/completableFuture) can already do that.  We can decide if the equivalent getContextService method is really wanted/needed here, and if not, I can remove it.  To start out with, I included it in the pull so that you have the opportunity to see what it would look like if we did include it. Please add your feedback on it in the review.  I would be happy with either outcome and will go with the majority on it.

Issue #40Completion stages backed by ManagedExecutorService
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/104

Issue #100ContextService methods for contextualizing individual dependent stage actions
pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/138

Issue #101Completion stages backed by a ContextService
pull (also contains the commit for #100 which is a prereq due to references made in JavaDoc): https://github.com/eclipse-ee4j/concurrency-api/pull/139
commit that is unique to this pull to review:
https://github.com/eclipse-ee4j/concurrency-api/pull/139/commits/ba9c817de5e47c4a32ee40c6bd46d97dcb3eea0e



_______________________________________________
cu-dev mailing list
cu-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/cu-dev

 

Back to the top