Dong Li created KYLIN-1814:
------------------------------

             Summary: Avoid "catch-log-throw" pattern in source code
                 Key: KYLIN-1814
                 URL: https://issues.apache.org/jira/browse/KYLIN-1814
             Project: Kylin
          Issue Type: Improvement
    Affects Versions: v1.5.2
            Reporter: Dong Li
            Assignee: Dong Li
            Priority: Minor


"catch-log-throw" will make log messages mess, and we need to go through all 
code and fix them. For example, In CubeController class:

private JobInstance buildInternal(String cubeName, long startTime, long 
endTime, //
        long startOffset, long endOffset, String buildType, boolean force) {
    try {
        String submitter = 
SecurityContextHolder.getContext().getAuthentication().getName();
        CubeInstance cube = jobService.getCubeManager().getCube(cubeName);
        return jobService.submitJob(cube, startTime, endTime, startOffset, 
endOffset, //
                CubeBuildTypeEnum.valueOf(buildType), force, submitter);
    } catch (Exception e) {
        logger.error(e.getLocalizedMessage(), e);
        throw new InternalErrorException(e.getLocalizedMessage());
    }
}



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

Reply via email to