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

WIP: Fine grained state access #261

Open
wants to merge 38 commits into
base: main
Choose a base branch
from
Open

Conversation

shreyashankar
Copy link
Collaborator

@shreyashankar shreyashankar commented Nov 21, 2023

Description

This is a breaking PR! Motion applications that update to this version will stop working.

Related Issues

[Link any relevant issues that are addressed in this pull request.]

Checklist

  • Add SSL param to State redis config
  • Remove unnecessary clones & test that performance is actually better
  • Handle MDataframe
  • Fix build script in Makefile
  • Add the pyo3 library to our config & release somehow
  • add if: "startsWith(github.ref, 'refs/tags/')" back to cd.yml
  • dynamic state loading policy -- prefetch keys based on the op, as well as smarter batching for redis
  • Delete unnecessary/commented out code
  • Think about backwards compatibility

Later:

  • Serialize large datasets in parallel

Screenshots

[Add any relevant screenshots or images to help illustrate the changes in this pull request.]

Additional Notes

[Add any additional information or context that you feel is important for reviewers to know about this pull request.]

@shreyashankar shreyashankar force-pushed the shreyashankar/removelock branch from c994961 to 216a58c Compare November 24, 2023 16:56
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.

1 participant