-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47899/
-----------------------------------------------------------
Review request for geode, Jens Deppe, Jinmei Liao, Kevin Duling, and Dan Smith.
Bugs: GEODE-1452
https://issues.apache.org/jira/browse/GEODE-1452
Repository: geode
Description
-------
GEODE-1452: Annotate disabled test methods with @Ignore and @Test
* remove some invalid tests
* remove some empty javadocs
* remove some dead code
Diffs
-----
extensions/geode-modules-hibernate/src/test/java/com/gemstone/gemfire/modules/HibernateJUnitTest.java
9a86f10
extensions/geode-modules-hibernate/src/test/java/com/gemstone/gemfire/modules/SecondVMTest.java
e3b6480
geode-core/src/test/java/com/gemstone/gemfire/cache/mapInterface/ExceptionHandlingJUnitTest.java
fc3c35c
geode-core/src/test/java/com/gemstone/gemfire/cache/query/QueryServiceJUnitTest.java
0c34de7
geode-core/src/test/java/com/gemstone/gemfire/cache/query/functional/CustomerOptimizationsJUnitTest.java
26e3aff
geode-core/src/test/java/com/gemstone/gemfire/cache/query/functional/INOperatorJUnitTest.java
fbc887c
geode-core/src/test/java/com/gemstone/gemfire/cache/query/functional/IUMRCompositeIteratorJUnitTest.java
7c627c1
geode-core/src/test/java/com/gemstone/gemfire/cache/query/functional/LimitClauseJUnitTest.java
5280dfb
geode-core/src/test/java/com/gemstone/gemfire/cache/query/functional/MiscJUnitTest.java
4bd3258
geode-core/src/test/java/com/gemstone/gemfire/cache/query/functional/MultiRegionIndexUsageJUnitTest.java
4b6071c
geode-core/src/test/java/com/gemstone/gemfire/cache/query/functional/NestedQueryJUnitTest.java
11dcb0e
geode-core/src/test/java/com/gemstone/gemfire/cache/query/functional/ResultsDataSerializabilityJUnitTest.java
cb45bad
geode-core/src/test/java/com/gemstone/gemfire/cache/query/internal/CompiledJunctionInternalsJUnitTest.java
c3e1a45
geode-core/src/test/java/com/gemstone/gemfire/cache/query/internal/QCompilerJUnitTest.java
bcf0b65
geode-core/src/test/java/com/gemstone/gemfire/cache/query/internal/index/IndexMaintainceJUnitTest.java
3ee758b
geode-core/src/test/java/com/gemstone/gemfire/cache/query/transaction/QueryAndJtaJUnitTest.java
162ffb1
geode-core/src/test/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystemJUnitTest.java
827091f
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/Bug34179TooManyFilesOpenJUnitTest.java
4eea76d
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/LIFOEvictionAlgoEnabledRegionJUnitTest.java
9df2468
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/OplogJUnitTest.java
d1be04d
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ha/BlockingHARegionJUnitTest.java
4b36b79
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ha/BlockingHARegionQueueJUnitTest.java
ffb4771
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ha/HARQAddOperationJUnitTest.java
1c7b0d0
geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ha/HARegionQueueJUnitTest.java
b7fae40
Diff: https://reviews.apache.org/r/47899/diff/
Testing
-------
precheckin (100% green)
Thanks,
Kirk Lund