You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@phoenix.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2015/09/11 21:45:45 UTC
[jira] [Commented] (PHOENIX-2252) Pherf - QueryExecutor creates a
Differ for performance test
[ https://issues.apache.org/jira/browse/PHOENIX-2252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14741444#comment-14741444 ]
Hudson commented on PHOENIX-2252:
---------------------------------
FAILURE: Integrated in Phoenix-master #897 (See [https://builds.apache.org/job/Phoenix-master/897/])
PHOENIX-2252 - QueryExecutor creates a Differ for performance test (cmarcel: rev 29535f342ff53dbdef03dc5822a5e7583be32cbb)
* phoenix-pherf/src/main/java/org/apache/phoenix/pherf/workload/QueryExecutor.java
> Pherf - QueryExecutor creates a Differ for performance test
> -----------------------------------------------------------
>
> Key: PHOENIX-2252
> URL: https://issues.apache.org/jira/browse/PHOENIX-2252
> Project: Phoenix
> Issue Type: Bug
> Affects Versions: 4.5.0
> Reporter: Cody Marcel
> Assignee: Cody Marcel
> Fix For: 4.5.2
>
> Attachments: PHOENIX-2252.patch
>
>
> Logic is reversed on the QueryExecutor. It should launch the differ when !workloadExecutor.isPerformance().
> if (workloadExecutor.isPerformance()) {
> thread =
> new MultithreadedDiffer(threadTime.getThreadName(), queryResult, threadTime,
> querySet.getNumberOfExecutions(), querySet.getExecutionDurationInMs());
> } else {
> thread =
> new MultiThreadedRunner(threadTime.getThreadName(), queryResult,
> dataModelResult, threadTime, querySet.getNumberOfExecutions(),
> querySet.getExecutionDurationInMs());
> }
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)