parquet-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ziva...@apache.org
Subject [parquet-format] branch master updated: PARQUET-1266: LogicalTypes union in parquet-format doesn't include UUID
Date Mon, 14 May 2018 16:28:00 GMT
This is an automated email from the ASF dual-hosted git repository.

zivanfi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/parquet-format.git


The following commit(s) were added to refs/heads/master by this push:
     new 3e6cd14  PARQUET-1266: LogicalTypes union in parquet-format doesn't include UUID
3e6cd14 is described below

commit 3e6cd1415eabdd8ec93881a5c89ae82fb150ab29
Author: Nandor Kollar <nkollar@cloudera.com>
AuthorDate: Thu Apr 5 10:01:42 2018 +0200

    PARQUET-1266: LogicalTypes union in parquet-format doesn't include UUID
---
 src/main/thrift/parquet.thrift | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/main/thrift/parquet.thrift b/src/main/thrift/parquet.thrift
index f3aac25..3b15cfe 100644
--- a/src/main/thrift/parquet.thrift
+++ b/src/main/thrift/parquet.thrift
@@ -318,7 +318,7 @@ struct BsonType {
  * following table.
  */
 union LogicalType {
-  1:  StringType STRING       // use ConvertedType UTF8 if encoding is UTF-8
+  1:  StringType STRING       // use ConvertedType UTF8
   2:  MapType MAP             // use ConvertedType MAP
   3:  ListType LIST           // use ConvertedType LIST
   4:  EnumType ENUM           // use ConvertedType ENUM
@@ -331,6 +331,7 @@ union LogicalType {
   11: NullType UNKNOWN        // no compatible ConvertedType
   12: JsonType JSON           // use ConvertedType JSON
   13: BsonType BSON           // use ConvertedType BSON
+  14: UUIDType UUID
 }
 
 /**
@@ -381,7 +382,7 @@ struct SchemaElement {
   9: optional i32 field_id;
 
   /**
-   * The logical type of this SchemaElement; only valid for primitives.
+   * The logical type of this SchemaElement
    *
    * LogicalType replaces ConvertedType, but ConvertedType is still required
    * for some logical types to ensure forward-compatibility in format v1.

-- 
To stop receiving notification emails like this one, please contact
zivanfi@apache.org.

Mime
View raw message