[
https://issues.apache.org/jira/browse/DRILL-7729?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17272442#comment-17272442
]
ASF GitHub Bot commented on DRILL-7729:
---------------------------------------
paul-rogers commented on pull request #2076:
URL: https://github.com/apache/drill/pull/2076#issuecomment-767847071
@luocooong, thanks for the reminder. Looks like I originally factored out
the Java time stuff for its own review. Got no takers, it seems, and I forgot
about it. So, when @cgivre asked me to finish the streaming JSON stuff, I just
included the time stuff as well.
If the team prefers, I can pull the Java time stuff out of the streaming
JSON PR and replace the code here with the updated version. Or, if the team is
OK with the single big PR, we can keep it in the streaming JSON PR.
How would you like to proceed?
----------------------------------------------------------------
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]
> Use java.time in column accessors
> ---------------------------------
>
> Key: DRILL-7729
> URL: https://issues.apache.org/jira/browse/DRILL-7729
> Project: Apache Drill
> Issue Type: Improvement
> Affects Versions: 1.17.0
> Reporter: Paul Rogers
> Assignee: Paul Rogers
> Priority: Major
> Fix For: 1.19.0
>
>
> Use {{java.time}} classes in the column accessors, except for {{Interval}},
> which has no {{java.time}} equivalent. Doing so allows us to create a row-set
> version of Drill's JSON writer.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
