You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "onichols-pivotal (GitHub)" <gi...@apache.org> on 2020/02/06 19:17:40 UTC

[GitHub] [geode] onichols-pivotal commented on pull request #4429: GEODE-7550: Param to only list gw receivers or senders

My 2 cents:
the user will probably ever only care about:
`list gateways`
`list gateways --receivers-only`
`list gateways --senders-only`

Anything else (explicitly appending `=false`, specifying both, etc) falls under "don't care" in my opinion.  You could throw an error in every permutation other than the 3 above, or do anything that feels consistent with principal-of-least-surprise.

@davebarnes97 has a great suggestion to think about maybe constraining the input space another way to avoid the need for so much validation, but it does lead to a slightly "less natural" command syntax imho.

[ Full content available at: https://github.com/apache/geode/pull/4429 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org