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:23:13 UTC
phoenix git commit: PHOENIX-4310 Remove unnecessary casts in
UngroupedAggregateRegionObserverIT
Repository: phoenix
Updated Branches:
refs/heads/master dc432b975 -> 7cdcb2313
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/7cdcb231
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/7cdcb231
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/7cdcb231
Branch: refs/heads/master
Commit: 7cdcb2313b08d2eaeb775f0c989642f8d416cfb6
Parents: dc432b9
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:22:05 2017 -0700
----------------------------------------------------------------------
.../end2end/UngroupedAggregateRegionObserverIT.java | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/7cdcb231/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));
}
}