Skip to content

Conversation

@KKould
Copy link
Member

@KKould KKould commented Nov 6, 2023

What is changed and how it works?

Code changes

  • Has Rust code change
  • Has CI related scripts change

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Note for reviewer

@KKould KKould requested review from lewiszlw and loloxwg and removed request for lewiszlw November 6, 2023 14:47
@KKould KKould self-assigned this Nov 6, 2023
@KKould KKould added enhancement New feature or request invalid This doesn't seem right labels Nov 6, 2023
@lewiszlw
Copy link
Member

lewiszlw commented Nov 7, 2023

This pr contains like/null expr, ColumnCatalog refactoring, constant calculation, column pruning, etc, not friendly to reviewer. You could split pr into smaller pieces next time.

@KKould KKould merged commit 09c1042 into KipData:main Nov 11, 2023
KKould added a commit to KKould/KipSQL that referenced this pull request Nov 11, 2023
…, `is not null` (KipData#93)

* refactor(column_pruning): reconstruct column clipping using Column detection under self-direction 1

* feat(simplify): add `constant_calculation` compute constant calculations ahead of time to avoid double calculations

* feat: support like operator

* feat: support is not null operator

* ci: nightly version

* config: nightly version update

* ci: config toolchain
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants