C-Loftus commented on issue #1096:
URL: https://github.com/apache/iceberg-go/issues/1096#issuecomment-4652955394
Now that #1158 is merged, @tanmayrauth would you be able to advise on your
preferred next steps?
I am inclined to
1. Create a new PR which condenses some of the methods in the
`HadoopCatalogFS` interface but keeps the same logic in the existing code. The
idea is to allow the interface to be easier to fulfill without changing any
behavior.
2. After the first PR is merged, create another new PR after that support
arbitrary schemes with `explicit "unsafe for concurrent writers" caveat
surfaced on CreateCatalog and in docs`
- This would extend #1158 to no longer have the `file://` prefix and
supporting an arbitrary scheme.
- Hold off on adding any `per-backend conditional-write story (S3
If-Match, GCS generation preconditions, etc.)` logic given the complexity.
- Create a new issue to track this, i.e. `feat(catalog: Concurrent
Write Support for Hadoop Cloud Storage Backend`
--
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]
