New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARROW-13916: [C++] Implement strftime on date32/64 types #11219
Conversation
c4cc286
to
e50eb84
Compare
@@ -566,6 +566,14 @@ TEST_F(ScalarTemporalTest, Strftime) { | |||
utf8(), string_microseconds, &options); | |||
CheckScalarUnary("strftime", timestamp(TimeUnit::NANO, "US/Hawaii"), nanoseconds, | |||
utf8(), string_nanoseconds, &options); | |||
|
|||
auto options2 = StrftimeOptions("%Y-%m-%d"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add a test with formatting hours/minutes/seconds? (I suppose those will just work, with all 0?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will add. I suspect so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
21c0bad
to
23b6bd2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you.
This is to resolve [ARROW-13916](https://issues.apache.org/jira/browse/ARROW-13916). Closes apache#11219 from rok/ARROW-13916 Authored-by: Rok <rok@mihevc.org> Signed-off-by: Antoine Pitrou <antoine@python.org>
This is to resolve ARROW-13916.