You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/06/28 15:04:47 UTC

[GitHub] [accumulo] milleruntime opened a new issue, #2787: Deprecate ScanInterpreter

milleruntime opened a new issue, #2787:
URL: https://github.com/apache/accumulo/issues/2787

   Since we now have use of JShell, I think it would be reasonable to just deprecate and eventually drop the scan interpreter stuff (especially since it's not even spelled correctly). It's a bit weird to have shell-specific config stored in the per-table config on the server-side anyway. JShell is a much more powerful and flexible tool, and a lot less work to support, than these quirky shell behaviors.
   
   Closes https://github.com/apache/accumulo/issues/2210
   


-- 
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.apache.org

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


[GitHub] [accumulo] cshannon commented on issue #2787: Deprecate ScanInterpreter

Posted by GitBox <gi...@apache.org>.
cshannon commented on issue #2787:
URL: https://github.com/apache/accumulo/issues/2787#issuecomment-1183328258

   I submitted a PR for this: #2806


-- 
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


[GitHub] [accumulo] milleruntime commented on issue #2787: Deprecate ScanInterpreter

Posted by GitBox <gi...@apache.org>.
milleruntime commented on issue #2787:
URL: https://github.com/apache/accumulo/issues/2787#issuecomment-1189002860

   Fixed in #2806 


-- 
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


[GitHub] [accumulo] cshannon commented on issue #2787: Deprecate ScanInterpreter

Posted by GitBox <gi...@apache.org>.
cshannon commented on issue #2787:
URL: https://github.com/apache/accumulo/issues/2787#issuecomment-1175454788

   @dlmarion mentioned to me that this should be just deprecating the public API for now and documenting (from here https://accumulo.apache.org/api/) so I can take a look at updating the docs for that.


-- 
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


[GitHub] [accumulo] milleruntime closed issue #2787: Deprecate ScanInterpreter

Posted by GitBox <gi...@apache.org>.
milleruntime closed issue #2787: Deprecate ScanInterpreter
URL: https://github.com/apache/accumulo/issues/2787


-- 
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


[GitHub] [accumulo] cshannon commented on issue #2787: Deprecate ScanInterpreter

Posted by GitBox <gi...@apache.org>.
cshannon commented on issue #2787:
URL: https://github.com/apache/accumulo/issues/2787#issuecomment-1172897241

   It looks like the scan interpreter stuff was already deprecated in #1140, should it just be removed now?


-- 
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