Skip to content

Java: Revert wrongly generated stats file#20385

Merged
IdrissRio merged 2 commits intomainfrom
idrissrio/java-regression-statsfile
Sep 11, 2025
Merged

Java: Revert wrongly generated stats file#20385
IdrissRio merged 2 commits intomainfrom
idrissrio/java-regression-statsfile

Conversation

@IdrissRio
Copy link
Copy Markdown
Contributor

@IdrissRio IdrissRio commented Sep 8, 2025

The stats file generated in commit
a6b68ec#diff-a4cfdde566085b7f1c1315487c13ef64e4869729ae7886b6c8f4267e3cd4eab6 was created using the wrong process.

This PR reverts that change and manually adds the new relation for isImplicitClass.

@github-actions github-actions Bot added the Java label Sep 8, 2025
@IdrissRio IdrissRio marked this pull request as ready for review September 10, 2025 08:24
@IdrissRio IdrissRio requested a review from a team as a code owner September 10, 2025 08:24
Copilot AI review requested due to automatic review settings September 10, 2025 08:24
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@owen-mc
Copy link
Copy Markdown
Contributor

owen-mc commented Sep 10, 2025

Btw, this would be easier to review if it was in two commits.

@IdrissRio
Copy link
Copy Markdown
Contributor Author

@owen-mc

Btw, this would be easier to review if it was in two commits.

I'll do it right away!

@IdrissRio IdrissRio force-pushed the idrissrio/java-regression-statsfile branch from 9e65bab to aa71d38 Compare September 10, 2025 09:06
@aschackmull
Copy link
Copy Markdown
Contributor

Could you please rebase on latest main and do a fresh dca run, such that we don't drown out any effects of the revised stats change by the currently non-terminating base branch.

The stats file generated in commit
a6b68ec#diff-a4cfdde566085b7f1c1315487c13ef64e4869729ae7886b6c8f4267e3cd4eab6
was created using the wrong process.

This commit reverts that change and manually adds the new relation
for isImplicitClass.
@IdrissRio IdrissRio force-pushed the idrissrio/java-regression-statsfile branch from aa71d38 to 762d2ef Compare September 10, 2025 12:45
Copy link
Copy Markdown
Contributor

@aschackmull aschackmull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The combination of the two dca runs look good.

@IdrissRio IdrissRio merged commit 59fc7aa into main Sep 11, 2025
13 checks passed
@IdrissRio IdrissRio deleted the idrissrio/java-regression-statsfile branch September 11, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants