JiaKe
JiaKe
> It seems Arrow's HDFS filesystem depends on some hdfs dynamic library to be present in the system and dynamically links against it. I see this PR does something similar...
> > Well it is unmaintained so it should'nt be the default and we should probably mark it deprecated and remove it in the future. > > Switching the default...
@assignUser Agree to remove hawa implementation entirely. @majetideepak What do you think?
@assignUser @majetideepak Thank you for your response. I have initiated a poll in the community at https://github.com/facebookincubator/velox/discussions/9969. Your participation and vote would be greatly appreciated.
> @JkSelf Thank you for your work. Is there any progress? It would be a really nice feature for me to replace with libhdfs so that we can support more...
@majetideepak @assignUser @xiaoxmeng Can you help to review this PR? Thanks.
@assignUser The InsertIntoHdfsTest is failing with a core dump in libhdfs.so when executed alongside the velox_hdfs_file test suite. ``` [ RUN ] InsertIntoHdfsTest.insertIntoHdfsTest I20240911 07:21:56.337330 59114 PeriodicStatsReporter.cpp:69] Starting PeriodicStatsReporter with...
@assignUser @majetideepak @kgpai Can you help to review this PR again? Thanks.
@majetideepak Can you help to review this PR? Thanks.
@majetideepak Can you help to review again? Thanks.