> On Sept. 25, 2015, 6:52 p.m., John Pullokkaran wrote: > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java, > > line 315 > > <https://reviews.apache.org/r/38768/diff/1/?file=1084857#file1084857line315> > > > > Shouldn't we fix the API to provide the stats in the same order as it > > was given? > > > > In this case cols are given in the same order as it appears in the > > table. > > Chaoyu Tang wrote: > You meant to fix the HMS getTableColumnStatistics API to return the stats > in the order of the requested columns? If so, what if some requested columns > do not have stats and miss in the returned list? In this case, the ordering > seems not much necessary or helpful. In addition, not all > getTableColumnStatistics callers are interested in the order of the returned > stats, so I think it probably more reasonable to leave this ordering burden > to its caller instead of API itself. Does it makes sense? Please let me know. > > John Pullokkaran wrote: > Thats fair.
Thanks. Actually the column stats is not returned in the order of columns specified in the table. It depends on the order that the column statistics were built and sql/jdo queries as well. How about my answer to your 2nd comment, does that make sense? - Chaoyu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38768/#review100654 ----------------------------------------------------------- On Sept. 25, 2015, 6:37 p.m., Chaoyu Tang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38768/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2015, 6:37 p.m.) > > > Review request for hive. > > > Bugs: HIVE-11964 > https://issues.apache.org/jira/browse/HIVE-11964 > > > Repository: hive-git > > > Description > ------- > > The stats in hiveColStats should be ordered in that of the columns in > nonPartColIndxsThatRqrStats/nonPartColNamesThatRqrStats before being put in > the hiveColStatsMap > === > for (int i = 0; i < hiveColStats.size(); i++) { > hiveColStatsMap.put(nonPartColIndxsThatRqrStats.get(i), > hiveColStats.get(i)); > } > > Since hiveColStats = StatsUtils.getTableColumnStats(hiveTblMetadata, > hiveNonPartitionCols, nonPartColNamesThatRqrStats); might not return the > stats in same order as those in nonPartColNamesThatRqrStats. > > > Diffs > ----- > > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java > 6c0bd25 > > Diff: https://reviews.apache.org/r/38768/diff/ > > > Testing > ------- > > Precommit tests > > > Thanks, > > Chaoyu Tang > >
