accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-1204) JSONServlet shows incorrect value for scans
Date Tue, 26 Mar 2013 14:05:16 GMT

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

Hudson commented on ACCUMULO-1204:
----------------------------------

Integrated in Accumulo-1.5 #54 (See [https://builds.apache.org/job/Accumulo-1.5/54/])
    ACCUMULO-1204 fix "scansessions" entry in json (Revision 1461105)

     Result = UNSTABLE
ecn : 
Files : 
* /accumulo/branches/1.5/server/src/main/java/org/apache/accumulo/server/monitor/servlets/JSONServlet.java

                
> JSONServlet shows incorrect value for scans
> -------------------------------------------
>
>                 Key: ACCUMULO-1204
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1204
>             Project: Accumulo
>          Issue Type: Bug
>          Components: monitor
>            Reporter: Christopher Tubbs
>            Assignee: Eric Newton
>             Fix For: 1.5.0
>
>   Original Estimate: 0.25h
>  Remaining Estimate: 0.25h
>
> In JSONServlet, I see
> {code:java}
> map.put("scans", scans);
> map.put("scans", scansessions);
> {code}
> This can't possibly be correct. It looks like a copy/paste error. A trivial fix is to
rename the map key to "scansessions" to be consistent with the rest of the code around it.
However, there's very little documentation on this API, so it's not quite clear what the value
for the "scansessions" variable even represents. Is there a better key name that is more descriptive
of what the value of that variable represents? If not, the trivial change should be implemented.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message