You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "JonasJ-ap (via GitHub)" <gi...@apache.org> on 2023/04/27 23:53:41 UTC

[GitHub] [iceberg] JonasJ-ap opened a new pull request, #7454: Delta Migration: Correct `snapshotDataFilesCount` in SnapshotDeltaLakeTable.Result and use Immutable to implement it

JonasJ-ap opened a new pull request, #7454:
URL: https://github.com/apache/iceberg/pull/7454

   Currently, `snapshotDeltaLakeTable.execute()`'s result, `snapshotDataFilesCount` only counts data files in the most recent snapshot of the table. The intended behavior is to report the total number of data files migrated, which include those in all previous snapshots.
   
   This PR fixes this issue and use immutables to implement `SnapshotDeltaLakeTable.Result` following up the https://github.com/apache/iceberg/pull/7089


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] JonasJ-ap commented on pull request #7454: Delta Migration: Correct `snapshotDataFilesCount` in SnapshotDeltaLakeTable.Result and use Immutable to implement it

Posted by "JonasJ-ap (via GitHub)" <gi...@apache.org>.
JonasJ-ap commented on PR #7454:
URL: https://github.com/apache/iceberg/pull/7454#issuecomment-1526788169

   @jackye1995 May I ask if you have time to take a look at this too?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] jackye1995 merged pull request #7454: Delta Migration: Correct `snapshotDataFilesCount` in SnapshotDeltaLakeTable.Result and use Immutable to implement it

Posted by "jackye1995 (via GitHub)" <gi...@apache.org>.
jackye1995 merged PR #7454:
URL: https://github.com/apache/iceberg/pull/7454


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org