You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by bl...@apache.org on 2019/11/06 18:39:20 UTC

[incubator-iceberg] branch master updated: Fix PreconditionsInvalidPlaceholder error (#615)

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

blue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-iceberg.git


The following commit(s) were added to refs/heads/master by this push:
     new 1f100bd  Fix PreconditionsInvalidPlaceholder error (#615)
1f100bd is described below

commit 1f100bdddca7e56a250b6ef93292d2433fefd880
Author: Fokko Driesprong <fo...@apache.org>
AuthorDate: Wed Nov 6 19:39:10 2019 +0100

    Fix PreconditionsInvalidPlaceholder error (#615)
---
 api/src/main/java/org/apache/iceberg/PartitionSpec.java        | 2 +-
 core/src/main/java/org/apache/iceberg/TableMetadataParser.java | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/api/src/main/java/org/apache/iceberg/PartitionSpec.java b/api/src/main/java/org/apache/iceberg/PartitionSpec.java
index 15979c7..e839774 100644
--- a/api/src/main/java/org/apache/iceberg/PartitionSpec.java
+++ b/api/src/main/java/org/apache/iceberg/PartitionSpec.java
@@ -465,7 +465,7 @@ public class PartitionSpec implements Serializable {
     Builder add(int sourceId, String name, String transform) {
       Types.NestedField column = schema.findField(sourceId);
       checkAndAddPartitionName(name, column.fieldId());
-      Preconditions.checkNotNull(column, "Cannot find source column: %d", sourceId);
+      Preconditions.checkNotNull(column, "Cannot find source column: %s", sourceId);
       fields.add(new PartitionField(sourceId, name, Transforms.fromString(column.type(), transform)));
       return this;
     }
diff --git a/core/src/main/java/org/apache/iceberg/TableMetadataParser.java b/core/src/main/java/org/apache/iceberg/TableMetadataParser.java
index 8feaa8a..6f73f3b 100644
--- a/core/src/main/java/org/apache/iceberg/TableMetadataParser.java
+++ b/core/src/main/java/org/apache/iceberg/TableMetadataParser.java
@@ -209,7 +209,7 @@ public class TableMetadataParser {
 
     int formatVersion = JsonUtil.getInt(FORMAT_VERSION, node);
     Preconditions.checkArgument(formatVersion == TableMetadata.TABLE_FORMAT_VERSION,
-        "Cannot read unsupported version %d", formatVersion);
+        "Cannot read unsupported version %s", formatVersion);
 
     String uuid = JsonUtil.getStringOrNull(TABLE_UUID, node);
     String location = JsonUtil.getString(LOCATION, node);