There was a comment on the github pull request that should be addressed.

Changed the usage from iequals() to strncasecmp (with #ifdef for MSC_VER). This is much more lightweight and we're not dealing with other than ASCII here.

Build tested: 1.1.7-1

server commit:
b5a7a22
engine commit:
1c23dc9

Command is no longer case sensitive, as the ticket requested.

I noticed the following issue. I don't know if it

mcsadmin> redistributeData stop
redistributedata Wed Feb 6 22:14:17 2019
WriteEngineServer returned status 1: Redistribute is not running. Command is ignored.
redistributeData must have one of START, STOP or STATUS

mcsadmin> redistributeData STOP
redistributedata Wed Feb 6 22:15:12 2019
WriteEngineServer returned status 1: Redistribute is not running. Command is ignored.
redistributeData must have one of START, STOP or STATUS

redistributeData START
Source dbroots: 1 2
Destination dbroots: 1 2

WriteEngineServer returned status 1: Cleared.
WriteEngineServer returned status 2: Redistribute is started.
redistributeData must have one of START, STOP or STATUS

The commands are valid. We should not output the last line message

Build verified: 1.1.8-1 nightly
server commit:
01cc1ef
engine commit:
0af6994
Still waiting for 1.2.4-1

Build verified: Github source 1.2.4-1

git commits:

/root/columnstore/mariadb-columnstore-server
commit e3d99393916f0231db02564dd5e316e803bdbbe9
Author: Andrew Hutchings <andrew@linuxjedi.co.uk>
Date: Mon Jan 14 16:20:01 2019 +0000

Disable Travis triggering on pull requests

/root/columnstore/mariadb-columnstore-server/mariadb-columnstore-engine
commit 92d662491ac08095e5998b69a9c730e9d208e8f4
Merge: 0dd33c6 45df72a
Author: Roman Nozdrin <drrtuy@gmail.com>
Date: Thu May 16 21:50:05 2019 +0300

Merge pull request #761 from mariadb-corporation/develop-1.2-merge-up-20190514

Merge develop-1.1 into develop-1.2