Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
228 changes: 228 additions & 0 deletions table/changelog_scan_task.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,228 @@
// Licensed to the Apache Software Foundation (ASF) under one
// or more contributor license agreements. See the NOTICE file
// distributed with this work for additional information
// regarding copyright ownership. The ASF licenses this file
// to you under the Apache License, Version 2.0 (the
// "License"); you may not use this file except in compliance
// with the License. You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing,
// software distributed under the License is distributed on an
// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.

package table

import (
"fmt"

"github.com/apache/iceberg-go"
)

// ChangelogOperation is the kind of change a changelog scan task produces.
type ChangelogOperation string

const (
ChangelogOpInsert ChangelogOperation = "INSERT"
ChangelogOpDelete ChangelogOperation = "DELETE"
ChangelogOpUpdateBefore ChangelogOperation = "UPDATE_BEFORE"
ChangelogOpUpdateAfter ChangelogOperation = "UPDATE_AFTER"
)

// ChangelogScanTask is a unit of work that produces changelog rows.
type ChangelogScanTask interface {
Operation() ChangelogOperation
ChangeOrdinal() int
CommitSnapshotID() int64
}

var (
_ ChangelogScanTask = AddedRowsScanTask{}
_ ChangelogScanTask = DeletedDataFileScanTask{}
_ ChangelogScanTask = DeletedRowsScanTask{}
)

// classifiedDeletes holds delete files split the same way FileScanTask does,
// without a second FileScanTask whose range and lineage fields would be zero.
type classifiedDeletes struct {
pos, eq, dv []iceberg.DataFile
}

func (d classifiedDeletes) files() []iceberg.DataFile {
out := make([]iceberg.DataFile, 0, len(d.pos)+len(d.eq)+len(d.dv))
out = append(out, d.pos...)
out = append(out, d.eq...)
out = append(out, d.dv...)

return out
}

// AddedRowsScanTask is a changelog insert produced by adding a data file.
// Matching delete files committed in the same snapshot, or from squashed
// snapshots, are applied while reading so deleted rows are not emitted as
// inserts.
type AddedRowsScanTask struct {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Design-direction question for this first slice: Java has these three implement a common ChangelogScanTask interface with operation(), changeOrdinal(), commitSnapshotId(). Here each type carries ChangeOrdinal/CommitSnapshotID but there's no shared interface and no Operation().

Without it the planning follow-up can't return a uniform []ChangelogScanTask and every consumer needs a type switch to tell inserts from deletes. I'd lean toward defining the interface plus a ChangelogOperation enum now so the follow-ups have something to build against, but if you'd rather defer until planning lands that's reasonable too. wdyt?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call — I added ChangelogScanTask with Operation(), ChangeOrdinal(), and CommitSnapshotID(), plus a ChangelogOperation enum matching Java (INSERT / DELETE / UPDATE_BEFORE / UPDATE_AFTER). The three task types implement it so the planning follow-up can return []ChangelogScanTask without a type switch just to tell inserts from deletes.

FileScanTask
changeOrdinal int
commitSnapshotID int64
}

// NewAddedRowsScanTask constructs an insert task for dataFile. deletes are
// delete files that apply while reading the added file. Position deletes,
// equality deletes, and deletion vectors are stored on the matching
// FileScanTask fields.
func NewAddedRowsScanTask(dataFile iceberg.DataFile, deletes []iceberg.DataFile, changeOrdinal int, commitSnapshotID int64) (AddedRowsScanTask, error) {
task, err := fileScanTaskWithDeletes(dataFile, deletes)
if err != nil {
return AddedRowsScanTask{}, err
}

return AddedRowsScanTask{
FileScanTask: task,
changeOrdinal: changeOrdinal,
commitSnapshotID: commitSnapshotID,
}, nil
}

func (t AddedRowsScanTask) Operation() ChangelogOperation { return ChangelogOpInsert }
func (t AddedRowsScanTask) ChangeOrdinal() int { return t.changeOrdinal }
func (t AddedRowsScanTask) CommitSnapshotID() int64 { return t.commitSnapshotID }

// Deletes returns every delete file applied while reading the added data
// file: position deletes, then equality deletes, then deletion vectors.
func (t AddedRowsScanTask) Deletes() []iceberg.DataFile {
return allDeleteFiles(t.FileScanTask)
}

