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

Posted to notifications@geode.apache.org by "DonalEvans (GitHub)" <gi...@apache.org> on 2019/11/14 17:08:31 UTC

[GitHub] [geode] DonalEvans commented on pull request #4324: GEODE-6991: Create alter query-service gfsh command

It's necessary to provide an option that is parsed as a single String (as opposed to a String array) because if a user specifies `AUTHORIZER_PARAMETERS` with an argument that includes a comma as part of one of the parameters (for example, in a regex using the X{n,m} quantifier) then gfsh will split that argument on the comma as part of its parsing. That being said, if there's a way to ensure that user input will only be split on the "correct" commas then having only one option would definitely be preferable.

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