You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "jackye1995 (via GitHub)" <gi...@apache.org> on 2023/02/11 20:52:43 UTC
[GitHub] [iceberg] jackye1995 opened a new issue, #6809: Consolidate file path creation to LocationProvider
jackye1995 opened a new issue, #6809:
URL: https://github.com/apache/iceberg/issues/6809
### Feature Request / Improvement
from https://github.com/apache/iceberg/pull/6772#issuecomment-1426873914, also copied here:
We have different places for creating file path names,
- `LocationProvider` for data files
- `TableOperations.metadataFileLocation` for manifests and manifest lists
- catalog specific impl for metadata file location (e.g. `BaseMetastoreTableOperations.newTableMetadataFilePath`)
We should consolidate all the file path creations to `LocationProvider` if possible. Any thoughts? @rdblue @nastra @RussellSpitzer
This would also help us enforce things like object storage mode to not just data files but also metadata files.
### 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] Consolidate file path creation to LocationProvider [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed issue #6809: Consolidate file path creation to LocationProvider
URL: https://github.com/apache/iceberg/issues/6809
--
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] github-actions[bot] commented on issue #6809: Consolidate file path creation to LocationProvider
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #6809:
URL: https://github.com/apache/iceberg/issues/6809#issuecomment-1676504877
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
Re: [I] Consolidate file path creation to LocationProvider [iceberg]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on issue #6809:
URL: https://github.com/apache/iceberg/issues/6809#issuecomment-1882020379
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
[GitHub] [iceberg] youngxinler commented on issue #6809: Consolidate file path creation to LocationProvider
Posted by "youngxinler (via GitHub)" <gi...@apache.org>.
youngxinler commented on issue #6809:
URL: https://github.com/apache/iceberg/issues/6809#issuecomment-1429060343
If this PR was added, I think it would be better to declare it to tell users that all added paths should follow POSIX compliant paths? Even though we have normalized it internally.
--
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