Skip to content

Comments

fix: protect in NormalizeColumns in ddl parser#3196

Merged
vagetablechicken merged 3 commits into4paradigm:mainfrom
vagetablechicken:main-ddl
Apr 17, 2023
Merged

fix: protect in NormalizeColumns in ddl parser#3196
vagetablechicken merged 3 commits into4paradigm:mainfrom
vagetablechicken:main-ddl

Conversation

@vagetablechicken
Copy link
Collaborator

@vagetablechicken vagetablechicken commented Apr 6, 2023

join on t1.c1=t2.c2 and date(t1.c2)=date(t2.c2);

  • add a join test in ddl parser test

@github-actions github-actions bot added the storage-engine openmldb storage engine. nameserver & tablet label Apr 6, 2023
@codecov
Copy link

codecov bot commented Apr 6, 2023

Codecov Report

Patch coverage: 93.33% and project coverage change: -3.52 ⚠️

Comparison is base (960c553) 79.12% compared to head (c59201a) 75.61%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #3196      +/-   ##
============================================
- Coverage     79.12%   75.61%   -3.52%     
- Complexity        0      393     +393     
============================================
  Files           514      678     +164     
  Lines        115795   125400    +9605     
  Branches          0     1181    +1181     
============================================
+ Hits          91626    94815    +3189     
- Misses        24169    30349    +6180     
- Partials          0      236     +236     
Impacted Files Coverage Δ
src/base/ddl_parser.cc 75.64% <77.77%> (+0.64%) ⬆️
src/base/ddl_parser_test.cc 99.54% <100.00%> (+0.02%) ⬆️

... and 205 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@vagetablechicken vagetablechicken merged commit 2e29972 into 4paradigm:main Apr 17, 2023
@vagetablechicken vagetablechicken deleted the main-ddl branch April 17, 2023 09:51
@lumianph lumianph added the alpha this feature is not fully tested or functional label Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alpha this feature is not fully tested or functional storage-engine openmldb storage engine. nameserver & tablet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants