-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52052/
-----------------------------------------------------------

Review request for geode, anilkumar gingade, Darrel Schneider, Eric Shu, Scott 
Jewell, Swapnil Bawaskar, and Dan Smith.


Repository: geode


Description
-------

Refactored “show missing-disk-stores command” and added a section to the
output to show missing colocated regions. Corrected messages in
LocalizedStrings, change “gemfire list-missing-disk-stores” to
“gfsh show missing-disk-stores”. Added new command function class to gather
both missing disk-store and missing colocated region information on a
distributed system member.

Parameterized show missing-disk-store error message.

Added new DataSerializable class for sending missing colocated region
results to gfsh command.

Refactored ColocationLogger. Class originally deferred updating the list
of missing colocated children (i.e. removing regions from the missing
list that are now present) for a region until the next logging
interval. This has been changed to update the list on-demand, whenever
getMissingChildRegions() is called.

Added DUnit test for new functionality in show missing-disk-store
command.

Added JUnit tests for new classes

New command oputput formats:

When there are no problems to report --

No missing disk store found
No missing colocated region found


Missing disk-stores (no missing colocated regions)

Missing Disk Stores
           Disk Store ID             |      Host      | Directory
------------------------------------ | -------------- | 
---------------------------------
8c204b3a-47ca-4bf4-ab3d-e6f704ea6ac6 | /10.118.33.251 | 
/vm1/DiskStoreCommandsDUnitTest1

No missing colocated region found


Missing colocated regions (no missing disk-stores) --

No missing disk store found

Missing Colocated Regions
    Host      | Distributed Member |               Parent Region               
| Missing Colocated Region
------------- | ------------------ | ----------------------------------------- 
| ------------------------
10.118.33.251 | Manager            | /testShowPersistentRecoveryFailuresRegion 
| /childRegion
10.118.33.251 | VM1                | /testShowPersistentRecoveryFailuresRegion 
| /childRegion


Missing disk-stores and missing colocated regions -

Missing Disk Stores
           Disk Store ID             |      Host      | Directory
------------------------------------ | -------------- | 
---------------------------------
09055140-cc35-48e1-a53a-91c0f5eb6644 | /10.118.33.251 | 
/vm1/DiskStoreCommandsDUnitTest1

Missing Colocated Regions
    Host      | Distributed Member |               Parent Region               
| Missing Colocated Region
------------- | ------------------ | ----------------------------------------- 
| ------------------------
10.118.33.251 | VM1                | /testShowPersistentRecoveryFailuresRegion 
| /childRegion


Diffs
-----

  
geode-core/src/main/java/org/apache/geode/internal/cache/ColocationLogger.java 
0a3754763062c8f72b776085533bc1aeae87064b 
  
geode-core/src/main/java/org/apache/geode/internal/cache/partitioned/ColocatedRegionDetails.java
 PRE-CREATION 
  geode-core/src/main/java/org/apache/geode/internal/i18n/LocalizedStrings.java 
b61201ef60b4dc3ea6d42e7ae30d305e2ebb7e35 
  
geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java
 53f910854ab3e929821d19a27b74f5e6fb6d7ecc 
  
geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ShowMissingDiskStoresFunction.java
 PRE-CREATION 
  
geode-core/src/main/java/org/apache/geode/management/internal/cli/i18n/CliStrings.java
 1f0420b5eab16208135da47d862dfec3048f5548 
  
geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/ColocatedRegionDetailsJUnitTest.java
 PRE-CREATION 
  
geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommandsDUnitTest.java
 9926ae67aedb4a520d4875e180b3408b35387bcd 
  
geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/ShowMissingDiskStoresFunctionJUnitTest.java
 PRE-CREATION 
  
geode-core/src/test/resources/org/apache/geode/codeAnalysis/sanctionedDataSerializables.txt
 2240daf7ccc5b2283633687220776e5894c7235c 
  
geode-core/src/test/resources/org/apache/geode/codeAnalysis/sanctionedSerializables.txt
 58b43955c61b2abdeb6e3c3aac11f6ab568a7d55 

Diff: https://reviews.apache.org/r/52052/diff/


Testing
-------

precheckin


Thanks,

Ken Howe

Reply via email to