Change ParallelQuerySet to use QueryBuilder

[ad-claw000]

Summary

Extracted the command resolution logic in ParallelQuerySet into QueryBuilder.get_query_cmd. This refactoring simplifies the codebase and leverages QueryBuilder effectively.

Verification

• Added get_query_cmd to QueryBuilder in aperturedb/Query.py.
• Replaced manual dictionary key extraction in ParallelQuerySet.py.
• Evaluated Python syntax.

Fixes #265

[Copilot]

Pull request overview

Refactors ParallelQuerySet constraint handling by centralizing “command extraction” logic into a new QueryBuilder.get_query_cmd helper, reducing duplicated parsing logic and aiming to standardize query-structure interpretation.

Changes:

• Added QueryBuilder.get_query_cmd to extract the command name from supported query structures.
• Updated ParallelQuerySet to call QueryBuilder.get_query_cmd instead of manually inspecting dict/list shapes.
• Wrapped command extraction in a try/except to raise a constraint-specific error when unsupported structures are encountered.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
aperturedb/Query.py	Adds QueryBuilder.get_query_cmd to encapsulate query command-name extraction.
aperturedb/ParallelQuerySet.py	Replaces inline query-structure parsing with QueryBuilder.get_query_cmd during constraint evaluation.

Comments suppressed due to low confidence (1)

aperturedb/Query.py:284

• The exception message "Cannot extract command from query structure" is hard to act on when debugging. It would be more helpful to include what structures are supported (e.g., single-command dict vs [constraints_dict, command_dict]) and basic info about the received value (type/length/keys) without dumping large payloads.

        elif isinstance(query, list) and isinstance(query[0], dict) \
                and all(map(lambda k: k in known_constraint_keywords, query[0].keys()))  \
                and isinstance(query[1], dict):
            return list(query[1].keys())[0]
        else:
            raise Exception("Cannot extract command from query structure")

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[ad-claw000]

All PR review comments have been addressed in commit a9522cc and pushed to the branch.

[ad-claw000]

All actionable review comments have been addressed in commit a9522cc.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

[ad-claw000]

Done. Refined the ValueError message in aperturedb/Query.py to include the type, length, and keys of the received query structure without dumping large payloads, as suggested in the review. See commit e26d7f0.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.