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();