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.

Sign up for GitHub

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

Jump to bottom

Fix bug in match_node #51

Open

HazelYuAhiru wants to merge 1 commit into main from fix/match_node

+39 −1

Collaborator

HazelYuAhiru commented Nov 26, 2024

Changes Made

Modified match_node method to ensure that query_node is of type dictionary before we access query_node.keys()
Added a new rule in rules.py and a new test case test_rewrite_stackoverflow_1 in test_query_rewriter.py for testing.

Testing

Passed all existing test cases defined under test


          fix bug in match_node and add test case

84365a2

HazelYuAhiru requested a review from baiqiushi

November 26, 2024 04:58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet