You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "yevgenypats (via GitHub)" <gi...@apache.org> on 2023/05/22 17:30:15 UTC

[GitHub] [arrow] yevgenypats commented on pull request #35687: GH-35686:[Go] Add AppendTime to TimestampBuilder

yevgenypats commented on PR #35687:
URL: https://github.com/apache/arrow/pull/35687#issuecomment-1557621256

   > @yevgenypats Do we want to expand the tests to test more units than just `Second`? Or do you think we're fine with just testing `Second`?
   
   I can add but maybe we want to move TimestampBuilder out of the template as it becomes quite hard to develop inside a template and Timestamp starts to diverge from all other numeric types. wdyt?


-- 
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-unsubscribe@arrow.apache.org

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