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/10/20 19:21:38 UTC
[2/2] 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/1be6aab2
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/1be6aab2
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/1be6aab2
Branch: refs/heads/4.x-HBase-0.98
Commit: 1be6aab22025293ae544adb3c7d5dfb53850b853
Parents: 6985735
Author: James Taylor <jt...@salesforce.com>
Authored: Fri Oct 20 12:19:37 2017 -0700
Committer: James Taylor <jt...@salesforce.com>
Committed: Fri Oct 20 12:19:37 2017 -0700
----------------------------------------------------------------------
.../end2end/UngroupedAggregateRegionObserverIT.java | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/1be6aab2/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 f1fc8d2..d64df2c 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
@@ -110,7 +110,7 @@ public class UngroupedAggregateRegionObserverIT extends ParallelStatsDisabledIT
stopCapturingIndexLog();
// uneventful - nothing should be logged
Mockito.verify(mockAppender, never())
- .doAppend((LoggingEvent) captorLoggingEvent.capture());
+ .doAppend(captorLoggingEvent.capture());
}
}
@@ -154,8 +154,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));
@@ -175,8 +175,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));
}
}