You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by la...@apache.org on 2019/06/23 21:09:21 UTC

[phoenix] branch master updated: PHOENIX-5365 IndexToolIT tests run multable times.

This is an automated email from the ASF dual-hosted git repository.

larsh pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/master by this push:
     new 100ff3a  PHOENIX-5365 IndexToolIT tests run multable times.
100ff3a is described below

commit 100ff3ac91e9975479bb35c1c1f2fecc06642669
Author: Lars Hofhansl <la...@apache.org>
AuthorDate: Sun Jun 23 14:09:48 2019 -0700

    PHOENIX-5365 IndexToolIT tests run multable times.
---
 .../src/it/java/org/apache/phoenix/end2end/IndexToolIT.java         | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolIT.java b/phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolIT.java
index 4605a4a..868ba35 100644
--- a/phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolIT.java
+++ b/phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolIT.java
@@ -76,6 +76,7 @@ import com.google.common.collect.Maps;
 public class IndexToolIT extends BaseUniqueNamesOwnClusterIT {
 
     private final boolean localIndex;
+    private final boolean mutable;
     private final boolean transactional;
     private final boolean directApi;
     private final String tableDDLOptions;
@@ -85,6 +86,7 @@ public class IndexToolIT extends BaseUniqueNamesOwnClusterIT {
     public IndexToolIT(String transactionProvider, boolean mutable, boolean localIndex,
             boolean directApi, boolean useSnapshot, boolean useTenantId) {
         this.localIndex = localIndex;
+        this.mutable = mutable;
         this.transactional = transactionProvider != null;
         this.directApi = directApi;
         this.useSnapshot = useSnapshot;
@@ -328,6 +330,8 @@ public class IndexToolIT extends BaseUniqueNamesOwnClusterIT {
 
     @Test
     public void testSaltedVariableLengthPK() throws Exception {
+        if (!mutable) return;
+        if (transactional) return;
         String schemaName = generateUniqueName();
         String dataTableName = generateUniqueName();
         String dataTableFullName = SchemaUtil.getTableName(schemaName, dataTableName);
@@ -372,6 +376,8 @@ public class IndexToolIT extends BaseUniqueNamesOwnClusterIT {
     @Test
     public void testSplitIndex() throws Exception {
         if (localIndex) return; // can't split local indexes
+        if (!mutable) return;
+        if (transactional) return;
         String schemaName = generateUniqueName();
         String dataTableName = generateUniqueName();
         String dataTableFullName = SchemaUtil.getTableName(schemaName, dataTableName);