bmaidics commented on a change in pull request #1263:
URL: https://github.com/apache/hive/pull/1263#discussion_r458643492



##########
File path: ql/src/test/results/clientpositive/llap/ppd_deterministic_expr.q.out
##########
@@ -198,6 +198,9 @@ PREHOOK: query: create view viewDeterministicUDFA 
partitioned on (vpart1, vpart2
 where part1 in ('US', 'CA')
 PREHOOK: type: CREATEVIEW
 PREHOOK: Input: default@testa
+PREHOOK: Input: default@testa@part1=CA/part2=ABC/part3=300

Review comment:
       @jcamachor , I think this behavior is expected. Since in this test, CBO 
is disabled, ColumnAccessInfo will be null, so on view creation, we'll run step 
6-8 after my change. Step 7 is the optimizer, and it runs SimpleFetchOptimizer, 
which will add these partitions to read from them. Is this answers your 
concern, or maybe I misunderstood your question?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to