You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by ao...@apache.org on 2020/10/29 18:14:39 UTC

[iceberg] branch master updated: Core: Fix ArrayIndexOutOfBounds for tables with migrated partition specs (#1676)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 3406e26  Core: Fix ArrayIndexOutOfBounds for tables with migrated partition specs (#1676)
3406e26 is described below

commit 3406e2683f1bd8b1abadd774f10f26eaa13d5b45
Author: Ryan Blue <bl...@apache.org>
AuthorDate: Thu Oct 29 11:14:30 2020 -0700

    Core: Fix ArrayIndexOutOfBounds for tables with migrated partition specs (#1676)
---
 core/src/main/java/org/apache/iceberg/ManifestsTable.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/java/org/apache/iceberg/ManifestsTable.java b/core/src/main/java/org/apache/iceberg/ManifestsTable.java
index ef442ee..35b6fe7 100644
--- a/core/src/main/java/org/apache/iceberg/ManifestsTable.java
+++ b/core/src/main/java/org/apache/iceberg/ManifestsTable.java
@@ -114,7 +114,7 @@ public class ManifestsTable extends BaseMetadataTable {
 
     List<StaticDataTask.Row> rows = Lists.newArrayList();
 
-    for (int i = 0; i < spec.fields().size(); i += 1) {
+    for (int i = 0; i < summaries.size(); i += 1) {
       ManifestFile.PartitionFieldSummary summary = summaries.get(i);
       rows.add(StaticDataTask.Row.of(
           summary.containsNull(),