You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@whirr.apache.org by to...@apache.org on 2011/01/16 07:47:11 UTC
svn commit: r1059499 - in /incubator/whirr/trunk: ./
cli/src/main/java/org/apache/whirr/cli/command/
core/src/main/java/org/apache/whirr/service/
services/cassandra/src/test/java/org/apache/whirr/service/cassandra/integration/
services/hadoop/src/test/...
Author: tomwhite
Date: Sun Jan 16 06:47:10 2011
New Revision: 1059499
URL: http://svn.apache.org/viewvc?rev=1059499&view=rev
Log:
WHIRR-211. Fix checkstyle errors for 0.3.0.
Modified:
incubator/whirr/trunk/CHANGES.txt
incubator/whirr/trunk/cli/src/main/java/org/apache/whirr/cli/command/AbstractClusterSpecCommand.java
incubator/whirr/trunk/core/src/main/java/org/apache/whirr/service/ClusterSpec.java
incubator/whirr/trunk/services/cassandra/src/test/java/org/apache/whirr/service/cassandra/integration/CassandraServiceTest.java
incubator/whirr/trunk/services/hadoop/src/test/java/org/apache/whirr/service/hadoop/integration/HadoopServiceController.java
incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/BasicServerClusterActionHandler.java
incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseMasterClusterActionHandler.java
incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseRegionServerClusterActionHandler.java
incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceController.java
incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceTest.java
incubator/whirr/trunk/services/zookeeper/src/test/java/org/apache/whirr/service/zookeeper/integration/ZooKeeperServiceTest.java
Modified: incubator/whirr/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/CHANGES.txt?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/CHANGES.txt (original)
+++ incubator/whirr/trunk/CHANGES.txt Sun Jan 16 06:47:10 2011
@@ -118,6 +118,8 @@ Trunk (unreleased changes)
WHIRR-210. Remove unneeded dependent libraries. (tomwhite)
+ WHIRR-211. Fix checkstyle errors for 0.3.0. (tomwhite)
+
Release 0.2.0 - 2010-11-04
NEW FEATURES
Modified: incubator/whirr/trunk/cli/src/main/java/org/apache/whirr/cli/command/AbstractClusterSpecCommand.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/cli/src/main/java/org/apache/whirr/cli/command/AbstractClusterSpecCommand.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/cli/src/main/java/org/apache/whirr/cli/command/AbstractClusterSpecCommand.java (original)
+++ incubator/whirr/trunk/cli/src/main/java/org/apache/whirr/cli/command/AbstractClusterSpecCommand.java Sun Jan 16 06:47:10 2011
@@ -20,7 +20,6 @@ package org.apache.whirr.cli.command;
import static org.apache.whirr.service.ClusterSpec.Property.CLUSTER_NAME;
import static org.apache.whirr.service.ClusterSpec.Property.IDENTITY;
-import static org.apache.whirr.service.ClusterSpec.Property.SERVICE_NAME;
import java.util.EnumSet;
import java.util.Map;
Modified: incubator/whirr/trunk/core/src/main/java/org/apache/whirr/service/ClusterSpec.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/core/src/main/java/org/apache/whirr/service/ClusterSpec.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/core/src/main/java/org/apache/whirr/service/ClusterSpec.java (original)
+++ incubator/whirr/trunk/core/src/main/java/org/apache/whirr/service/ClusterSpec.java Sun Jan 16 06:47:10 2011
@@ -35,7 +35,6 @@ import java.io.FileReader;
import java.io.IOException;
import java.io.UnsupportedEncodingException;
import java.net.URLEncoder;
-import java.util.Arrays;
import java.util.Iterator;
import java.util.List;
import java.util.Map;
Modified: incubator/whirr/trunk/services/cassandra/src/test/java/org/apache/whirr/service/cassandra/integration/CassandraServiceTest.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/services/cassandra/src/test/java/org/apache/whirr/service/cassandra/integration/CassandraServiceTest.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/services/cassandra/src/test/java/org/apache/whirr/service/cassandra/integration/CassandraServiceTest.java (original)
+++ incubator/whirr/trunk/services/cassandra/src/test/java/org/apache/whirr/service/cassandra/integration/CassandraServiceTest.java Sun Jan 16 06:47:10 2011
@@ -27,7 +27,6 @@ import java.util.Map;
import java.util.Set;
import org.apache.cassandra.thrift.Cassandra;
-import org.apache.cassandra.thrift.TokenRange;
import org.apache.commons.configuration.CompositeConfiguration;
import org.apache.commons.configuration.PropertiesConfiguration;
import org.apache.thrift.TException;
@@ -40,7 +39,6 @@ import org.apache.whirr.service.Cluster.
import org.apache.whirr.service.ClusterSpec;
import org.apache.whirr.service.Service;
import org.apache.whirr.service.cassandra.CassandraClusterActionHandler;
-import org.apache.whirr.ssh.KeyPair;
import org.junit.After;
import org.junit.Before;
import org.junit.Test;
Modified: incubator/whirr/trunk/services/hadoop/src/test/java/org/apache/whirr/service/hadoop/integration/HadoopServiceController.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/services/hadoop/src/test/java/org/apache/whirr/service/hadoop/integration/HadoopServiceController.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/services/hadoop/src/test/java/org/apache/whirr/service/hadoop/integration/HadoopServiceController.java (original)
+++ incubator/whirr/trunk/services/hadoop/src/test/java/org/apache/whirr/service/hadoop/integration/HadoopServiceController.java Sun Jan 16 06:47:10 2011
@@ -19,7 +19,6 @@
package org.apache.whirr.service.hadoop.integration;
import java.io.IOException;
-import java.util.Map;
import java.util.Map.Entry;
import org.apache.commons.configuration.CompositeConfiguration;
@@ -35,7 +34,6 @@ import org.apache.whirr.service.Cluster;
import org.apache.whirr.service.ClusterSpec;
import org.apache.whirr.service.Service;
import org.apache.whirr.service.hadoop.HadoopProxy;
-import org.apache.whirr.ssh.KeyPair;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
Modified: incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/BasicServerClusterActionHandler.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/BasicServerClusterActionHandler.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/BasicServerClusterActionHandler.java (original)
+++ incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/BasicServerClusterActionHandler.java Sun Jan 16 06:47:10 2011
@@ -18,16 +18,20 @@
package org.apache.whirr.service.hbase;
-import org.apache.whirr.net.DnsUtil;
-import org.apache.whirr.service.*;
-import org.apache.whirr.service.jclouds.FirewallSettings;
-import org.apache.whirr.service.zookeeper.ZooKeeperCluster;
-import org.jclouds.compute.ComputeServiceContext;
+import static org.apache.whirr.service.RolePredicates.role;
import java.io.IOException;
import java.net.InetAddress;
-import static org.apache.whirr.service.RolePredicates.role;
+import org.apache.whirr.net.DnsUtil;
+import org.apache.whirr.service.Cluster;
+import org.apache.whirr.service.ClusterActionEvent;
+import org.apache.whirr.service.ClusterActionHandlerSupport;
+import org.apache.whirr.service.ClusterSpec;
+import org.apache.whirr.service.ComputeServiceContextBuilder;
+import org.apache.whirr.service.jclouds.FirewallSettings;
+import org.apache.whirr.service.zookeeper.ZooKeeperCluster;
+import org.jclouds.compute.ComputeServiceContext;
/**
* Provides a base class for servers like REST or Avro.
Modified: incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseMasterClusterActionHandler.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseMasterClusterActionHandler.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseMasterClusterActionHandler.java (original)
+++ incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseMasterClusterActionHandler.java Sun Jan 16 06:47:10 2011
@@ -18,12 +18,25 @@
package org.apache.whirr.service.hbase;
+import static org.apache.whirr.service.RolePredicates.role;
+
import com.google.common.base.Charsets;
import com.google.common.base.Joiner;
import com.google.common.io.Files;
+
+import java.io.File;
+import java.io.IOException;
+import java.net.InetAddress;
+import java.util.Map.Entry;
+import java.util.Properties;
+
import org.apache.whirr.net.DnsUtil;
-import org.apache.whirr.service.*;
+import org.apache.whirr.service.Cluster;
import org.apache.whirr.service.Cluster.Instance;
+import org.apache.whirr.service.ClusterActionEvent;
+import org.apache.whirr.service.ClusterActionHandlerSupport;
+import org.apache.whirr.service.ClusterSpec;
+import org.apache.whirr.service.ComputeServiceContextBuilder;
import org.apache.whirr.service.hadoop.HadoopProxy;
import org.apache.whirr.service.jclouds.FirewallSettings;
import org.apache.whirr.service.zookeeper.ZooKeeperCluster;
@@ -31,14 +44,6 @@ import org.jclouds.compute.ComputeServic
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import java.io.File;
-import java.io.IOException;
-import java.net.InetAddress;
-import java.util.Map.Entry;
-import java.util.Properties;
-
-import static org.apache.whirr.service.RolePredicates.role;
-
public class HBaseMasterClusterActionHandler extends ClusterActionHandlerSupport {
private static final Logger LOG =
Modified: incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseRegionServerClusterActionHandler.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseRegionServerClusterActionHandler.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseRegionServerClusterActionHandler.java (original)
+++ incubator/whirr/trunk/services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseRegionServerClusterActionHandler.java Sun Jan 16 06:47:10 2011
@@ -18,18 +18,22 @@
package org.apache.whirr.service.hbase;
+import static org.apache.whirr.service.RolePredicates.role;
+
+import java.io.IOException;
+import java.net.InetAddress;
+
import org.apache.whirr.net.DnsUtil;
-import org.apache.whirr.service.*;
+import org.apache.whirr.service.Cluster;
import org.apache.whirr.service.Cluster.Instance;
+import org.apache.whirr.service.ClusterActionEvent;
+import org.apache.whirr.service.ClusterActionHandlerSupport;
+import org.apache.whirr.service.ClusterSpec;
+import org.apache.whirr.service.ComputeServiceContextBuilder;
import org.apache.whirr.service.jclouds.FirewallSettings;
import org.apache.whirr.service.zookeeper.ZooKeeperCluster;
import org.jclouds.compute.ComputeServiceContext;
-import java.io.IOException;
-import java.net.InetAddress;
-
-import static org.apache.whirr.service.RolePredicates.role;
-
public class HBaseRegionServerClusterActionHandler extends ClusterActionHandlerSupport {
public static final String ROLE = "hbase-regionserver";
Modified: incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceController.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceController.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceController.java (original)
+++ incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceController.java Sun Jan 16 06:47:10 2011
@@ -29,12 +29,10 @@ import org.apache.whirr.service.Cluster;
import org.apache.whirr.service.ClusterSpec;
import org.apache.whirr.service.Service;
import org.apache.whirr.service.hadoop.HadoopProxy;
-import org.apache.whirr.ssh.KeyPair;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import java.io.IOException;
-import java.util.Map;
import java.util.Map.Entry;
public class HBaseServiceController {
Modified: incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceTest.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceTest.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceTest.java (original)
+++ incubator/whirr/trunk/services/hbase/src/test/java/org/apache/whirr/service/hbase/integration/HBaseServiceTest.java Sun Jan 16 06:47:10 2011
@@ -18,24 +18,26 @@
package org.apache.whirr.service.hbase.integration;
+import static org.junit.Assert.assertTrue;
+
import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.hbase.HBaseTestingUtility;
-import org.apache.hadoop.hbase.client.*;
+import org.apache.hadoop.hbase.client.HTable;
+import org.apache.hadoop.hbase.client.Put;
+import org.apache.hadoop.hbase.client.Result;
+import org.apache.hadoop.hbase.client.ResultScanner;
+import org.apache.hadoop.hbase.client.Scan;
import org.apache.hadoop.hbase.util.Bytes;
import org.junit.AfterClass;
import org.junit.BeforeClass;
import org.junit.Test;
-import static junit.framework.Assert.assertEquals;
-import static junit.framework.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
-
public class HBaseServiceTest {
- private static byte [] ROW = Bytes.toBytes("testRow");
- private static byte [] FAMILY = Bytes.toBytes("testFamily");
- private static byte [] QUALIFIER = Bytes.toBytes("testQualifier");
- private static byte [] VALUE = Bytes.toBytes("testValue");
+ private static final byte [] ROW = Bytes.toBytes("testRow");
+ private static final byte [] FAMILY = Bytes.toBytes("testFamily");
+ private static final byte [] QUALIFIER = Bytes.toBytes("testQualifier");
+ private static final byte [] VALUE = Bytes.toBytes("testValue");
private static HBaseServiceController controller =
HBaseServiceController.getInstance();
@@ -54,13 +56,13 @@ public class HBaseServiceTest {
public void test() throws Exception {
Configuration conf = controller.getConfiguration();
HBaseTestingUtility testUtil = new HBaseTestingUtility(conf);
- byte [] TABLE = Bytes.toBytes("testtable");
- HTable ht = testUtil.createTable(TABLE, FAMILY);
+ byte [] table = Bytes.toBytes("testtable");
+ HTable ht = testUtil.createTable(table, FAMILY);
Put put = new Put(ROW);
put.add(FAMILY, QUALIFIER, VALUE);
ht.put(put);
Scan scan = new Scan();
- scan.addColumn(FAMILY, TABLE);
+ scan.addColumn(FAMILY, table);
ResultScanner scanner = ht.getScanner(scan);
Result result = scanner.next();
assertTrue("Expected null result", result == null);
Modified: incubator/whirr/trunk/services/zookeeper/src/test/java/org/apache/whirr/service/zookeeper/integration/ZooKeeperServiceTest.java
URL: http://svn.apache.org/viewvc/incubator/whirr/trunk/services/zookeeper/src/test/java/org/apache/whirr/service/zookeeper/integration/ZooKeeperServiceTest.java?rev=1059499&r1=1059498&r2=1059499&view=diff
==============================================================================
--- incubator/whirr/trunk/services/zookeeper/src/test/java/org/apache/whirr/service/zookeeper/integration/ZooKeeperServiceTest.java (original)
+++ incubator/whirr/trunk/services/zookeeper/src/test/java/org/apache/whirr/service/zookeeper/integration/ZooKeeperServiceTest.java Sun Jan 16 06:47:10 2011
@@ -21,7 +21,6 @@ package org.apache.whirr.service.zookeep
import static junit.framework.Assert.assertEquals;
import java.io.IOException;
-import java.util.Map;
import java.util.concurrent.CountDownLatch;
import org.apache.commons.configuration.CompositeConfiguration;
@@ -30,7 +29,6 @@ import org.apache.whirr.service.Cluster;
import org.apache.whirr.service.ClusterSpec;
import org.apache.whirr.service.Service;
import org.apache.whirr.service.zookeeper.ZooKeeperCluster;
-import org.apache.whirr.ssh.KeyPair;
import org.apache.zookeeper.CreateMode;
import org.apache.zookeeper.WatchedEvent;
import org.apache.zookeeper.Watcher;