You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ab...@apache.org on 2021/12/13 10:31:22 UTC
[hive] branch master updated: HIVE-25794: CombineHiveRecordReader: log statements in a loop leads to memory pressure (#2861) (Laszlo Bodor reviewed by Rajesh Balamohan, David Mollitor, Panagiotis Garefalakis, Syed Shameerur Rahman)
This is an automated email from the ASF dual-hosted git repository.
abstractdog pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git
The following commit(s) were added to refs/heads/master by this push:
new 9b3627f HIVE-25794: CombineHiveRecordReader: log statements in a loop leads to memory pressure (#2861) (Laszlo Bodor reviewed by Rajesh Balamohan, David Mollitor, Panagiotis Garefalakis, Syed Shameerur Rahman)
9b3627f is described below
commit 9b3627f92447297c4a3c857d46cd7fc2445495b4
Author: Bodor Laszlo <bo...@gmail.com>
AuthorDate: Mon Dec 13 11:31:09 2021 +0100
HIVE-25794: CombineHiveRecordReader: log statements in a loop leads to memory pressure (#2861) (Laszlo Bodor reviewed by Rajesh Balamohan, David Mollitor, Panagiotis Garefalakis, Syed Shameerur Rahman)
---
ql/src/java/org/apache/hadoop/hive/ql/io/CombineHiveRecordReader.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/io/CombineHiveRecordReader.java b/ql/src/java/org/apache/hadoop/hive/ql/io/CombineHiveRecordReader.java
index f10460df..7a68786 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/io/CombineHiveRecordReader.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/io/CombineHiveRecordReader.java
@@ -113,7 +113,7 @@ public class CombineHiveRecordReader<K extends WritableComparable, V extends Wri
for (Path path : hsplit.getPaths()) {
PartitionDesc otherPart = HiveFileFormatUtils.getFromPathRecursively(
pathToPartInfo, path, cache);
- LOG.debug("Found spec for " + path + " " + otherPart + " from " + pathToPartInfo);
+ LOG.debug("Found spec for {} {} from {}", path, otherPart, pathToPartInfo);
if (part == null) {
part = otherPart;
} else if (otherPart != part) { // Assume we should have the exact same object.