// DeletedDataFileScanTask is a changelog delete produced by removing a data
// file. ExistingDeletes are delete files that were already present and must
// be applied so only rows that were live when the file was removed appear as
// deletes.
type DeletedDataFileScanTask struct {
FileScanTask
changeOrdinal int
commitSnapshotID int64
}

// NewDeletedDataFileScanTask constructs a delete task for a removed data file.
func NewDeletedDataFileScanTask(dataFile iceberg.DataFile, existingDeletes []iceberg.DataFile, changeOrdinal int, commitSnapshotID int64) (DeletedDataFileScanTask, error) {
task, err := fileScanTaskWithDeletes(dataFile, existingDeletes)
if err != nil {
return DeletedDataFileScanTask{}, err
}

return DeletedDataFileScanTask{
FileScanTask: task,
changeOrdinal: changeOrdinal,
commitSnapshotID: commitSnapshotID,
}, nil
}

func (t DeletedDataFileScanTask) Operation() ChangelogOperation { return ChangelogOpDelete }
func (t DeletedDataFileScanTask) ChangeOrdinal() int { return t.changeOrdinal }
func (t DeletedDataFileScanTask) CommitSnapshotID() int64 { return t.commitSnapshotID }

// ExistingDeletes returns delete files that applied before the data file was
// removed.
func (t DeletedDataFileScanTask) ExistingDeletes() []iceberg.DataFile {
return allDeleteFiles(t.FileScanTask)
}

// DeletedRowsScanTask is a changelog delete produced by adding delete files
// against a data file that remains in the table. AddedDeletes remove rows
// that should appear in the changelog. ExistingDeletes already applied and
// those rows must not be emitted again.
type DeletedRowsScanTask struct {
FileScanTask
addedDeletes classifiedDeletes
changeOrdinal int
commitSnapshotID int64
}

// NewDeletedRowsScanTask constructs a row-level delete task. existingDeletes
// are stored on the embedded FileScanTask so later readers can reuse the
// normal scan delete path for the live-row baseline.
func NewDeletedRowsScanTask(dataFile iceberg.DataFile, addedDeletes, existingDeletes []iceberg.DataFile, changeOrdinal int, commitSnapshotID int64) (DeletedRowsScanTask, error) {
existing, err := fileScanTaskWithDeletes(dataFile, existingDeletes)
if err != nil {
return DeletedRowsScanTask{}, err
}

added, err := classifyDeleteFiles(addedDeletes)
if err != nil {
return DeletedRowsScanTask{}, err
}

return DeletedRowsScanTask{
FileScanTask: existing,
addedDeletes: added,
changeOrdinal: changeOrdinal,
commitSnapshotID: commitSnapshotID,
}, nil
}

func (t DeletedRowsScanTask) Operation() ChangelogOperation { return ChangelogOpDelete }
func (t DeletedRowsScanTask) ChangeOrdinal() int { return t.changeOrdinal }
func (t DeletedRowsScanTask) CommitSnapshotID() int64 { return t.commitSnapshotID }

// AddedDeletes returns delete files whose removals should appear in the
// changelog.
func (t DeletedRowsScanTask) AddedDeletes() []iceberg.DataFile {
return t.addedDeletes.files()
}

// ExistingDeletes returns delete files that already applied before this
// snapshot's added deletes.
func (t DeletedRowsScanTask) ExistingDeletes() []iceberg.DataFile {
return allDeleteFiles(t.FileScanTask)
}

func fileScanTaskWithDeletes(dataFile iceberg.DataFile, deletes []iceberg.DataFile) (FileScanTask, error) {
classified, err := classifyDeleteFiles(deletes)
if err != nil {
return FileScanTask{}, err
}

return FileScanTask{
File: dataFile,
DeleteFiles: classified.pos,
EqualityDeleteFiles: classified.eq,
DeletionVectorFiles: classified.dv,
}, nil
}

func classifyDeleteFiles(files []iceberg.DataFile) (classifiedDeletes, error) {
var out classifiedDeletes
for _, f := range files {
kind, err := classifyDataFile(f)
if err != nil {
return classifiedDeletes{}, err
}

switch kind {
case dataFileKindPosDeletes:
out.pos = append(out.pos, f)
case dataFileKindEqDeletes:
out.eq = append(out.eq, f)
case dataFileKindDeletionVector:
out.dv = append(out.dv, f)
default:
return classifiedDeletes{}, fmt.Errorf("%w: expected delete file, got content type %s",
ErrInvalidMetadata, f.ContentType())
}
}

return out, nil
}

