[ 
https://issues.apache.org/jira/browse/CALCITE-5357?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17693090#comment-17693090
 ] 

Julian Hyde commented on CALCITE-5357:
--------------------------------------

[~tjbanghart], I have reviewed your PR 3034 and it looks good. I have made some 
further edits - fix a threading issue by introducing a ThreadLocal, enable the 
relevant tests in big-query.iq, convert FormatModel into an interface, convert 
FormatModels into a utility class, and use SqlBasicFunction instead of 
SqlBigQueryFormatDatetimeFuncton - which you can see in 
[julianhyde/2980-cast-format|https://github.com/julianhyde/calcite/tree/2980-cast-format].
 If those changes look OK I'll squash, rebase, merge.

> Implement BigQuery FORMAT_TIMESTAMP, FORMAT_DATE
> ------------------------------------------------
>
>                 Key: CALCITE-5357
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5357
>             Project: Calcite
>          Issue Type: Sub-task
>          Components: core
>            Reporter: TJ Banghart
>            Assignee: TJ Banghart
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 6h
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to