You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2020/10/01 02:21:29 UTC
[GitHub] [iceberg] jackye1995 opened a new issue #1538: Dynamically load LocationProvider implementation
jackye1995 opened a new issue #1538:
URL: https://github.com/apache/iceberg/issues/1538
Currently the `LocationProviders` class only load either `DefaultLocationProvider` which follows the Hive standard or the `ObjectStoreLocationProvider` that is optimized for cloud storage use case. Use cases like #1531 will become increasingly common, and we should allow users to load their custom location providers without the need to change core package.
Therefore, I propose dynamically loading the `LocationProvider` implementation in the method `LocationProviders.locationsFor` and use new table property `write.location-provider.class` to determine the class. Table property `write.object-storage.enabled` will still be supported to keep backwards compatibility.
Plan to submit a PR in 1-2 days, please let me know if there is any concern, thanks.
----------------------------------------------------------------
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.
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] rdblue closed issue #1538: Dynamically load LocationProvider implementation
Posted by GitBox <gi...@apache.org>.
rdblue closed issue #1538:
URL: https://github.com/apache/iceberg/issues/1538
----------------------------------------------------------------
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.
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] mickjermsurawong-stripe commented on issue #1538: Dynamically load LocationProvider implementation
Posted by GitBox <gi...@apache.org>.
mickjermsurawong-stripe commented on issue #1538:
URL: https://github.com/apache/iceberg/issues/1538#issuecomment-701894239
This PR https://github.com/apache/iceberg/issues/1538 was re-purposed from solving specific problem of data localization to address this more general use case.
----------------------------------------------------------------
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.
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] mickjermsurawong-stripe edited a comment on issue #1538: Dynamically load LocationProvider implementation
Posted by GitBox <gi...@apache.org>.
mickjermsurawong-stripe edited a comment on issue #1538:
URL: https://github.com/apache/iceberg/issues/1538#issuecomment-701894239
This PR https://github.com/apache/iceberg/pull/1531 was re-purposed from solving specific problem of data localization to address this more general use case.
----------------------------------------------------------------
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.
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