srowen commented on issue #23567: [MINOR][BUILD] ensure call to translate_component has correct number of arguments URL: https://github.com/apache/spark/pull/23567#issuecomment-455144196 Thanks @ipwright , I opened https://github.com/apache/spark/pull/23571 to fix many of the rest of the small issues, and https://github.com/apache/spark/pull/23570 to fix a bug. I'm a fan of static analysis just because it occasionally catches real bugs. The only minor issue I find is that the 'unused import' detection doesn't account for doctest usages. The Java code was already pretty OK as IntelliJ inspection had turned up most of the problems, but I don't think we'd made a comprehensive effort on Python. Oege mentioned that Scala analysis might come eventually, which would obviously be really good too, but is a lot harder.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
