----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54385/#review158041 -----------------------------------------------------------
Fix it, then Ship it! Fix, then ship! geode-docs/managing/autoreconnect/member-reconnect.html.md.erb (line 27) <https://reviews.apache.org/r/54385/#comment228712> This seems a little awkward. Perhaps: > "and, by default, automatically restarts into" geode-docs/managing/autoreconnect/member-reconnect.html.md.erb (line 46) <https://reviews.apache.org/r/54385/#comment228715> by setting the property to geode-docs/managing/autoreconnect/member-reconnect.html.md.erb (line 52) <https://reviews.apache.org/r/54385/#comment228716> setting the property to - Joey McAllister On Dec. 5, 2016, 6:57 p.m., Karen Miller wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54385/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2016, 6:57 p.m.) > > > Review request for geode, Bruce Schuchardt, Dave Barnes, Joey McAllister, and > Kirk Lund. > > > Repository: geode > > > Description > ------- > > Autoreconnect assumes valid XML configuration (from the > default use of the cluster config svc). That doesn't > work if the cache is configured using API calls. So, > document this, and suggest disabling either default use > of autoreconnect or default use of the cluster configuration > service. > > > Diffs > ----- > > geode-docs/managing/autoreconnect/member-reconnect.html.md.erb > 916d301a523f052efe83c33fa318898bc75273e5 > > Diff: https://reviews.apache.org/r/54385/diff/ > > > Testing > ------- > > gradle rat check passes > > Book builds with no broken links. > > > Thanks, > > Karen Miller > >
