gaborkaszab commented on PR #13987:
URL: https://github.com/apache/iceberg/pull/13987#issuecomment-3254129652
I'm not sure I get the original use-case here. When you do a commit() you
don't have to run a refresh() to get the new snapshot ID.
I created this one in TestRESTCatalog to verify:
`@Test
public void testSomething() {
catalog().createNamespace(TABLE.namespace());
Table tbl = catalog().createTable(TABLE, SCHEMA);
tbl.newAppend().appendFile(FILE_A).commit();
long s1 = tbl.currentSnapshot().snapshotId();
tbl.newAppend().appendFile(FILE_B).commit();
long s2 = tbl.currentSnapshot().snapshotId();
assertThat(s1).isNotEqualTo(s2);
}`
Anyway, this change is very invasive on the API from, and as the build
shows, breaks compatibility.
--
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-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
