KiteSoar commented on issue #17549: URL: https://github.com/apache/hudi/issues/17549#issuecomment-3668529104
> > I plan to place the HoodieStorageFactory inside the hudi-io module (under org.apache.hudi.storage) , while strictly defaulting to HoodieHadoopStorage for backward compatibility, > > StorageFactory should not be put in `hudi-io` because you need `HOODIE_STORAGE_CLASS`(in `hudi-common`) to determine the storage type. `HOODIE_STORAGE_CLASS` is set to `HoodieHadoopStorage` by default, so no need to worry about the backward compatibility here Ah, good catch! I missed that the config is defined in hudi-common. I also checked `HoodieStorageConfig` and saw that `HOODIE_STORAGE_CLASS` already has `HoodieHadoopStorage` as the default value, so backward compatibility is indeed automatically handled.I'll place the factory in hudi-common then and start working on it. -- 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]
