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

async.q and bypassing getDatalogQuery() #291

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

async.q and bypassing getDatalogQuery() #291

wants to merge 6 commits into from

Conversation

mdroidian
Copy link
Collaborator

@mdroidian mdroidian commented Nov 20, 2024

proof of concept, but still TODO would be

  • generate query / complementQuery on relation save
  • handle /context/ and /anchor/ if those are still used

.. and cleanup the code😁 I was testing a bunch of stuff and didn't revisit all of it.

@mdroidian
Copy link
Collaborator Author

mdroidian commented Nov 21, 2024

Added a QueryTester.tsx to mimic fireQuery, with a bunch of results.
https://github.com/RoamJS/query-builder/pull/291/files#diff-3da15b5c2c39a89b7c2b7864e251037906a51f874ef11b71979279f9a4096c4b

The piece that is still evading me is exactly how exactly a blocking function getDatalogQuery() inside of fireQuery which is wrapped in Promise.all(queries.map(fireQuery)) changes the query response times.

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