-
Notifications
You must be signed in to change notification settings - Fork 186
add support for HH:mm:ss #850
add support for HH:mm:ss #850
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #850 +/- ##
==========================================
Coverage 99.85% 99.85%
- Complexity 2102 2115 +13
==========================================
Files 209 214 +5
Lines 4678 4738 +60
Branches 308 308
==========================================
+ Hits 4671 4731 +60
Misses 5 5
Partials 2 2 Continue to review full report at Codecov.
|
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.
LGTM, thanks for the fix!
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.
Please add UT to cover the STRICT_DATE_FORMATTER
...n/opendistroforelasticsearch/sql/elasticsearch/data/value/ElasticsearchExprValueFactory.java
Outdated
Show resolved
Hide resolved
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.
Thanks for the fix!
* add STRICT_DATE_FORMATTER * fix chekstyle error * update for HH:MM:SS Co-authored-by: Rupal Mahajan <>
Issue: #849
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.