You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "pudidic (via GitHub)" <gi...@apache.org> on 2023/05/03 07:27:05 UTC
[GitHub] [iceberg] pudidic opened a new issue, #7509: Provide a protected TableMetadata() constructor to allow inheritance
pudidic opened a new issue, #7509:
URL: https://github.com/apache/iceberg/issues/7509
### Feature Request / Improvement
I'm designing a service that transforms absolute paths in metadata layers. It needs to replace snapshot objects with new absolute paths. I tried to expand TableMetadata. However, TableMetadata has only package-private constructor. A protected TableMetadata() constructor is required for the inheritance.
Current:
```
public class TableMetadata {
...
@SuppressWarnings("checkstyle:CyclomaticComplexity")
TableMetadata(
String metadataFileLocation,
int formatVersion,
String uuid,
...
```
Suggested:
```
public class TableMetadata {
...
/** Empty constructor for inheritance */
protected TableMetadata() {
this.metadataFileLocation = null;
this.formatVersion = -1;
this.uuid = null;
this.location = null;
this.lastSequenceNumber = -1;
this.lastUpdatedMillis = -1;
this.lastColumnId = -1;
this.currentSchemaId = -1;
this.schemas = null;
this.defaultSpecId = -1;
this.specs = null;
this.lastAssignedPartitionId = -1;
this.defaultSortOrderId = -1;
this.sortOrders = null;
this.properties = null;
this.currentSnapshotId = -1;
this.schemasById = null;
this.specsById = null;
this.sortOrdersById = null;
this.snapshotLog = null;
this.previousFiles = null;
this.statisticsFiles = null;
this.changes = null;
}
@SuppressWarnings("checkstyle:CyclomaticComplexity")
TableMetadata(
String metadataFileLocation,
int formatVersion,
String uuid,
...
```
### Query engine
None
--
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.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
Re: [I] Provide a protected TableMetadata() constructor to allow inheritance [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #7509:
URL: https://github.com/apache/iceberg/issues/7509#issuecomment-1811593099
This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale'
--
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
Re: [I] Provide a protected TableMetadata() constructor to allow inheritance [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed issue #7509: Provide a protected TableMetadata() constructor to allow inheritance
URL: https://github.com/apache/iceberg/issues/7509
--
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
Re: [I] Provide a protected TableMetadata() constructor to allow inheritance [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #7509:
URL: https://github.com/apache/iceberg/issues/7509#issuecomment-1786237312
This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occurs. To permanently prevent this issue from being considered stale, add the label 'not-stale', but commenting on the issue is preferred when possible.
--
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