yangshangqing95 commented on code in PR #17134:
URL: https://github.com/apache/iceberg/pull/17134#discussion_r3554020622
##########
core/src/main/java/org/apache/iceberg/AllDataFilesTable.java:
##########
@@ -30,7 +30,7 @@
public class AllDataFilesTable extends BaseFilesTable {
AllDataFilesTable(Table table) {
- this(table, table.name() + ".all_data_files");
+ this(table, table.name() + "." +
MetadataTableType.ALL_DATA_FILES.tableName());
Review Comment:
Yes, sure. These refactors are related to and required by the changes in
this PR, though. Another option is that I can wait until this PR is merged,
then submit the refactors separately in a follow-up PR. Would that work for you?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
