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

Make server codegen more deterministic by ensuring service methods ordering #988 #992

Merged
merged 1 commit into from Jun 3, 2020

Conversation

charlietsai
Copy link
Contributor

From what I could tell, the .toSet transformations related to Service.methods were done to remove duplicates. Changing to a Seq via .distinct to ensure the ordering is deterministic.

Tests continue to pass with sbt test.

Also checked the example produced by

sbt "project akka-grpc-plugin-tester-scala" test
sbt "project akka-grpc-plugin-tester-java" test

and they look consistent with what was generated before.

@lightbend-cla-validator
Copy link

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@akka-ci
Copy link

akka-ci commented Jun 3, 2020

Thank you for your pull request! After a quick sanity check one of the team will reply with 'OK TO TEST' to kick off our automated validation on Jenkins. This compiles the project, runs the tests, and checks for things like binary compatibility and source code formatting. When two team members have also manually reviewed and (perhaps after asking for some amendments) accepted your contribution, it should be good to be merged.

For more details about our contributing process, check out CONTRIBUTING.md - and feel free to ask!

@raboof
Copy link
Member

raboof commented Jun 3, 2020

refs #988

@charlietsai charlietsai force-pushed the deterministic-server-codegen branch from 8c378fe to 277ed2c Compare June 3, 2020 08:37
@lightbend-cla-validator
Copy link

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@charlietsai charlietsai force-pushed the deterministic-server-codegen branch from 277ed2c to d4bb438 Compare June 3, 2020 08:56
@lightbend-cla-validator
Copy link

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@charlietsai
Copy link
Contributor Author

sorry for the noise, had to fix my author/comitter information

@raboof
Copy link
Member

raboof commented Jun 3, 2020

OK TO TEST

@raboof
Copy link
Member

raboof commented Jun 3, 2020

sorry for the noise, had to fix my author/comitter information

No problem, verified the CLA is now OK (even though the validator hasn't updated yet)

Copy link
Member

@raboof raboof left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

@raboof raboof merged commit 16db11d into akka:master Jun 3, 2020
@charlietsai
Copy link
Contributor Author

awesome, thanks for the quick turnaround!

@raboof
Copy link
Member

raboof commented Jun 3, 2020

Sure, thanks for the improvement!

@charlietsai charlietsai deleted the deterministic-server-codegen branch August 5, 2020 06:35
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.

None yet

4 participants