You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hawq.apache.org by nh...@apache.org on 2015/10/02 19:25:58 UTC
incubator-hawq git commit: HAWQ-5. Remove comments
Repository: incubator-hawq
Updated Branches:
refs/heads/HAWQ-5 945b2d1bd -> 0fa0b7690
HAWQ-5. Remove comments
Project: http://git-wip-us.apache.org/repos/asf/incubator-hawq/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-hawq/commit/0fa0b769
Tree: http://git-wip-us.apache.org/repos/asf/incubator-hawq/tree/0fa0b769
Diff: http://git-wip-us.apache.org/repos/asf/incubator-hawq/diff/0fa0b769
Branch: refs/heads/HAWQ-5
Commit: 0fa0b76905851cb0987499296c9871888c793ead
Parents: 945b2d1
Author: Noa Horn <nh...@pivotal.io>
Authored: Fri Oct 2 10:25:51 2015 -0700
Committer: Noa Horn <nh...@pivotal.io>
Committed: Fri Oct 2 10:25:51 2015 -0700
----------------------------------------------------------------------
.../com/pivotal/pxf/plugins/hbase/HBaseAccessorTest.java | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-hawq/blob/0fa0b769/pxf/pxf-hbase/src/test/java/com/pivotal/pxf/plugins/hbase/HBaseAccessorTest.java
----------------------------------------------------------------------
diff --git a/pxf/pxf-hbase/src/test/java/com/pivotal/pxf/plugins/hbase/HBaseAccessorTest.java b/pxf/pxf-hbase/src/test/java/com/pivotal/pxf/plugins/hbase/HBaseAccessorTest.java
index 5341b45..51a6c02 100644
--- a/pxf/pxf-hbase/src/test/java/com/pivotal/pxf/plugins/hbase/HBaseAccessorTest.java
+++ b/pxf/pxf-hbase/src/test/java/com/pivotal/pxf/plugins/hbase/HBaseAccessorTest.java
@@ -33,7 +33,7 @@ public class HBaseAccessorTest {
HBaseAccessor accessor;
/*
- * After each test is done, close the accessor
+ * After each test is done, close the accessor
* if it was created
*/
@After
@@ -49,10 +49,10 @@ public class HBaseAccessorTest {
/*
* Test construction of HBaseAccessor.
* Actually no need for this as it is tested in all other tests
- * constructing HBaseAccessor but it serves as a simple example
+ * constructing HBaseAccessor but it serves as a simple example
* of mocking
*
- * HBaseAccessor is created and then HBaseTupleDescriptioncreation
+ * HBaseAccessor is created and then HBaseTupleDescriptioncreation
* is verified
*/
@Test
@@ -64,7 +64,7 @@ public class HBaseAccessorTest {
/*
* Test Open returns false when table has no regions
- *
+ *
* Done by returning an empty Map from getRegionLocations
* Verify Scan object doesn't contain any columns / filters
* Verify scan did not start
@@ -113,8 +113,6 @@ public class HBaseAccessorTest {
hbaseConfiguration = mock(Configuration.class);
when(HBaseConfiguration.create()).thenReturn(hbaseConfiguration);
- //table = mock(HTable.class);
- //PowerMockito.whenNew(HTable.class).withArguments(hbaseConfiguration, tableName.getBytes()).thenReturn(table);
// Make sure we mock static functions in ConnectionFactory
PowerMockito.mockStatic(ConnectionFactory.class);