New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(interactive): Support DateFormats in GIE Runtime #3197
Conversation
9084c62
to
2eaf0dd
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3197 +/- ##
===========================================
- Coverage 72.55% 42.05% -30.50%
===========================================
Files 102 101 -1
Lines 11086 10985 -101
===========================================
- Hits 8043 4620 -3423
- Misses 3043 6365 +3322 see 63 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
@@ -37,6 +38,10 @@ pub enum RawType { | |||
ULLong, | |||
Float, | |||
String, | |||
DateFormat, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Date
instead of DateFormat
, and applies to the follows.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
@@ -409,13 +414,194 @@ impl PartialOrd for Primitives { | |||
} | |||
} | |||
|
|||
#[derive(Clone, Debug, Hash)] | |||
pub enum DateFormats { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seem like should be DateTimeFormats
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
<!-- Thanks for your contribution! please review https://github.com/alibaba/GraphScope/blob/main/CONTRIBUTING.md before opening an issue. --> ## What do these changes do? <!-- Please give a short brief about these changes. --> As titled. ## Related issue number <!-- Are there any issues opened that will be resolved by merging this change? --> Fixes alibaba#3179 Committed-by: Hongyi ZHANG from Dev container
What do these changes do?
As titled.
Related issue number
Fixes #3179