Review Request 32799: DRILL-2596: CTAS plan should be shown in the Web UI

2015-04-03 Thread Sean Hsuan-Yi Chu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32799/ --- Review request for drill, Aman Sinha and Jason Altekruse. Bugs: DRILL-2596

Re: Review Request 32799: DRILL-2596: CTAS plan should be shown in the Web UI

2015-04-03 Thread Aman Sinha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32799/#review78810 --- exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/Dril

Re: Review Request 32799: DRILL-2596: CTAS plan should be shown in the Web UI

2015-04-03 Thread Sean Hsuan-Yi Chu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32799/ --- (Updated April 3, 2015, 8:07 p.m.) Review request for drill, Aman Sinha and Jas

Re: Review Request 32799: DRILL-2596: CTAS plan should be shown in the Web UI

2015-04-03 Thread Sean Hsuan-Yi Chu
> On April 3, 2015, 5:56 p.m., Aman Sinha wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java, > > line 137 > > > > > > Is it necessary to create a separate getSqlHandler meth

Re: Review Request 32799: DRILL-2596: CTAS plan should be shown in the Web UI

2015-04-03 Thread Aman Sinha
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32799/#review78868 --- Ship it! Ship It! - Aman Sinha On April 3, 2015, 8:07 p.m., Sean