Skip to content

Commit

Permalink
feat: update substrait to 0.57.1 (#274)
Browse files Browse the repository at this point in the history
* cargo update

* update substrait submodule to 0.57.1

* Update module to compile with substrait 0.57.1

* Fix interval handling and clippy errors

* Switch to returning diagnostics where possible

* Update test suite to 0.57.1

* Add some checks for subseconds and precision checking

* Update pre-commit hooks to functional versions

* YAML formatting

* Fix clippy on generated code

* Polish from code review

* Apply suggestions from code review

Co-authored-by: Matthijs Brobbel <m1brobbel@gmail.com>

* Clearer variable name

* Allow no precision if subseconds are not specified

Required for test case tpc-h01 to pass, where `days: 1` is specified, but subseconds and precision are not.

* Clarify comment

---------

Co-authored-by: Matthijs Brobbel <m1brobbel@gmail.com>
  • Loading branch information
wackywendell and mbrobbel authored Oct 10, 2024
1 parent 8b22bdb commit 5f6b8fc
Show file tree
Hide file tree
Showing 27 changed files with 547 additions and 358 deletions.
4 changes: 2 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,15 @@ repos:
hooks:
- id: buf-lint
- repo: https://github.com/adrienverge/yamllint.git
rev: v1.26.0
rev: v1.35.1
hooks:
- id: yamllint
args: [-c=.yamllint.yaml]
- repo: https://github.com/psf/black
rev: 22.3.0
hooks:
- id: black
- repo: https://gitlab.com/pycqa/flake8
- repo: https://github.com/pycqa/flake8
rev: 4.0.1
hooks:
- id: flake8
Expand Down
Loading

0 comments on commit 5f6b8fc

Please sign in to comment.