Skip to content

Ensure query is only used once per query, rather than per source#250

Draft
will-langdale wants to merge 4 commits intomainfrom
feature/one-transfer
Draft

Ensure query is only used once per query, rather than per source#250
will-langdale wants to merge 4 commits intomainfrom
feature/one-transfer

Conversation

@will-langdale
Copy link
Copy Markdown
Collaborator

This is a draft PR to show some highly AI-authored code for ensuring only one call is made to query().

Warning

This draft PR that mainly exists to retain the high-level changes that fix the problem. It's extremely noisy and messy, and I haven't had time to cook it down to the important bits.

🛠️ Changes proposed in this pull request

  • Update the query() SQL query to return the right data
  • Extend the query data Arrow schema to include a dictionary of the various sources
  • Update the client to receive and process the new data

👀 Guidance to review

None.

🤖 AI declaration

All of it, none of it reviewed.

🔗 Relevant links

None.

✅ Checklist:

  • This is the smallest, simplest solution to the problem
  • I've read our code standards and this code follows them
  • All new code is tested
  • I've updated all relevant documentation (select all that apply)
    • API documentation (docstrings and indexes)
    • Tutorials
    • Developer docs

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