Skip to content

feat: support sqlite insert or statement - #281

Merged
Dandandan merged 1 commit into
apache:mainfrom
zhangli-pear:feat_sqlite_insert_or
Feb 9, 2021
Merged

Dandandan merged 1 commit into
apache:mainfrom
zhangli-pear:feat_sqlite_insert_or

Conversation

@zhangli-pear

Copy link
Copy Markdown
Contributor

@coveralls

coveralls commented Nov 6, 2020 •

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 546765859

  • 46 of 49 (93.88%) changed or added relevant lines in 3 files are covered.
  • 11 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.07%) to 90.023%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/ast/mod.rs 13 14 92.86%
src/parser.rs 15 16 93.75%
tests/sqlparser_common.rs 18 19 94.74%
Files with Coverage Reduction New Missed Lines %
src/parser.rs 1 86.2%
src/ast/mod.rs 10 71.75%
Totals Coverage Status
Change from base Build 545529532: -0.07%
Covered Lines: 5387
Relevant Lines: 5984

💛 - Coveralls

Comment thread src/ast/mod.rs Outdated
Comment thread src/ast/mod.rs Outdated
@Dandandan

Copy link
Copy Markdown
Contributor

Thanks for the contribution! I left a few comments :)

@zhangli-pear

Copy link
Copy Markdown
Contributor Author

I’v changed code following discussion above, please review again.BTW, how long does it take to publish a new version to crates.io?

@zhangli-pear

Copy link
Copy Markdown
Contributor Author

Is there anybody still following this thread?

@Dandandan

Copy link
Copy Markdown
Contributor

Thanks @zhangli-pear for building this feature.
Sorry for waiting so long to have a look at it.

Comment thread src/ast/mod.rs Outdated
@Dandandan

Copy link
Copy Markdown
Contributor

LGTM now @nickolay do you maybe have any comments?

Comment thread src/parser.rs Outdated
Comment thread src/parser.rs Outdated
@Dandandan

Copy link
Copy Markdown
Contributor

@zhangli-pear can you rebase against the main branch?

@zhangli-pear

Copy link
Copy Markdown
Contributor Author

Done. I didn't notice the conflict before😂.

@Dandandan

Copy link
Copy Markdown
Contributor

Thanks @zhangli-pear for the changes 👍

@Dandandan
Dandandan merged commit add8991 into apache:main Feb 9, 2021
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

Successfully merging this pull request may close these issues.

4 participants