You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tajo.apache.org by "Jihoon Son (JIRA)" <ji...@apache.org> on 2015/09/04 04:18:45 UTC

[jira] [Updated] (TAJO-1330) Resolve findbug warnings

     [ https://issues.apache.org/jira/browse/TAJO-1330?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jihoon Son updated TAJO-1330:
-----------------------------
    Fix Version/s:     (was: 0.11.0)

> Resolve findbug warnings
> ------------------------
>
>                 Key: TAJO-1330
>                 URL: https://issues.apache.org/jira/browse/TAJO-1330
>             Project: Tajo
>          Issue Type: Bug
>            Reporter: Hyunsik Choi
>
> {noformat}
> -1 overall. Here are the results of testing the latest attachment 
> http://issues.apache.org/jira/secure/attachment/12696401/TAJO-1260_3.patch
> against master revision release-0.9.0-rc0-167-gae7862b.
> +1 @author. The patch does not contain any @author tags.
> +1 tests included. The patch appears to include 12 new or modified test files.
> +1 javac. The applied patch does not increase the total number of javac compiler warnings.
> +1 javadoc. The applied patch does not increase the total number of javadoc warnings.
> +1 checkstyle. The patch generated 0 code style errors.
> -1 findbugs. The patch appears to introduce 212 new Findbugs (version 2.0.3) warnings.
> -1 release audit. The applied patch generated 373 release audit warnings.
> +1 core tests. The patch passed unit tests in tajo-algebra tajo-core tajo-plan.
> Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/584//testReport/
> Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt
> Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-plan.html
> Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html
> Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/584//console
> This message is automatically generated.
> {noformat}
> So far, we haven't made much effort to fix findbug errors after we adopt findbugs. Now is the time to concern the quality of source code too.
> This work may take a long time because the number of find bug warnings is more than 200. We need to make sub tasks or new jira for them.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)