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

some persistent ops still replay infinitely on 'static #1050

Closed
3 of 4 tasks
jhellerstein opened this issue Jan 30, 2024 · 1 comment
Closed
3 of 4 tasks

some persistent ops still replay infinitely on 'static #1050

jhellerstein opened this issue Jan 30, 2024 · 1 comment
Assignees
Labels
bug Something isn't working hydroflow syntax Hydroflow's custom surface syntax P0 Highest priority
Milestone

Comments

@jhellerstein
Copy link
Contributor

jhellerstein commented Jan 30, 2024

As surfaced in #800, fold_keyed<'static... causes indefinite replay in a cycle. It should behave like persist as in #906 and check if context.is_first_run_this_tick().

Need to check if the same problem arises with fold and reduce and reduce_keyed and lattice_fold and _lattice_fold_batch.

Fixed?

@jhellerstein jhellerstein added this to the 0.7 Release milestone Jan 30, 2024
@MingweiSamuel MingweiSamuel changed the title persistent ops replay infinitely on 'static some persistent ops still replay infinitely on 'static Jan 31, 2024
@MingweiSamuel MingweiSamuel added bug Something isn't working hydroflow syntax Hydroflow's custom surface syntax labels Jan 31, 2024
@MingweiSamuel
Copy link
Member

And _lattice_join_fused_join has the problem

@MingweiSamuel MingweiSamuel added the P0 Highest priority label Feb 5, 2024
MingweiSamuel added a commit to MingweiSamuel/hydroflow that referenced this issue Mar 1, 2024
MingweiSamuel added a commit to MingweiSamuel/hydroflow that referenced this issue Mar 1, 2024
MingweiSamuel added a commit to MingweiSamuel/hydroflow that referenced this issue Mar 4, 2024
MingweiSamuel added a commit to MingweiSamuel/hydroflow that referenced this issue Mar 4, 2024
MingweiSamuel added a commit to MingweiSamuel/hydroflow that referenced this issue Mar 4, 2024
MingweiSamuel added a commit to MingweiSamuel/hydroflow that referenced this issue Mar 4, 2024
MingweiSamuel added a commit to MingweiSamuel/hydroflow that referenced this issue Mar 4, 2024
MingweiSamuel added a commit that referenced this issue Mar 4, 2024
MingweiSamuel added a commit that referenced this issue Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hydroflow syntax Hydroflow's custom surface syntax P0 Highest priority
Projects
None yet
Development

No branches or pull requests

2 participants