-
Notifications
You must be signed in to change notification settings - Fork 209
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
Remove Store trait related cfg features, #1091
Conversation
Enable all by default.
Pull Request Test Coverage Report for Build 4144994883
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This code looks good. Would it be possible to update the README as there are some outdated sections present
v0.14 release will be .. next month so it would be too early to update README at this moment. |
Enable all by default. Compile time feature control causes too much management cost. We already have test cases in test-suite splitted based on store trait features.
Remove Store trait related cfg features, (gluesql#1091) Enable all by default. Compile time feature control causes too much management cost. We already have test cases in test-suite splitted based on store trait features. Support `values` query in AST builder (gluesql#1041) Currently, ASTBuilder does not support Values Query yet. This PR supports Values Query in ASTBuilder e.g. let actual = values(vec!["1, 'a'", "2, 'b'"]) .order_by(vec!["column1 desc"]) .build(); let expected = "VALUES(1, 'a'), (2, 'b') ORDER BY column1 desc";
Enable all by default.
Compile time feature control causes too much management cost.
We already have test cases in
test-suite
splitted based on store trait features.