chore: use templatem member method for incremental plan files

Author: WZhuo

src/iceberg/table_scan.cc

@@ -644,23 +644,21 @@ Result<std::vector<std::shared_ptr<FileScanTask>>> DataTableScan::PlanFiles() co
 // to the two-arg virtual PlanFiles() override in the concrete subclass.
 // Defined as a friend to access the protected two-arg PlanFiles().
 template <typename ScanTaskType>
-Result<std::vector<std::shared_ptr<ScanTaskType>>> ResolvePlanFiles(
-    const IncrementalScan<ScanTaskType>& scan) {
-  if (IsScanCurrentLineage(scan.context())) {
-    if (scan.metadata()->current_snapshot_id == kInvalidSnapshotId) {
+Result<std::vector<std::shared_ptr<ScanTaskType>>>
+IncrementalScan<ScanTaskType>::PlanFiles() const {
+  if (IsScanCurrentLineage(context_)) {
+    if (metadata_->current_snapshot_id == kInvalidSnapshotId) {
       return std::vector<std::shared_ptr<ScanTaskType>>{};
     }
   }
 
-  ICEBERG_ASSIGN_OR_RAISE(
-      int64_t to_snapshot_id_inclusive,
-      internal::ToSnapshotIdInclusive(scan.context(), *scan.metadata()));
+  ICEBERG_ASSIGN_OR_RAISE(int64_t to_snapshot_id_inclusive,
+                          internal::ToSnapshotIdInclusive(context_, *metadata_));
   ICEBERG_ASSIGN_OR_RAISE(
       std::optional<int64_t> from_snapshot_id_exclusive,
-      internal::FromSnapshotIdExclusive(scan.context(), *scan.metadata(),
-                                        to_snapshot_id_inclusive));
+      internal::FromSnapshotIdExclusive(context_, *metadata_, to_snapshot_id_inclusive));
 
-  return scan.PlanFiles(from_snapshot_id_exclusive, to_snapshot_id_inclusive);
+  return PlanFiles(from_snapshot_id_exclusive, to_snapshot_id_inclusive);
 }
 
 // IncrementalAppendScan implementation
@@ -675,11 +673,6 @@ Result<std::unique_ptr<IncrementalAppendScan>> IncrementalAppendScan::Make(
       std::move(metadata), std::move(schema), std::move(io), std::move(context)));
 }
 
-Result<std::vector<std::shared_ptr<FileScanTask>>> IncrementalAppendScan::PlanFiles()
-    const {
-  return ResolvePlanFiles<FileScanTask>(*this);
-}
-
 Result<std::vector<std::shared_ptr<FileScanTask>>> IncrementalAppendScan::PlanFiles(
     std::optional<int64_t> from_snapshot_id_exclusive,
     int64_t to_snapshot_id_inclusive) const {
@@ -754,11 +747,6 @@ Result<std::unique_ptr<IncrementalChangelogScan>> IncrementalChangelogScan::Make
   return NotImplemented("IncrementalChangelogScan is not implemented");
 }
 
-Result<std::vector<std::shared_ptr<ChangelogScanTask>>>
-IncrementalChangelogScan::PlanFiles() const {
-  return ResolvePlanFiles<ChangelogScanTask>(*this);
-}
-
 Result<std::vector<std::shared_ptr<ChangelogScanTask>>>
 IncrementalChangelogScan::PlanFiles(std::optional<int64_t> from_snapshot_id_exclusive,
                                     int64_t to_snapshot_id_inclusive) const {

src/iceberg/table_scan.h

@@ -360,7 +360,7 @@ class IncrementalScan : public TableScan {
  public:
   ~IncrementalScan() override = default;
 
-  virtual Result<std::vector<std::shared_ptr<ScanTaskType>>> PlanFiles() const = 0;
+  Result<std::vector<std::shared_ptr<ScanTaskType>>> PlanFiles() const;
 
  protected:
   virtual Result<std::vector<std::shared_ptr<ScanTaskType>>> PlanFiles(
@@ -385,8 +385,6 @@ class ICEBERG_EXPORT IncrementalAppendScan : public IncrementalScan<FileScanTask
 
   ~IncrementalAppendScan() override = default;
 
-  Result<std::vector<std::shared_ptr<FileScanTask>>> PlanFiles() const override;
-
  protected:
   Result<std::vector<std::shared_ptr<FileScanTask>>> PlanFiles(
       std::optional<int64_t> from_snapshot_id_exclusive,
@@ -406,8 +404,6 @@ class ICEBERG_EXPORT IncrementalChangelogScan
 
   ~IncrementalChangelogScan() override = default;
 
-  Result<std::vector<std::shared_ptr<ChangelogScanTask>>> PlanFiles() const override;
-
  protected:
   Result<std::vector<std::shared_ptr<ChangelogScanTask>>> PlanFiles(
       std::optional<int64_t> from_snapshot_id_exclusive,