func allDeleteFiles(task FileScanTask) []iceberg.DataFile {
return classifiedDeletes{
pos: task.DeleteFiles,
eq: task.EqualityDeleteFiles,
dv: task.DeletionVectorFiles,
}.files()
}
118 changes: 118 additions & 0 deletions table/changelog_scan_task_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,118 @@
// Licensed to the Apache Software Foundation (ASF) under one
// or more contributor license agreements. See the NOTICE file
// distributed with this work for additional information
// regarding copyright ownership. The ASF licenses this file
// to you under the Apache License, Version 2.0 (the
// "License"); you may not use this file except in compliance
// with the License. You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing,
// software distributed under the License is distributed on an
// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.

package table

import (
"testing"

"github.com/apache/iceberg-go"
"github.com/stretchr/testify/require"
)

func changelogTestDataFile(t *testing.T, path string, content iceberg.ManifestEntryContent, format iceberg.FileFormat) iceberg.DataFile {
t.Helper()

b, err := iceberg.NewDataFileBuilder(*iceberg.UnpartitionedSpec,
content, path, format, nil, nil, nil, 10, 1024)
require.NoError(t, err)

return b.Build()
}

func TestAddedRowsScanTaskAppliesSameSnapshotDeletes(t *testing.T) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These three tests only exercise the happy path: every input is a cleanly-typed delete file. There's no case for the branch that matters most, a plain data file (or nil) in the deletes slice, which today is silently dropped. If that path becomes an error (per the classifyDeleteFiles comment), I'd want a test asserting the error; if it stays a skip, a test that documents the intent.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added TestClassifyDeleteFiles for the pos/eq/dv split and for a data file in the deletes slice, which now errors with ErrInvalidMetadata.

data := changelogTestDataFile(t, "data/f1.parquet", iceberg.EntryContentData, iceberg.ParquetFile)
posDel := changelogTestDataFile(t, "deletes/d1.parquet", iceberg.EntryContentPosDeletes, iceberg.ParquetFile)
eqDel := changelogTestDataFile(t, "deletes/d2.parquet", iceberg.EntryContentEqDeletes, iceberg.ParquetFile)
dv := changelogTestDataFile(t, "deletes/d3.puffin", iceberg.EntryContentPosDeletes, iceberg.PuffinFile)

task, err := NewAddedRowsScanTask(data, []iceberg.DataFile{eqDel, dv, posDel}, 0, 42)
require.NoError(t, err)

require.Equal(t, ChangelogOpInsert, task.Operation())
require.Equal(t, 0, task.ChangeOrdinal())
require.Equal(t, int64(42), task.CommitSnapshotID())
require.Equal(t, data.FilePath(), task.File.FilePath())
require.Equal(t, []iceberg.DataFile{posDel}, task.DeleteFiles)
require.Equal(t, []iceberg.DataFile{eqDel}, task.EqualityDeleteFiles)
require.Equal(t, []iceberg.DataFile{dv}, task.DeletionVectorFiles)
require.Equal(t, []iceberg.DataFile{posDel, eqDel, dv}, task.Deletes())
}

func TestDeletedDataFileScanTaskKeepsExistingDeletes(t *testing.T) {
data := changelogTestDataFile(t, "data/f2.parquet", iceberg.EntryContentData, iceberg.ParquetFile)
existing := changelogTestDataFile(t, "deletes/d1.parquet", iceberg.EntryContentPosDeletes, iceberg.ParquetFile)

task, err := NewDeletedDataFileScanTask(data, []iceberg.DataFile{existing}, 1, 43)
require.NoError(t, err)

require.Equal(t, ChangelogOpDelete, task.Operation())
require.Equal(t, 1, task.ChangeOrdinal())
require.Equal(t, int64(43), task.CommitSnapshotID())
require.Equal(t, []iceberg.DataFile{existing}, task.ExistingDeletes())
require.Equal(t, []iceberg.DataFile{existing}, task.DeleteFiles)
}

