----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55442/#review161458 -----------------------------------------------------------
geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/cli/LuceneIndexDetails.java (line 48) <https://reviews.apache.org/r/55442/#comment232726> How you know which server is for which index's detail? For example, you will show 2 lines for the same index, but which line is for which server? - xiaojian zhou On Jan. 12, 2017, 2:49 a.m., nabarun nag wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55442/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2017, 2:49 a.m.) > > > Review request for geode, Barry Oglesby, Jason Huynh, Dan Smith, and xiaojian > zhou. > > > Repository: geode > > > Description > ------- > > Issue: > > Initially: > We could not link the index information to the server storing the index. > > ``` > Index Name | Region Path | Indexed Fields | Field Analyzer | Status > | Query Executions | Updates | Commits | Documents > ---------- | ----------- | ---------------------- | -------------- | > ----------- | ---------------- | ------- | ------- | --------- > testIndex | /testRegion | [__REGION_VALUE_FIELD] | {} | > Initialized | 0 | 3 | 3 | 2 > testIndex | /testRegion | [__REGION_VALUE_FIELD] | {} | > Initialized | 0 | 0 | 0 | 1 > ``` > > > Solution: Added a server column. > ``` > Index Name | Region Path | Server Name | > Indexed Fields | Field Analyzer | Status | Query Executions | > Updates | Commits | Documents > ---------- | ----------- | ------------------------------------------ | > ---------------------- | -------------- | ----------- | ---------------- | > ------- | ------- | --------- > testIndex | /testRegion | server2 -- 10.0.0.8(server2:2814)<v2>:1026 | > [__REGION_VALUE_FIELD] | {} | Initialized | 0 | 0 > | 0 | 2 > testIndex | /testRegion | server1 -- 10.0.0.8(server1:3037)<v4>:1025 | > [__REGION_VALUE_FIELD] | {} | Initialized | 0 | 0 > | 0 | 1 > ``` > > > Diffs > ----- > > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/cli/LuceneIndexCommands.java > 6a5a1e0 > > geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/cli/LuceneIndexDetails.java > cc96df2 > > Diff: https://reviews.apache.org/r/55442/diff/ > > > Testing > ------- > > 1. Precheck > 2. Running it on gfsh terminal > > > Thanks, > > nabarun nag > >
