harjitdotsingh commented on pull request #2255: URL: https://github.com/apache/accumulo/pull/2255#issuecomment-916442011
We will keep it backwards compatible On Thu, Sep 9, 2021 at 5:11 PM EdColeman ***@***.***> wrote: > ***@***.**** commented on this pull request. > ------------------------------ > > In shell/src/main/java/org/apache/accumulo/shell/commands/ScanCommand.java > <https://github.com/apache/accumulo/pull/2255#discussion_r705701541>: > > > @@ -285,6 +288,19 @@ protected void fetchColumns(final CommandLine cl, final ScannerBase scanner, > } > } > > + protected String[] extractColumnFamily(final String columnString) { > > It is likely that people have scripts that use -c. Removing that as an > option will break those scripts. People using -c currently are limited to > not having : in the colFam, but they are coping. > > Adding -cq and -cf provide additional capability when (the current -c) is > insufficient for their needs. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/apache/accumulo/pull/2255#discussion_r705701541>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AAASJ7QKPUIIRBWRO5MUT73UBEPILANCNFSM5DGYZX4A> > . > Triage notifications on the go with GitHub Mobile for iOS > <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> > or Android > <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. > > -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
