You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Jason Huynh <jh...@pivotal.io> on 2016/12/05 22:29:14 UTC

Review Request 54396: GEODE-2183: cleaning out some print statements

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

Review request for geode, nabarun nag and Dan Smith.


Repository: geode


Description
-------

Noticed a lot of test/query results being logged and removed them.
This is not an actual fix for this ticket but was found due to working on this ticket...


Diffs
-----

  geode-core/src/test/java/org/apache/geode/cache/query/BugJUnitTest.java 6d1c52a 
  geode-core/src/test/java/org/apache/geode/cache/query/RegionJUnitTest.java 1ba858b 
  geode-core/src/test/java/org/apache/geode/cache/query/TypedIteratorJUnitTest.java 50a9773 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/INOperatorJUnitTest.java 870411b 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IUM6Bug32345ReJUnitTest.java b8e696f 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IUMJUnitTest.java 76d918d 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IUMRCompositeIteratorJUnitTest.java a8bb730 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IndependentOperandsInWhereClause2JUnitTest.java e31ee50 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexCreationJUnitTest.java afae38a 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexMaintenanceAsynchJUnitTest.java f8321eb 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexOperatorJUnitTest.java 1081807 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexPrimaryKeyUsageJUnitTest.java 6df6b93 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexUsageInNestedQueryJUnitTest.java 6eb35d6 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexUseMultFrmSnglCondJUnitTest.java cc59c0f 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexWithSngleFrmAndMultCondQryJUnitTest.java 63b5aaa 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IteratorTypeDefEmpJUnitTest.java a5c287f 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IteratorTypeDefJUnitTest.java e7f45ff 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/IteratorTypeDefaultTypesJUnitTest.java 7b57986 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/NestedQueryJUnitTest.java de2b75f 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/StructMemberAccessJUnitTest.java 0585b57 
  geode-core/src/test/java/org/apache/geode/cache/query/functional/TestNewFunctionSSorRSIntegrationTest.java 7528cfb 
  geode-core/src/test/java/org/apache/geode/cache/query/internal/index/IndexMaintainceJUnitTest.java 01ccf28 

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


Testing
-------


Thanks,

Jason Huynh


Re: Review Request 54396: GEODE-2183: cleaning out some print statements

Posted by Dan Smith <ds...@pivotal.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54396/#review158071
-----------------------------------------------------------


Ship it!




Ship It!

- Dan Smith


On Dec. 5, 2016, 10:29 p.m., Jason Huynh wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/54396/
> -----------------------------------------------------------
> 
> (Updated Dec. 5, 2016, 10:29 p.m.)
> 
> 
> Review request for geode, nabarun nag and Dan Smith.
> 
> 
> Repository: geode
> 
> 
> Description
> -------
> 
> Noticed a lot of test/query results being logged and removed them.
> This is not an actual fix for this ticket but was found due to working on this ticket...
> 
> 
> Diffs
> -----
> 
>   geode-core/src/test/java/org/apache/geode/cache/query/BugJUnitTest.java 6d1c52a 
>   geode-core/src/test/java/org/apache/geode/cache/query/RegionJUnitTest.java 1ba858b 
>   geode-core/src/test/java/org/apache/geode/cache/query/TypedIteratorJUnitTest.java 50a9773 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/INOperatorJUnitTest.java 870411b 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IUM6Bug32345ReJUnitTest.java b8e696f 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IUMJUnitTest.java 76d918d 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IUMRCompositeIteratorJUnitTest.java a8bb730 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IndependentOperandsInWhereClause2JUnitTest.java e31ee50 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexCreationJUnitTest.java afae38a 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexMaintenanceAsynchJUnitTest.java f8321eb 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexOperatorJUnitTest.java 1081807 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexPrimaryKeyUsageJUnitTest.java 6df6b93 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexUsageInNestedQueryJUnitTest.java 6eb35d6 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexUseMultFrmSnglCondJUnitTest.java cc59c0f 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IndexWithSngleFrmAndMultCondQryJUnitTest.java 63b5aaa 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IteratorTypeDefEmpJUnitTest.java a5c287f 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IteratorTypeDefJUnitTest.java e7f45ff 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/IteratorTypeDefaultTypesJUnitTest.java 7b57986 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/NestedQueryJUnitTest.java de2b75f 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/StructMemberAccessJUnitTest.java 0585b57 
>   geode-core/src/test/java/org/apache/geode/cache/query/functional/TestNewFunctionSSorRSIntegrationTest.java 7528cfb 
>   geode-core/src/test/java/org/apache/geode/cache/query/internal/index/IndexMaintainceJUnitTest.java 01ccf28 
> 
> Diff: https://reviews.apache.org/r/54396/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jason Huynh
> 
>