You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by ma...@apache.org on 2016/02/02 04:59:54 UTC
[20/50] [abbrv] phoenix git commit: PHOENIX-2616 Indexes over
immutable tables not marked as immutable
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 <jt...@salesforce.com>
Authored: Wed Jan 20 21:04:03 2016 -0800
Committer: James Taylor <jt...@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());