Github user JamesRTaylor commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/303#discussion_r191925360
  
    --- Diff: 
phoenix-core/src/it/java/org/apache/phoenix/end2end/ExplainPlanWithStatsEnabledIT.java
 ---
    @@ -1202,33 +1202,44 @@ private void 
testUseStatsForParallelizationOnSaltedTable(boolean useStatsFlag, b
             assertEquals("B", rs.getString(1));
         }
     
    -   @Test
    -   public void testUseStatsForParallelizationProperyOnViewIndex() throws 
SQLException {
    -           String tableName = generateUniqueName();
    -           String viewName = generateUniqueName();
    -           String tenantViewName = generateUniqueName();
    -           String viewIndexName = generateUniqueName();
    -           boolean useStats = !DEFAULT_USE_STATS_FOR_PARALLELIZATION;
    -           try (Connection conn = DriverManager.getConnection(getUrl())) {
    -                   conn.createStatement()
    -                                   .execute("create table " + tableName
    -                                                   + "(tenantId CHAR(15) 
NOT NULL, pk1 integer NOT NULL, v varchar CONSTRAINT PK PRIMARY KEY "
    -                                                   + "(tenantId, pk1)) 
MULTI_TENANT=true");
    -                   try (Connection tenantConn = 
getTenantConnection("tenant1")) {
    -                           conn.createStatement().execute("CREATE VIEW " + 
viewName + " AS SELECT * FROM " + tableName);
    -                           conn.createStatement().execute("CREATE INDEX " 
+ viewIndexName + " on " + viewName + " (v) ");
    -                           tenantConn.createStatement().execute("CREATE 
VIEW " + tenantViewName + " AS SELECT * FROM " + viewName);
    -                           conn.createStatement()
    -                                           .execute("ALTER TABLE " + 
tableName + " set USE_STATS_FOR_PARALLELIZATION=" + useStats);
    -                           // fetch the latest view ptable 
    -                           PhoenixRuntime.getTableNoCache(tenantConn, 
viewName);
    -                           PhoenixConnection phxConn = 
conn.unwrap(PhoenixConnection.class);
    -                           PTable viewIndex = phxConn.getTable(new 
PTableKey(phxConn.getTenantId(), viewIndexName));
    -                           assertEquals("USE_STATS_FOR_PARALLELIZATION 
property set incorrectly", useStats,
    -                                           PhoenixConfigurationUtil
    -                                                           
.getStatsForParallelizationProp(tenantConn.unwrap(PhoenixConnection.class), 
viewIndex));
    -                   }
    -           }
    -   }
    +    @Test
    +    public void testUseStatsForParallelizationProperyOnViewIndex() throws 
SQLException {
    +        String tableName = generateUniqueName();
    +        String viewName = generateUniqueName();
    +        String tenantViewName = generateUniqueName();
    +        String viewIndexName = generateUniqueName();
    +        boolean useStats = !DEFAULT_USE_STATS_FOR_PARALLELIZATION;
    +        try (Connection conn = DriverManager.getConnection(getUrl())) {
    +            conn.createStatement()
    +                    .execute("create table " + tableName
    +                            + "(tenantId CHAR(15) NOT NULL, pk1 integer 
NOT NULL, v varchar CONSTRAINT PK PRIMARY KEY "
    +                            + "(tenantId, pk1)) MULTI_TENANT=true");
    +            try (Connection tenantConn = getTenantConnection("tenant1")) {
    +                conn.createStatement().execute("CREATE VIEW " + viewName + 
" AS SELECT * FROM " + tableName);
    +                conn.createStatement().execute("CREATE INDEX " + 
viewIndexName + " on " + viewName + " (v) ");
    +                tenantConn.createStatement().execute("CREATE VIEW " + 
tenantViewName + " AS SELECT * FROM " + viewName);
    +                conn.createStatement()
    +                        .execute("ALTER TABLE " + tableName + " set 
USE_STATS_FOR_PARALLELIZATION=" + useStats);
    +                // changing a property on a base table does not change the 
property on a view
    --- End diff --
    
    Shouldn't setting a property on the base table impact the view as well? In 
this case, USE_STATS_FOR_PARALLELIZATION only makes sense to set on a physical 
table. I think we only look it up from a physical table as well, so this is 
somewhat moot, but in general, I'd think that setting a property on a base 
table should be seen by it's views if the property has not been set there.


---

Reply via email to