This is an automated email from the ASF dual-hosted git repository.

wesm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 10a571b  ARROW-5545: [C++][Docs] Clarify expectation of UTC values for 
timestamps with time zones
10a571b is described below

commit 10a571b4334da95f00411a64959170fd08e0dba1
Author: TP Boudreau <tpboudr...@gmail.com>
AuthorDate: Fri Jun 14 07:59:40 2019 -0500

    ARROW-5545: [C++][Docs] Clarify expectation of UTC values for timestamps 
with time zones
    
    Doxygen comments only.  No code changes.
    
    Author: TP Boudreau <tpboudr...@gmail.com>
    
    Closes #4555 from tpboudreau/ARROW-5545 and squashes the following commits:
    
    305b0d84d <TP Boudreau> Add comments for three temporal DataTypes
---
 cpp/src/arrow/type.h | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/cpp/src/arrow/type.h b/cpp/src/arrow/type.h
index 98d2e4e..b581739 100644
--- a/cpp/src/arrow/type.h
+++ b/cpp/src/arrow/type.h
@@ -769,6 +769,8 @@ class ARROW_EXPORT TimeType : public TemporalType, public 
ParametricType {
   TimeUnit::type unit_;
 };
 
+/// Concrete type class for 32-bit time data (as number of seconds or 
milliseconds
+/// since midnight)
 class ARROW_EXPORT Time32Type : public TimeType {
  public:
   static constexpr Type::type type_id = Type::TIME32;
@@ -783,6 +785,8 @@ class ARROW_EXPORT Time32Type : public TimeType {
   std::string name() const override { return "time32"; }
 };
 
+/// Concrete type class for 64-bit time data (as number of microseconds or 
nanoseconds
+/// since midnight)
 class ARROW_EXPORT Time64Type : public TimeType {
  public:
   static constexpr Type::type type_id = Type::TIME64;
@@ -797,6 +801,38 @@ class ARROW_EXPORT Time64Type : public TimeType {
   std::string name() const override { return "time64"; }
 };
 
+/// \brief Concrete type class for datetime data (as number of seconds, 
milliseconds,
+/// microseconds or nanoseconds since UNIX epoch)
+///
+/// If supplied, the timezone string should take either the form (i) 
"Area/Location",
+/// with values drawn from the names in the IANA Time Zone Database (such as
+/// "Europe/Zurich"); or (ii) "(+|-)HH:MM" indicating an absolute offset from 
GMT
+/// (such as "-08:00").  To indicate a native UTC timestamp, one of the 
strings "UTC",
+/// "Etc/UTC" or "+00:00" should be used.
+///
+/// If any non-empty string is supplied as the timezone for a TimestampType, 
then the
+/// Arrow field containing that timestamp type (and by extension the column 
associated
+/// with such a field) is considered "timezone-aware".  The integer arrays 
that comprise
+/// a timezone-aware column must contain UTC normalized datetime values, 
regardless of
+/// the contents of their timezone string.  More precisely, (i) the producer 
of a
+/// timezone-aware column must populate its constituent arrays with valid UTC 
values
+/// (performing offset conversions from non-UTC values if necessary); and (ii) 
the
+/// consumer of a timezone-aware column may assume that the column's values 
are directly
+/// comparable (that is, with no offset adjustment required) to the values of 
any other
+/// timezone-aware column or to any other valid UTC datetime value (provided 
all values
+/// are expressed in the same units).
+///
+/// If a TimestampType is constructed without a timezone (or, equivalently, if 
the
+/// timezone supplied is an empty string) then the resulting Arrow field 
(column) is
+/// considered "timezone-naive".  The producer of a timezone-naive column may 
populate
+/// its constituent integer arrays with datetime values from any timezone; the 
consumer
+/// of a timezone-naive column should make no assumptions about the 
interoperability or
+/// comparability of the values of such a column with those of any other 
timestamp
+/// column or datetime value.
+///
+/// If a timezone-aware field contains a recognized timezone, its values may be
+/// localized to that locale upon display; the values of timezone-naive fields 
must
+/// always be displayed "as is", with no localization performed on them.
 class ARROW_EXPORT TimestampType : public TemporalType, public ParametricType {
  public:
   using Unit = TimeUnit;

Reply via email to