func TestDeletedRowsScanTaskSeparatesAddedAndExistingDeletes(t *testing.T) {
data := changelogTestDataFile(t, "data/f2.parquet", iceberg.EntryContentData, iceberg.ParquetFile)
added := changelogTestDataFile(t, "deletes/d2.parquet", iceberg.EntryContentEqDeletes, iceberg.ParquetFile)
existing := changelogTestDataFile(t, "deletes/d1.parquet", iceberg.EntryContentPosDeletes, iceberg.ParquetFile)

task, err := NewDeletedRowsScanTask(data, []iceberg.DataFile{added}, []iceberg.DataFile{existing}, 2, 44)
require.NoError(t, err)

require.Equal(t, ChangelogOpDelete, task.Operation())
require.Equal(t, 2, task.ChangeOrdinal())
require.Equal(t, int64(44), task.CommitSnapshotID())
require.Equal(t, []iceberg.DataFile{added}, task.AddedDeletes())
require.Equal(t, []iceberg.DataFile{existing}, task.ExistingDeletes())
require.Equal(t, existing.FilePath(), task.DeleteFiles[0].FilePath())
require.Empty(t, task.EqualityDeleteFiles)
}

func TestChangelogScanTaskInterface(t *testing.T) {
data := changelogTestDataFile(t, "data/f1.parquet", iceberg.EntryContentData, iceberg.ParquetFile)

added, err := NewAddedRowsScanTask(data, nil, 0, 1)
require.NoError(t, err)
deletedFile, err := NewDeletedDataFileScanTask(data, nil, 1, 2)
require.NoError(t, err)
deletedRows, err := NewDeletedRowsScanTask(data, nil, nil, 2, 3)
require.NoError(t, err)

tasks := []ChangelogScanTask{added, deletedFile, deletedRows}
require.Equal(t, ChangelogOpInsert, tasks[0].Operation())
require.Equal(t, ChangelogOpDelete, tasks[1].Operation())
require.Equal(t, ChangelogOpDelete, tasks[2].Operation())
}

func TestClassifyDeleteFiles(t *testing.T) {
posDel := changelogTestDataFile(t, "deletes/d1.parquet", iceberg.EntryContentPosDeletes, iceberg.ParquetFile)
eqDel := changelogTestDataFile(t, "deletes/d2.parquet", iceberg.EntryContentEqDeletes, iceberg.ParquetFile)
dv := changelogTestDataFile(t, "deletes/d3.puffin", iceberg.EntryContentPosDeletes, iceberg.PuffinFile)
data := changelogTestDataFile(t, "data/f1.parquet", iceberg.EntryContentData, iceberg.ParquetFile)

got, err := classifyDeleteFiles([]iceberg.DataFile{eqDel, dv, posDel})
require.NoError(t, err)
require.Equal(t, []iceberg.DataFile{posDel}, got.pos)
require.Equal(t, []iceberg.DataFile{eqDel}, got.eq)
require.Equal(t, []iceberg.DataFile{dv}, got.dv)

_, err = classifyDeleteFiles([]iceberg.DataFile{data})
require.ErrorIs(t, err, ErrInvalidMetadata)
require.ErrorContains(t, err, "expected delete file")
}
29 changes: 29 additions & 0 deletions table/scanner.go
Original file line number Diff line number Diff line change
Expand Up @@ -376,6 +376,35 @@ func IsDeletionVector(df iceberg.DataFile) bool {
df.ContentType() == iceberg.EntryContentPosDeletes
}

type dataFileKind int

const (
dataFileKindData dataFileKind = iota
dataFileKindPosDeletes
dataFileKindEqDeletes
dataFileKindDeletionVector
)

// classifyDataFile buckets a file by content type. Deletion vectors are
// Puffin position-delete files and are split out from regular pos-deletes.
func classifyDataFile(f iceberg.DataFile) (dataFileKind, error) {
switch f.ContentType() {
case iceberg.EntryContentData:
return dataFileKindData, nil
case iceberg.EntryContentPosDeletes:
if IsDeletionVector(f) {
return dataFileKindDeletionVector, nil
}

return dataFileKindPosDeletes, nil
case iceberg.EntryContentEqDeletes:
return dataFileKindEqDeletes, nil
default:
return 0, fmt.Errorf("%w: unknown DataFileContent type (%s)",
ErrInvalidMetadata, f.ContentType())
}
}

// Scan represents a table scan. It implements [io.Closer]; callers should
// close it when they are done, including early exits after remote planning
// succeeds but before all records are consumed.
Expand Down
Loading