[GitHub] [iceberg] rdblue commented on pull request #2807: [2806]Dell EMC ECS catalog implementation

[GitBox <gi...@apache.org>]

rdblue commented on pull request #2807:
URL: https://github.com/apache/iceberg/pull/2807#issuecomment-878685881


   @wang-x-xia, thanks for working on this. I'm glad to see proposed support for EMC!
   
   I think the first thing to do is to get this into more manageable chunks to review and commit. Is it possible to divide this into a FileIO implementation PR and then a Catalog and TableOperations PR? It would also be really helpful to add a bit more about what you're proposing to the description. For example: How does the catalog work? What is the atomic operation you're using?


-- 
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