Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CoreLockingHelper: handle new Node types when retrieving task info from executionQueue #233

Merged

Conversation

rpalcolea
Copy link
Member

No description provided.

@rpalcolea rpalcolea force-pushed the CoreLockingHelper/handle-new-nodes-inexecution-graph branch 2 times, most recently from 9b8fcf5 to 6a4b257 Compare January 18, 2022 22:55
@rpalcolea rpalcolea force-pushed the CoreLockingHelper/handle-new-nodes-inexecution-graph branch from 6a4b257 to 308e513 Compare January 18, 2022 23:23
@rpalcolea rpalcolea merged commit 4ef318f into main Jan 18, 2022
@rpalcolea rpalcolea deleted the CoreLockingHelper/handle-new-nodes-inexecution-graph branch January 18, 2022 23:53
@rpalcolea
Copy link
Member Author

heads up @OdysseusLives

https://github.com/gradle/gradle/blob/master/subprojects/core/src/main/java/org/gradle/execution/plan/OrdinalNode.java was introduced for 7.4 and it is the last node in the executionQueue now.

Changed the logic to query for the last task node.

I guess at some point would be nice if we could move to public APIs for this or eventually remove it 😅

@OdysseusLives
Copy link
Member

Thanks for the heads-up! I see what you mean there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants