> On Feb. 3, 2015, 8:25 p.m., John Pullokkaran wrote: > > ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorUtils.java, line 175 > > <https://reviews.apache.org/r/30254/diff/4/?file=846475#file846475line175> > > > > What happens when you have more than one operator for the given class? > > wouldn't it overwrite the old value?
It is a multimap, so it will have a list of values (in this case operators) for every key (in this case, classes). - Jesús ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30254/#review70820 ----------------------------------------------------------- On Feb. 3, 2015, 7:51 p.m., Jesús Camacho Rodríguez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30254/ > ----------------------------------------------------------- > > (Updated Feb. 3, 2015, 7:51 p.m.) > > > Review request for hive and John Pullokkaran. > > > Bugs: HIVE-9444 > https://issues.apache.org/jira/browse/HIVE-9444 > > > Repository: hive-git > > > Description > ------- > > HIVE-9444 > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorUtils.java > 4d9d1da77394125cfcb9ac9ccf1c00528664b981 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/GlobalLimitOptimizer.java > c9848dacd1a02db321583c2b91eb6d7317c295ff > ql/src/java/org/apache/hadoop/hive/ql/plan/GroupByDesc.java > f031b281ff64706cdbbc1695a6f51282127487fa > ql/src/java/org/apache/hadoop/hive/ql/plan/ReduceSinkDesc.java > 43f8321107304c38ecd9f603cd3f8ce43e8496c8 > > Diff: https://reviews.apache.org/r/30254/diff/ > > > Testing > ------- > > Existing tests (global_limit.q). > > > Thanks, > > Jesús Camacho Rodríguez > >
