[core] Remove Builder of SplitReadProvider.Context

Author: JingsongLi

paimon-core/src/main/java/org/apache/paimon/table/source/AppendTableRead.java

@@ -127,9 +127,7 @@ public InnerTableRead withLimit(int limit) {
     public RecordReader<InternalRow> reader(Split split) throws IOException {
         DataSplit dataSplit = (DataSplit) split;
         for (SplitReadProvider readProvider : readProviders) {
-            if (readProvider.match(
-                    dataSplit,
-                    SplitReadProvider.Context.builder().withForceKeepDelete(false).build())) {
+            if (readProvider.match(dataSplit, new SplitReadProvider.Context(false))) {
                 return readProvider.get().get().createReader(dataSplit);
             }
         }

paimon-core/src/main/java/org/apache/paimon/table/source/KeyValueTableRead.java

@@ -160,11 +160,7 @@ public TableRead withIOManager(IOManager ioManager) {
     public RecordReader<InternalRow> reader(Split split) throws IOException {
         DataSplit dataSplit = (DataSplit) split;
         for (SplitReadProvider readProvider : readProviders) {
-            if (readProvider.match(
-                    dataSplit,
-                    SplitReadProvider.Context.builder()
-                            .withForceKeepDelete(forceKeepDelete)
-                            .build())) {
+            if (readProvider.match(dataSplit, new SplitReadProvider.Context(forceKeepDelete))) {
                 return readProvider.get().get().createReader(dataSplit);
             }
         }

paimon-core/src/main/java/org/apache/paimon/table/source/splitread/SplitReadProvider.java

@@ -33,32 +33,12 @@ public interface SplitReadProvider {
     /** Split reader provider context. */
     class Context {
 
-        public static final Context DEFAULT = Context.builder().build();
-
         private final boolean forceKeepDelete;
 
-        private Context(boolean forceKeepDelete) {
+        public Context(boolean forceKeepDelete) {
             this.forceKeepDelete = forceKeepDelete;
         }
 
-        public static Context.Builder builder() {
-            return new Builder();
-        }
-
-        public static class Builder {
-
-            private boolean forceKeepDelete;
-
-            public Context build() {
-                return new Context(forceKeepDelete);
-            }
-
-            public Builder withForceKeepDelete(boolean forceKeepDelete) {
-                this.forceKeepDelete = forceKeepDelete;
-                return this;
-            }
-        }
-
         public boolean forceKeepDelete() {
             return forceKeepDelete;
         }

paimon-core/src/test/java/org/apache/paimon/table/source/splitread/DataEvolutionSplitReadProviderTest.java

@@ -40,6 +40,9 @@
 /** Tests for {@link DataEvolutionSplitReadProvider}. */
 public class DataEvolutionSplitReadProviderTest {
 
+    public static final SplitReadProvider.Context DEFAULT_CONTEXT =
+            new SplitReadProvider.Context(false);
+
     private Supplier<DataEvolutionSplitRead> mockSupplier;
     private SplitReadConfig mockSplitReadConfig;
     private DataEvolutionSplitRead mockSplitRead;
@@ -80,15 +83,15 @@ public void testGetAndInitialization() {
     public void testMatchWithNoFiles() {
         DataSplit split = mock(DataSplit.class);
         when(split.dataFiles()).thenReturn(Collections.emptyList());
-        assertThat(provider.match(split, SplitReadProvider.Context.DEFAULT)).isFalse();
+        assertThat(provider.match(split, DEFAULT_CONTEXT)).isFalse();
     }
 
     @Test
     public void testMatchWithOneFile() {
         DataSplit split = mock(DataSplit.class);
         DataFileMeta file1 = mock(DataFileMeta.class);
         when(split.dataFiles()).thenReturn(Collections.singletonList(file1));
-        assertThat(provider.match(split, SplitReadProvider.Context.DEFAULT)).isFalse();
+        assertThat(provider.match(split, DEFAULT_CONTEXT)).isFalse();
     }
 
     @Test
@@ -103,7 +106,7 @@ public void testMatchWithNullFirstRowId() {
         when(file1.fileName()).thenReturn("test1.parquet");
         when(file2.fileName()).thenReturn("test2.parquet");
 
-        assertThat(provider.match(split, SplitReadProvider.Context.DEFAULT)).isFalse();
+        assertThat(provider.match(split, DEFAULT_CONTEXT)).isFalse();
     }
 
     @Test
@@ -118,7 +121,7 @@ public void testMatchWithDifferentFirstRowIds() {
         when(file1.fileName()).thenReturn("test1.parquet");
         when(file2.fileName()).thenReturn("test2.parquet");
 
-        assertThat(provider.match(split, SplitReadProvider.Context.DEFAULT)).isFalse();
+        assertThat(provider.match(split, DEFAULT_CONTEXT)).isFalse();
     }
 
     @Test
@@ -136,13 +139,7 @@ public void testMatchSuccess() {
         when(file2.fileName()).thenReturn("test2.parquet");
 
         // The forceKeepDelete parameter is not used in match, so test both values
-        assertThat(
-                        provider.match(
-                                split,
-                                SplitReadProvider.Context.builder()
-                                        .withForceKeepDelete(true)
-                                        .build()))
-                .isTrue();
-        assertThat(provider.match(split, SplitReadProvider.Context.DEFAULT)).isTrue();
+        assertThat(provider.match(split, new SplitReadProvider.Context(true))).isTrue();
+        assertThat(provider.match(split, DEFAULT_CONTEXT)).isTrue();
     }
 }