You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by pv...@apache.org on 2018/05/08 10:06:09 UTC
hive git commit: HIVE-18881: Lower Logging for FSStatsAggregator
(Antal Sinkovits, via Peter Vary)
Repository: hive
Updated Branches:
refs/heads/master b0eb4151a -> 8c7e7a602
HIVE-18881: Lower Logging for FSStatsAggregator (Antal Sinkovits, via Peter Vary)
Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/8c7e7a60
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/8c7e7a60
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/8c7e7a60
Branch: refs/heads/master
Commit: 8c7e7a602542131a7db49a25980ab7ce2783b49d
Parents: b0eb415
Author: Antal Sinkovits <as...@cloudera.com>
Authored: Tue May 8 12:05:29 2018 +0200
Committer: Peter Vary <pv...@cloudera.com>
Committed: Tue May 8 12:05:29 2018 +0200
----------------------------------------------------------------------
.../org/apache/hadoop/hive/ql/stats/fs/FSStatsAggregator.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hive/blob/8c7e7a60/ql/src/java/org/apache/hadoop/hive/ql/stats/fs/FSStatsAggregator.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/stats/fs/FSStatsAggregator.java b/ql/src/java/org/apache/hadoop/hive/ql/stats/fs/FSStatsAggregator.java
index 6d2de0a..cb5fee4 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/stats/fs/FSStatsAggregator.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/stats/fs/FSStatsAggregator.java
@@ -86,7 +86,7 @@ public class FSStatsAggregator implements StatsAggregator {
@Override
public String aggregateStats(String partID, String statType) {
long counter = 0;
- Utilities.FILE_OP_LOGGER.debug("Part ID: " + partID + "\t" + statType);
+ Utilities.FILE_OP_LOGGER.debug("Part ID: {}, {}", partID, statType);
for (Map<String,Map<String,String>> statsMap : statsList) {
Map<String,String> partStat = statsMap.get(partID);
if (null == partStat) { // not all partitions are scanned in all mappers, so this could be null.
@@ -98,7 +98,7 @@ public class FSStatsAggregator implements StatsAggregator {
}
counter += Long.parseLong(statVal);
}
- Utilities.FILE_OP_LOGGER.info("Read stats for : " + partID + "\t" + statType + "\t" + counter);
+ Utilities.FILE_OP_LOGGER.info("Read stats for {}, {}, {}: ", partID, statType, counter);
return String.valueOf(counter);
}