Closed yoavcloud closed 2 weeks ago
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
src/ast/mod.rs | 32 | 36 | 88.89% | ||
<!-- | Total: | 91 | 95 | 95.79% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
src/ast/mod.rs | 1 | 82.14% | ||
src/dialect/postgresql.rs | 2 | 73.5% | ||
<!-- | Total: | 3 | --> |
Totals | |
---|---|
Change from base Build 11580333290: | 0.02% |
Covered Lines: | 30500 |
Relevant Lines: | 34118 |
Sorry, did a last change and forgot cargo fmt... Changes committed now, if you can run the CI again that would be great @alamb
🚀
@iffyio @alamb why isn't this merged?
@iffyio @alamb why isn't this merged?
Sorry -- am behind.
Thank you!
This PR adds support for the above statements in the Hive dialect, making the previously MySQL-only structs a bit more generic.