You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by ja...@apache.org on 2017/11/15 18:48:07 UTC

[12/37] phoenix git commit: PHOENIX-4310 Remove unnecessary casts in UngroupedAggregateRegionObserverIT

PHOENIX-4310 Remove unnecessary casts in UngroupedAggregateRegionObserverIT


Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/3f453e15
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/3f453e15
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/3f453e15

Branch: refs/heads/4.x-HBase-1.1
Commit: 3f453e152c2315e68fb9afe702fd650e4d0d3bef
Parents: a49aed8
Author: James Taylor <jt...@salesforce.com>
Authored: Fri Oct 20 12:19:37 2017 -0700
Committer: James Taylor <jt...@salesforce.com>
Committed: Wed Nov 15 10:41:23 2017 -0800

----------------------------------------------------------------------
 .../end2end/UngroupedAggregateRegionObserverIT.java       | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/3f453e15/phoenix-core/src/it/java/org/apache/phoenix/end2end/UngroupedAggregateRegionObserverIT.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/it/java/org/apache/phoenix/end2end/UngroupedAggregateRegionObserverIT.java b/phoenix-core/src/it/java/org/apache/phoenix/end2end/UngroupedAggregateRegionObserverIT.java
index 3efd40e..0ae1bb5 100644
--- a/phoenix-core/src/it/java/org/apache/phoenix/end2end/UngroupedAggregateRegionObserverIT.java
+++ b/phoenix-core/src/it/java/org/apache/phoenix/end2end/UngroupedAggregateRegionObserverIT.java
@@ -82,7 +82,7 @@ public class UngroupedAggregateRegionObserverIT extends ParallelStatsDisabledIT
             stopCapturingIndexLog();
             // uneventful - nothing should be logged
             Mockito.verify(mockAppender, never())
-                    .doAppend((LoggingEvent) captorLoggingEvent.capture());
+                    .doAppend(captorLoggingEvent.capture());
         }
     }
 
@@ -126,8 +126,8 @@ public class UngroupedAggregateRegionObserverIT extends ParallelStatsDisabledIT
         ungroupedObserver.clearTsOnDisabledIndexes(tableToCompact);
         stopCapturingIndexLog();
         // an event should've been logged
-        Mockito.verify(mockAppender).doAppend((LoggingEvent) captorLoggingEvent.capture());
-        LoggingEvent loggingEvent = (LoggingEvent) captorLoggingEvent.getValue();
+        Mockito.verify(mockAppender).doAppend(captorLoggingEvent.capture());
+        LoggingEvent loggingEvent = captorLoggingEvent.getValue();
         assertThat(loggingEvent.getLevel(), is(Level.INFO));
         // index should be permanently disabled (disabletime of 0)
         assertTrue(TestUtil.checkIndexState(pConn, indexTableFullName, PIndexState.DISABLE, 0L));
@@ -147,8 +147,8 @@ public class UngroupedAggregateRegionObserverIT extends ParallelStatsDisabledIT
             ungroupedObserver.clearTsOnDisabledIndexes(nonPhoenixTable);
             stopCapturingIndexLog();
             // a debug level event should've been logged
-            Mockito.verify(mockAppender).doAppend((LoggingEvent) captorLoggingEvent.capture());
-            LoggingEvent loggingEvent = (LoggingEvent) captorLoggingEvent.getValue();
+            Mockito.verify(mockAppender).doAppend(captorLoggingEvent.capture());
+            LoggingEvent loggingEvent = captorLoggingEvent.getValue();
             assertThat(loggingEvent.getLevel(), is(Level.DEBUG));
         }
     }