vinothchandar commented on a change in pull request #2342:
URL: https://github.com/apache/hudi/pull/2342#discussion_r547152380



##########
File path: 
hudi-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadata.java
##########
@@ -66,24 +58,13 @@
  * If the metadata table does not exist, RPC calls are used to retrieve file 
listings from the file system.
  * No updates are applied to the table and it is not synced.
  */
-public class HoodieBackedTableMetadata implements HoodieTableMetadata {
+public class HoodieBackedTableMetadata extends AbstractHoodieTableMetadata {

Review comment:
       okay that also makes sense. Personally, I would not have done the split, 
until a second kind of metadata impl shows up. but udit added the 
FileSytemBackedMetadata, it does not need the merging per se.. Still we can 
keep it like this for now. 




----------------------------------------------------------------
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:
[email protected]


Reply via email to