elek commented on pull request #989: URL: https://github.com/apache/hadoop-ozone/pull/989#issuecomment-636147143
Thanks to open this issue @Simon0806 I agree, that it's a problem that the defaults are not merged currently and we should fix it. 1. Small comments: As I see the original structure (storing the classes under `libs/`) is removed. It was requried for the isolated class loader. And it's not sure how can it work. 2. Currently the hadoop-27 acceptane tests are broken, I am not sure if legacy works well. (If I understood well your steps under the test section, this is just the first problem). 3. A bigger comment: we decided to remove this legacy magic all together. (There is a big limitation: Legacy with isloated class loader can't support secure Hadoop). The work to remove legacy and replace with something simpler started weeks ago (See related [issue](https://issues.apache.org/jira/browse/HDDS-3458) and mailing list thred), and fortunatelly it closes to the end. The final patch is uploaded under #992. If you are looking for a working, Hadoop 2.7 compatible client, give it a try. I would appreciate any help with the testing. This patch remove the Isolated/Filtered class loader from the picture and creates hadoop2.x compatible jar files. If you really need this patch, I am fine to merge it somehow, but it requires more testing as the acceptance test of hadoop27-ozone is turned off. For me, it seems to be less effort to test and use #992, but I am open to any suggestion. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
