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

Consider implementing a version of before-update that doesn't fetch the originals #136

Open
camsaul opened this issue Mar 24, 2023 · 0 comments
Labels
enhancement New feature or request medium priority performance under consideration Possible ideas that may be implemented in the future

Comments

@camsaul
Copy link
Owner

camsaul commented Mar 24, 2023

Sometimes all you want to do is add a key like :updated_at (t/offset-date-time), there is really no need to fetch the original instance in this case. Seems like a significant performance win.

But really the last thing we need to do is make Toucan 2 even more complicated, so maybe we can noodle and think of a way that we can do this automagically. Can before-update determine whether any columns are accessed from the original object somehow, and only fetch it if they are? TBD.

@camsaul camsaul added enhancement New feature or request under consideration Possible ideas that may be implemented in the future medium priority performance labels Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request medium priority performance under consideration Possible ideas that may be implemented in the future
Projects
None yet
Development

No branches or pull requests

1 participant