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

Query Library: Split query Manager into build, execute, output #103

Closed
anish-mudaraddi opened this issue Aug 25, 2023 · 1 comment
Closed

Comments

@anish-mudaraddi
Copy link
Collaborator

anish-mudaraddi commented Aug 25, 2023

We need to break QueryManager.test_and_build_query into separate methods so it's easier to test

class QueryManager:

   # args omitted for brevity
    def build_query():
    def execute_query():
    def output_query():

    def run_full_query():
        built = self.build_query()
        result = self.execute_query(built)
        return self.output_query(result)
@anish-mudaraddi
Copy link
Collaborator Author

deprecated in favor of using patterns and workflows #177 #176

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

No branches or pull requests

1 participant