...(continued) And perhaps the real point of the ticket was missed: there's
no such option as "--shutdown-locators", so the message definitely needs a
rewrite.


On Mon, Oct 23, 2017 at 8:51 AM, Dave Barnes <dbar...@pivotal.io> wrote:

> Swapnil,
> Are you suggesting that the gfsh shutdown command should shut down
> locators by default, not just data members? Sounds to me like a breaking
> change.
> If you area suggesting that the --include=locators option, when specified,
> should be accepted without the "=true" bit, which is now mandatory, I agree.
>
> On Mon, Oct 23, 2017 at 3:04 AM, Swapnil Bawaskar (JIRA) <j...@apache.org>
> wrote:
>
>>
>>     [ https://issues.apache.org/jira/browse/GEODE-3843?page=com.
>> atlassian.jira.plugin.system.issuetabpanels:comment-tabpane
>> l&focusedCommentId=16214921#comment-16214921 ]
>>
>> Swapnil Bawaskar commented on GEODE-3843:
>> -----------------------------------------
>>
>> All gfsh command options have a default value. In this case the default
>> value should be made true (if it is not already) rather than trying to fix
>> the message.
>>
>> > gfsh shutdown error message suggests non-existent option
>> > --------------------------------------------------------
>> >
>> >                 Key: GEODE-3843
>> >                 URL: https://issues.apache.org/jira/browse/GEODE-3843
>> >             Project: Geode
>> >          Issue Type: Bug
>> >          Components: gfsh
>> >            Reporter: Dave Barnes
>> >            Assignee: Srikanth Manvi
>> >            Priority: Minor
>> >
>> > 1. Start gfsh.
>> > 2. Start a locator.
>> > 3. Issue the 'shutdown' command.
>> > gfsh prompts for 'do you really want to...', then issues an error
>> message alerting you to the lack of data nodes. OK so far.
>> > This is followed by a line recommending that you use
>> '--shutdown-locators' to stop your locator.
>> > Should be '--include-locators=true'.
>>
>>
>>
>> --
>> This message was sent by Atlassian JIRA
>> (v6.4.14#64029)
>>
>
>

Reply via email to