vinishjail97 commented on PR #591: URL: https://github.com/apache/incubator-xtable/pull/591#issuecomment-2550476140
@ashvina I will create two separate PR's to avoid the confusion. 1. Interfaces for CatalogSyncClient and CatalogSync. 2. Integration of catalog syncs in XTable conversion controller using RunCatalogSync. We are not changing the way incremental sync works for table formats, the sync method in `ConversionController` still exists, the functionality of `syncTableAcrossCatalogs` is to synchronize the gap a table in source catalog to target catalog. If there's a need to tableFormat sync, we synchronize the table format first otherwise skip it. After that, the targetTable's metadata is synchronized to the target catalogs using the catalogTableIdentifier provided. I will add a small RFC document as well in the second PR for clarity. -- 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: commits-unsubscr...@xtable.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
