morningman commented on a change in pull request #4495:
URL: https://github.com/apache/incubator-doris/pull/4495#discussion_r481747384
##########
File path: fe/fe-core/src/main/java/org/apache/doris/analysis/DateLiteral.java
##########
@@ -632,4 +669,435 @@ public long getSecond() {
public int hashCode() {
return 31 * super.hashCode() +
Objects.hashCode(unixTimestamp(TimeZone.getDefault()));
}
+
+ // parset the date string value in 'value' by 'format' pattern.
+ // return the next position to parse if hasSubVal is true.
+ // throw InvalidFormatException if encounter errors.
+ // this method is exaclty same as from_date_format_str() in
be/src/runtime/datetime_value.cpp
+ // change this method should also change that.
+ public int fromDateFormatStr(String format, String value, boolean
hasSubVal) throws InvalidFormatException {
Review comment:
It is now only used for DateLiteral, and this method is also in
`datetime_value.cpp` in BE.
So I think we can just leave it here.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
