Index: ql/src/test/org/apache/hadoop/hive/ql/hooks/PostExecutePrinter.java =================================================================== --- ql/src/test/org/apache/hadoop/hive/ql/hooks/PostExecutePrinter.java (revision 989949) +++ ql/src/test/org/apache/hadoop/hive/ql/hooks/PostExecutePrinter.java (working copy) @@ -109,12 +109,8 @@ console.printError("POSTHOOK: type: " + sess.getCommandType()); } - for (ReadEntity re : inputs) { - console.printError("POSTHOOK: Input: " + re.toString()); - } - for (WriteEntity we : outputs) { - console.printError("POSTHOOK: Output: " + we.toString()); - } + PreExecutePrinter.printEntities(console, inputs, "POSTHOOK: Input: "); + PreExecutePrinter.printEntities(console, outputs, "POSTHOOK: Output: "); // Also print out the generic lineage information if there is any if (linfo != null) { Index: ql/src/test/org/apache/hadoop/hive/ql/hooks/PreExecutePrinter.java =================================================================== --- ql/src/test/org/apache/hadoop/hive/ql/hooks/PreExecutePrinter.java (revision 989949) +++ ql/src/test/org/apache/hadoop/hive/ql/hooks/PreExecutePrinter.java (working copy) @@ -18,6 +18,9 @@ package org.apache.hadoop.hive.ql.hooks; +import java.util.ArrayList; +import java.util.Collections; +import java.util.List; import java.util.Set; import org.apache.hadoop.hive.ql.session.SessionState; @@ -46,12 +49,18 @@ console.printError("PREHOOK: type: " + sess.getCommandType()); } - for (ReadEntity re : inputs) { - console.printError("PREHOOK: Input: " + re.toString()); + printEntities(console, inputs, "PREHOOK: Input: "); + printEntities(console, outputs, "PREHOOK: Output: "); + } + + static void printEntities(LogHelper console, Set entities, String prefix) { + List strings = new ArrayList(); + for (Object o : entities) { + strings.add(o.toString()); } - for (WriteEntity we : outputs) { - console.printError("PREHOOK: Output: " + we.toString()); + Collections.sort(strings); + for (String s : strings) { + console.printError(prefix + s); } } - }