rok commented on a change in pull request #11026:
URL: https://github.com/apache/arrow/pull/11026#discussion_r709400411



##########
File path: cpp/src/arrow/compute/kernels/scalar_temporal.cc
##########
@@ -1006,6 +1080,15 @@ const FunctionDoc iso_week_doc{
      "cannot be found in the timezone database."),
     {"values"}};
 
+const FunctionDoc week_doc{
+    "Extract week of year number",
+    ("First week has the majority (4 or more) of its days in January.\n"
+     "Year can have 52 or 53 weeks. Week numbering can start with 0 or 1 using 
"
+     "DayOfWeekOptions.one_based_numbering.\n"
+     "Returns an error if timestamp has a defined timezone. Null values return 
null."),

Review comment:
       Leftover from before this would generally be possible. Removed.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to