Follow up HUDI-27 : Call super.close() in HoodieWraperFileSystem::close()

[vinothchandar]

• FileSystem.close() does clean up of cache etc.

[n3nash]

Can you please share you understanding of super.close() in this context ? So, super is basically the current instance of FileSystem being extended by the current instance of HoodieWrapperFileSystem. The HoodieWrapperFileSystem also holds an instance of the underlying file system as a member variable. What wil super.close() behavior be here once it is invoked ?

[vinothchandar]

Well super.close() is pretty obvious. Its the FIleSystem class's close(), which does some clean up of the cache etc. @bvaradar pointed out that it might be good to clean it up for the HoodieWrapperFileSystem instance..

I still think closing the wrapped file system, is not a good idea, since its cached globally across jvm.

[n3nash]

Okay, fair enough.