[GitHub] [accumulo] ctubbsii commented on pull request #3265: Remove deprecated scan interpreter and formatter

["ctubbsii (via GitHub)" <gi...@apache.org>]

ctubbsii commented on PR #3265:
URL: https://github.com/apache/accumulo/pull/3265#issuecomment-1487669752

   The scan interpreter was deprecated in favor of urging users to use JShell for processing/parsing their scan input ranges and formatting the output, if they need anything more complicated than basic dumping of bytes. The interpreter was related to the formatter for scans, and both options were deprecated on the scan command (and its derivatives), but not all the formatter options were properly marked as deprecated. In order to fully remove them, they should be marked as deprecated in a 2.1 release first. The formatter stuff is a little bit entangled, though... so this PR mostly addresses the interpreter stuff only.
   
   I will need to see if all the ITs pass with this, though, because I suspect some ITs for the shell may still expect the interpreter/formatter stuff to be there. Even after code reviews, I won't merge this until all ITs pass with it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org