You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by dw...@apache.org on 2022/05/16 18:09:15 UTC

[iceberg] branch master updated: Remove redundant assignment (#4755)

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

dweeks 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 c7bd72771 Remove redundant assignment (#4755)
c7bd72771 is described below

commit c7bd7277116344fc2638f15e59285386083e2f50
Author: Piotr Findeisen <pi...@gmail.com>
AuthorDate: Mon May 16 20:09:08 2022 +0200

    Remove redundant assignment (#4755)
    
    `refresh()` updates `this.base`.
---
 core/src/main/java/org/apache/iceberg/SnapshotProducer.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/java/org/apache/iceberg/SnapshotProducer.java b/core/src/main/java/org/apache/iceberg/SnapshotProducer.java
index 08a61b39f..0f7eed5e4 100644
--- a/core/src/main/java/org/apache/iceberg/SnapshotProducer.java
+++ b/core/src/main/java/org/apache/iceberg/SnapshotProducer.java
@@ -166,7 +166,7 @@ abstract class SnapshotProducer<ThisT> implements SnapshotUpdate<ThisT> {
 
   @Override
   public Snapshot apply() {
-    this.base = refresh();
+    refresh();
     Long parentSnapshotId = base.currentSnapshot() != null ?
         base.currentSnapshot().snapshotId() : null;
     long sequenceNumber = base.nextSequenceNumber();