viirya commented on pull request #2297: URL: https://github.com/apache/hadoop/pull/2297#issuecomment-691712861
Checked the cc warnings and the related code. They are code long time ago, e.g., 2014, and not touched here. Many of the cc warnings are `warning: dynamic exception specifications are deprecated in C++11 [-Wdeprecated]`. I guess it is either due to that we didn't check such warnings when the code was committed, or compilation tools upgrade? I think it is not caused by this change. Because we removed some .c and .h files, so the CI build triggered related building. So I am not sure which one is good, fixing these compilation warnings, or ignoring them? ---------------------------------------------------------------- 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]
