Skip to content
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

Adding support for much more permissive date parsing for ExprTimestam… - #1118

Open
FreCap wants to merge 1 commit into
opendistro-for-elasticsearch:developfrom
FreCap:add-support-timestamp-without-t
Open

Adding support for much more permissive date parsing for ExprTimestam…#1118
FreCap wants to merge 1 commit into
opendistro-for-elasticsearch:developfrom
FreCap:add-support-timestamp-without-t

Conversation

@FreCap

@FreCap FreCap commented Jun 12, 2021

Copy link
Copy Markdown
Contributor

Adding support for much more permissive date parsing for ExprTimestampValue creation (support of T or not T, autofill missing part with 0s)

Issue #, if available:

  1. after you add a date in format "yyyy-MM-dd HH:mm:ss.ZZZZZZ" (parseable by bare ES)
  2. when "SELECT * FROM XYZ" it fails

These other formats instead are already supported
"yyyy-MM-dd'T'HH:mm:ss.ZZZZZZ"
"yyyy-MM-dd HH:mm:ss"

so I'm filling the gap

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

…pValue creation (support of T or not T, autofill missing part with 0s)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant