----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53661/#review155668 -----------------------------------------------------------
I noticed one variation to the pattern: In geode-docs/rest_apps/setup_config.html.md.erb at line 100, did you miss one conversion of start-dev-rest-api to start-rest-api, or was this intentional? - Dave Barnes On Nov. 10, 2016, 10:44 p.m., Karen Miller wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53661/ > ----------------------------------------------------------- > > (Updated Nov. 10, 2016, 10:44 p.m.) > > > Review request for geode, Dave Barnes, Jinmei Liao, Joey McAllister, and > Kevin Duling. > > > Repository: geode > > > Description > ------- > > - Add 3 missing gfsh start server options: > --http-service-port > --http-service-bind-address > --start-rest-api > > - Update examples to use these options, instead of using > the older --J=-gemfire. specification. > > > Diffs > ----- > > geode-docs/configuring/cluster_config/gfsh_remote.html.md.erb > c8ea24096c76a5c03bd7a902d225ee4566932b29 > geode-docs/rest_apps/setup_config.html.md.erb > 604dd059290f8da41e2e05e0946e6a7eaa01e6ca > geode-docs/tools_modules/gfsh/command-pages/start.html.md.erb > ff703cb4a7fc49a1bffa92cf9cc2207a6cce2d32 > > Diff: https://reviews.apache.org/r/53661/diff/ > > > Testing > ------- > > gradle rat check passes > > > Thanks, > > Karen Miller > >