virajjasani commented on pull request #2555: URL: https://github.com/apache/hbase/pull/2555#issuecomment-712065910
I was taking look at #2556 , @sandeepvinayak it looks like you mentioned `rowKey.startsWith(rowPrefix)` taking care of both cases. I think it looks reasonable from simplifying the available option but if we think user can pay more attention to `row` and `rowPrefix` both as separate options, maybe we can include 2 options as well. If we want to go with this current approach (I think it might be enough to have single option), we can document option description to ensure user understands that `-w` is used for both: exact and prefix of row (although it's intuitive, but while reading we might pay more attention to what description says clearly than applying our own logic :) ). WDYT @wchevreuil ? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
