prashantwason commented on issue #1450: [MINOR] Adding .codecov.yml to set exclusions for code coverage reports. URL: https://github.com/apache/incubator-hudi/pull/1450#issuecomment-606200998 Sure, I will file a JIRA. I think you misunderstood my comments so let me clarify. 1. I also prefer that everything is unit tested rather than excluded. Many projects enforce that checkins do not reduce code coverage numbers so maybe HUDI should do that too. 2. HoodieInputFormat.java is a migration class so I feel it is ok to exclude it. https://github.com/apache/incubator-hudi/blob/master/hudi-hadoop-mr/src/main/java/com/uber/hoodie/hadoop/HoodieInputFormat.java 3. Exclusions are never permanent. We can remove entries from it when we get code coverage. 4. I only excluded those utilities which were not being executed as part of unit testing.
---------------------------------------------------------------- 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] With regards, Apache Git Services
