> On June 6, 2015, midnight, Aman Sinha wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/parser/UnsupportedOperatorsVisitor.java, > > line 126 > > <https://reviews.apache.org/r/34954/diff/1/?file=976827#file976827line126> > > > > There's some confusion about what exactly multiple-partition means. Is > > it multiple columns in a single partition-by or different columns in 2 or > > more partition-by clauses. Also, the error should represent something > > about different order-by criteria not supported. > > Sean Hsuan-Yi Chu wrote: > I changed it to this: > Multiple windows defined in a single select list is not supported
Can you pls change this to 'Multiple window definitions in a single SELECT list is not currently supported' - Aman ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34954/#review86868 ----------------------------------------------------------- On June 6, 2015, 12:39 a.m., Sean Hsuan-Yi Chu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34954/ > ----------------------------------------------------------- > > (Updated June 6, 2015, 12:39 a.m.) > > > Review request for drill and Aman Sinha. > > > Bugs: DRILL-3196 > https://issues.apache.org/jira/browse/DRILL-3196 > > > Repository: drill-git > > > Description > ------- > > After validation, detech if there is no than one partition > > > Diffs > ----- > > > exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/parser/UnsupportedOperatorsVisitor.java > f1ec851 > exec/java-exec/src/test/java/org/apache/drill/exec/TestWindowFunctions.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/34954/diff/ > > > Testing > ------- > > All the requested tests > > > Thanks, > > Sean Hsuan-Yi Chu > >
