drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DRILL-3983) Small test improvements
Date Tue, 27 Oct 2015 17:46:27 GMT

    [ https://issues.apache.org/jira/browse/DRILL-3983?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14976834#comment-14976834
] 

ASF GitHub Bot commented on DRILL-3983:
---------------------------------------

Github user julienledem commented on the pull request:

    https://github.com/apache/drill/pull/221#issuecomment-151584120
  
    @adeneche I'm all for removing verbose logging. In this case, it sounds like we should
go after bad logging rather than removing information from errors. 
    Having "verbose errors" will actually enable less logging:
    Currently the server side (drillbit) stack trace is lost when the exception goes through
the RPC layer. Which means we rely on logging those errors on the server side to find the
original stack trace.
    the "verbose errors" setting will add the stacktrace in the exception message so that
we get both the client and server side stacktrace from the client side exception. which would
allow removing the server side logging in tests.
    
    I think we should follow this general principle: If an exception is dealt with it should
not be logged. Only exception that can not be dealt with should be logged as errors or warning
depending on the effect.
    A negative test would catch an exception and verify it. If logging happens at some layer
bellow it, this sounds like it should be refactored.


> Small test improvements
> -----------------------
>
>                 Key: DRILL-3983
>                 URL: https://issues.apache.org/jira/browse/DRILL-3983
>             Project: Apache Drill
>          Issue Type: Test
>            Reporter: Julien Le Dem
>            Assignee: Julien Le Dem
>




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

Mime
View raw message