[ 
https://issues.apache.org/jira/browse/PHOENIX-1728?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14508305#comment-14508305
 ] 

James Taylor commented on PHOENIX-1728:
---------------------------------------

Thanks for the patch, [~cody.mar...@gmail.com]. Here's some feedback:
- I'm seeing the following compiler error in WorkloadExecutor in Eclipse:
{code}
    The value of the field WorkloadExecutor.properties is not used      
WorkloadExecutor.java   /pherf/src/main/java/org/apache/phoenix/pherf/workload  
line 42 Java Problem
{code}
- Looks like the unit tests aren't being run when you do a mvn package - 
there's another skip that you need to remove in your pom and then they run
- I can't get your IT tests to run through mvn test. Might be related, but 
src/it/java is not showing up in Eclipse as a directory containing Java files. 
Looking at your pom, it looks very different than the other poms - I tried 
various things, but had no luck. Someone like [~jesse_yates] or [~gabriel.reid] 
would probably be able to spot it. They should run with this command:
{code}
mvn test -Dtest=org.apache.phoenix.pherf.*IT -DfailIfNoTests=false
{code}

> Pherf - Make tests use mini cluster
> -----------------------------------
>
>                 Key: PHOENIX-1728
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1728
>             Project: Phoenix
>          Issue Type: Sub-task
>            Reporter: Cody Marcel
>            Priority: Minor
>              Labels: newbie
>
> Some unit tests currently depend on a cluster being available or they will 
> fail. Make these tests use mini cluster.
> Tests are currently disabled in the build. We need to enable these.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to