[GitHub] [accumulo] ddanielr commented on pull request #3684: consolidate y/n prompt in shell

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

ddanielr commented on PR #3684:
URL: https://github.com/apache/accumulo/pull/3684#issuecomment-1676213191

   > Set this a draft - there may be an issue in ShellIT - this should not be merged until that is resolved
   
   Found the issue in ShellIT was due to the forceOpt handling in the `ConfigCommand`
   
   Issue could be replicated by checking out [5ac8ac](https://github.com/apache/accumulo/pull/3684/commits/5ac8ac443afadc2466406012ad998db0bfcb091b) and running the following test
   ```
   mvn clean verify -Dit.test=org.apache.accumulo.test.shell.ShellIT#configTest -Dtest=foo -Dspotbugs.skip
   ```
   
   This issue has been fixed in [ea20d0](https://github.com/apache/accumulo/pull/3684/commits/ea20d048d5c138dec77092a017e03bfd983aa5fd)


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