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:34:39 UTC

[11/40] 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/a4b0d251
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/a4b0d251
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/a4b0d251

Branch: refs/heads/4.x-HBase-1.2
Commit: a4b0d251f841b878479074b1e79cbc85f0c62cc8
Parents: e1a81cb
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:02:13 2017 -0800

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


http://git-wip-us.apache.org/repos/asf/phoenix/blob/a4b0d251/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));
         }
     }