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

Josh Elser commented on ACCUMULO-4128:
--------------------------------------

bq. The shell command `setscaniter` appears to be non-existent in the master 
branch. The test was not checking the output of the shell to confirm that the 
iterator was applied.

Wow, good find, Dave! I totally missed that scrolling through the output.

bq.  More concerning is that SetScanIterCommand is not marked deprecated in the 
1.8 branch. The new command looks to be `setiter -scan`.

By the letter of the law, shell commands are presently covered by semver so 
they can technically be deleted immediately (playing devil's advocate -- I 
think this is terrible and shell commands should also be covered in our public 
api). Did you happen to do any digging to figure out what commit/issue changed 
this in master?

> ShellServerIT#scansWithClassLoaderContext fails consistently
> ------------------------------------------------------------
>
>                 Key: ACCUMULO-4128
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4128
>             Project: Accumulo
>          Issue Type: Bug
>          Components: test
>            Reporter: Eric Newton
>            Assignee: Dave Marion
>             Fix For: 2.0.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>




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

Reply via email to