PHOENIX-2616 Indexes over immutable tables not marked as immutable

Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/d345b880
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/d345b880
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/d345b880

Branch: refs/heads/calcite
Commit: d345b8802590612e69181c5fcb093dce69143b67
Parents: 7795f80
Author: James Taylor <jtay...@salesforce.com>
Authored: Wed Jan 20 21:04:03 2016 -0800
Committer: James Taylor <jtay...@salesforce.com>
Committed: Wed Jan 20 21:04:03 2016 -0800

----------------------------------------------------------------------
 .../src/it/java/org/apache/phoenix/end2end/index/IndexIT.java    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/d345b880/phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexIT.java
----------------------------------------------------------------------
diff --git 
a/phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexIT.java 
b/phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexIT.java
index e369dae..6d54076 100644
--- a/phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexIT.java
+++ b/phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexIT.java
@@ -70,6 +70,7 @@ public class IndexIT extends BaseHBaseManagedTimeIT {
        
        private final boolean localIndex;
     private final boolean transactional;
+    private final boolean mutable;
        private final String tableDDLOptions;
        private final String tableName;
     private final String indexName;
@@ -79,6 +80,7 @@ public class IndexIT extends BaseHBaseManagedTimeIT {
        public IndexIT(boolean localIndex, boolean mutable, boolean 
transactional) {
                this.localIndex = localIndex;
                this.transactional = transactional;
+               this.mutable = mutable;
                StringBuilder optionBuilder = new StringBuilder();
                if (!mutable) 
                        optionBuilder.append(" IMMUTABLE_ROWS=true ");
@@ -461,7 +463,7 @@ public class IndexIT extends BaseHBaseManagedTimeIT {
                // Can't set IMMUTABLE_ROWS, MULTI_TENANT or 
DEFAULT_COLUMN_FAMILY_NAME on an index
                assertNull(indexTable.getDefaultFamilyName());
                assertFalse(indexTable.isMultiTenant());
-               assertFalse(indexTable.isImmutableRows());
+               assertEquals(mutable, !indexTable.isImmutableRows()); // Should 
match table
                if(localIndex) {
                    assertEquals(10, indexTable.getBucketNum().intValue());
                    assertTrue(indexTable.isWALDisabled());

Reply via email to