parquet-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From l...@apache.org
Subject parquet-mr git commit: PARQUET-495: Fix mismatches in Types class comments
Date Mon, 01 Feb 2016 18:09:19 GMT
Repository: parquet-mr
Updated Branches:
  refs/heads/master 57694790f -> 63d5ae78a


PARQUET-495: Fix mismatches in Types class comments

To produce
> required group User {
    required int64 id;
    **optional** binary email (UTF8);
 }

we should do:
>
Types.requiredGroup()
      .required(INT64).named("id")
      .~~**required** (BINARY).as(UTF8).named("email")~~
      .**optional** (BINARY).as(UTF8).named("email")
      .named("User")

@rdblue @liancheng would you mind taking a look at it when you have time? Thanks!

Author: Liwei Lin <proflin.me@gmail.com>
Author: proflin <proflin.me@gmail.com>

Closes #317 from proflin/PARQUET-495--Fix-mismatches-in-Types-class-comments and squashes
the following commits:

f26d57d [Liwei Lin] PARQUET-495: Fix mismatches in Types class comments
839b458 [proflin] Merge remote-tracking branch 'refs/remotes/apache/master'


Project: http://git-wip-us.apache.org/repos/asf/parquet-mr/repo
Commit: http://git-wip-us.apache.org/repos/asf/parquet-mr/commit/63d5ae78
Tree: http://git-wip-us.apache.org/repos/asf/parquet-mr/tree/63d5ae78
Diff: http://git-wip-us.apache.org/repos/asf/parquet-mr/diff/63d5ae78

Branch: refs/heads/master
Commit: 63d5ae78ac00d710c9bf631f8cb9ed6d07e6a2c9
Parents: 5769479
Author: Liwei Lin <proflin.me@gmail.com>
Authored: Mon Feb 1 10:09:20 2016 -0800
Committer: Cheng Lian <lian@databricks.com>
Committed: Mon Feb 1 10:09:20 2016 -0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/parquet/schema/Types.java         | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/parquet-mr/blob/63d5ae78/parquet-column/src/main/java/org/apache/parquet/schema/Types.java
----------------------------------------------------------------------
diff --git a/parquet-column/src/main/java/org/apache/parquet/schema/Types.java b/parquet-column/src/main/java/org/apache/parquet/schema/Types.java
index 4e7e881..6d61b50 100644
--- a/parquet-column/src/main/java/org/apache/parquet/schema/Types.java
+++ b/parquet-column/src/main/java/org/apache/parquet/schema/Types.java
@@ -59,7 +59,7 @@ import org.apache.parquet.schema.Type.ID;
  *   // }
  *   Types.requiredGroup()
  *            .required(INT64).named("id")
- *            .required(BINARY).as(UTF8).named("email")
+ *            .optional(BINARY).as(UTF8).named("email")
  *        .named("User")
  * </pre>
  * <p>
@@ -82,7 +82,7 @@ import org.apache.parquet.schema.Type.ID;
  *   // }
  *   Types.requiredGroup()
  *            .required(INT64).named("id")
- *            .required(BINARY).as(UTF8).named("email")
+ *            .optional(BINARY).as(UTF8).named("email")
  *            .optionalGroup()
  *                .required(BINARY).as(UTF8).named("street")
  *                .required(INT32).named("zipcode")
@@ -161,7 +161,7 @@ import org.apache.parquet.schema.Type.ID;
  *   // }
  *   Types.buildMessage()
  *            .required(INT64).named("id")
- *            .required(BINARY).as(UTF8).named("email")
+ *            .optional(BINARY).as(UTF8).named("email")
  *            .optionalGroup()
  *                .required(BINARY).as(UTF8).named("street")
  *                .required(INT32).named("zipcode")


Mime
View raw message