DanCodedThis commented on issue #6876:
URL: https://github.com/apache/datafusion/issues/6876#issuecomment-2621860544

   @edmondop I think I agree with your statement. 
   - Also, does the new `TypeSignatureClass` + `logical_string()` allow parsing 
`month` as not a column/keyword, but as `Utf8` without the single quotes? In 
our own implementation, there is a preprocessing that fixes that in a hacky 
way. If we later add the Snowflake way to DF, I don't think I will be able to 
reproduce it here.
   - I saw the new `date_part` two weeks/week and a half ago on the Apache 
docs, and they were using `month` not `'month'` in the example, perhaps it was 
a mistake in the docs that was fixed, or I didn't perceive it correctly, I 
could swear I saw it.
   - I tried doing something similar for the aforementioned `date_diff`, but 
the example that I was referring too was gone by that time I wrote it and 
tested it. (didn't work)


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to