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

[GORDO-1647] Switch on actors per default in pregel #18587

Closed
wants to merge 4 commits into from

Conversation

jvolmer
Copy link
Contributor

@jvolmer jvolmer commented Apr 5, 2023

With this PR, pregel tests will run using actors instead of the old pregel implementation. This PR is used to fix all the red tests when using actors.

@jvolmer jvolmer added 3 Pregel Graph processing DO_NOT_MERGE labels Apr 5, 2023
@jvolmer jvolmer requested a review from markuspf April 5, 2023 08:26
@jvolmer jvolmer self-assigned this Apr 5, 2023
@jvolmer jvolmer requested a review from a team as a code owner April 5, 2023 08:26
@cla-bot cla-bot bot added the cla-signed label Apr 5, 2023
Base automatically changed from feature/make-pregel-work-with-actors to devel April 5, 2023 14:06
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch from 1e73257 to dbfa608 Compare April 5, 2023 14:10
@jvolmer jvolmer changed the base branch from devel to feature/fix-more-actor-stuff April 6, 2023 08:37
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch from dbfa608 to c81231e Compare April 6, 2023 08:39
@jvolmer jvolmer force-pushed the feature/fix-more-actor-stuff branch from f16279b to 98d486c Compare April 11, 2023 15:44
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch from c81231e to d559e0c Compare April 11, 2023 21:11
@jvolmer jvolmer changed the base branch from feature/fix-more-actor-stuff to feature/gordo-1617-egress-from-actors April 11, 2023 21:24
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch 3 times, most recently from f5f1f9f to 26749d3 Compare April 17, 2023 15:06
@jvolmer jvolmer force-pushed the feature/gordo-1617-egress-from-actors branch from 44c67e2 to 4a0eb8e Compare April 17, 2023 17:25
@jvolmer jvolmer requested a review from a team as a code owner April 17, 2023 17:25
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch 5 times, most recently from 05bc93b to 37b13ea Compare April 18, 2023 15:17
@jvolmer jvolmer force-pushed the feature/gordo-1617-egress-from-actors branch from 4a0eb8e to 953e83d Compare April 18, 2023 16:25
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch 2 times, most recently from 09ebf3c to 959e65c Compare April 18, 2023 16:44
@jvolmer jvolmer force-pushed the feature/gordo-1617-egress-from-actors branch from 953e83d to cdec2ca Compare April 18, 2023 17:17
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch 2 times, most recently from 7e4f14a to 00ff220 Compare April 18, 2023 20:02
@jvolmer jvolmer force-pushed the feature/gordo-1617-egress-from-actors branch from cdec2ca to 06640bb Compare April 19, 2023 12:14
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch 2 times, most recently from c7891a7 to 63d16f2 Compare April 27, 2023 20:36
@jvolmer jvolmer force-pushed the feature/gordo-1617-egress-from-actors branch from f4ef28f to 7ecc5b8 Compare May 2, 2023 12:31
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch from 63d16f2 to 48052b7 Compare May 2, 2023 12:32
@jvolmer jvolmer changed the base branch from feature/gordo-1617-egress-from-actors to feature/fix-even-more-actor-stuff May 2, 2023 12:38
@jvolmer jvolmer force-pushed the feature/fix-even-more-actor-stuff branch from fd39085 to 5f3cec2 Compare May 3, 2023 07:56
Base automatically changed from feature/fix-even-more-actor-stuff to devel May 3, 2023 09:59
@jvolmer jvolmer force-pushed the feature/switch-on-actors-per-default-in-pregel branch 3 times, most recently from 864d4bc to 16f1c99 Compare May 5, 2023 12:43
@jvolmer jvolmer changed the base branch from devel to feature/pregel-with-actors-fixes May 5, 2023 12:47
@markuspf markuspf force-pushed the feature/pregel-with-actors-fixes branch from a8ae393 to cc185b4 Compare May 10, 2023 09:46
Base automatically changed from feature/pregel-with-actors-fixes to devel May 10, 2023 12:41
@adityamukho adityamukho self-assigned this May 24, 2023
@adityamukho adityamukho changed the title [PREGEL] Switch on actors per default in pregel [GORDO-1647] Switch on actors per default in pregel May 24, 2023
@adityamukho adityamukho requested review from mpoeter and a team as code owners May 25, 2023 10:59
Copy link
Contributor

@MBkkt MBkkt left a comment

Choose a reason for hiding this comment

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

merge devel to your branch probably wasn't correct

@adityamukho adityamukho force-pushed the feature/switch-on-actors-per-default-in-pregel branch from 83639cb to a7b6f2a Compare May 26, 2023 08:54
@adityamukho adityamukho requested a review from a team as a code owner June 12, 2023 02:44
@adityamukho adityamukho force-pushed the feature/switch-on-actors-per-default-in-pregel branch from bd508cc to 041b726 Compare June 12, 2023 02:48
@adityamukho adityamukho force-pushed the feature/switch-on-actors-per-default-in-pregel branch from 649c847 to 38aaaa7 Compare June 14, 2023 03:26
@jvolmer jvolmer closed this May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants