LeonxLJX commented on issue #43847: URL: https://github.com/apache/superset/issues/43847#issuecomment-5536289695
I'd like to take this one. The bug: digits-only date strings like '2017' or '20260903' are being parsed as epoch milliseconds, so '2017-01-01' is treated as 1970-01-01T00:00:02.017 instead of 2017-01-01. Root cause is most likely in dateutil.parser / arrow's fuzzy parser matching the 4-digit-year regex before the ISO-8601 regex. **Alternatives I considered:** (a) require a separator for short year strings (smallest fix), (b) drop the 4-digit shortcut entirely (cleanest, IMO), (c) document the workaround. **Suggestion to maintainer:** which date parser library is in the path here (dateutil? arrow? or Superset's own parser)? I want to confirm the parser before drafting a fix. (claiming via @LeonxLJX) -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
