-----------------------------------------------------------
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
