You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@yetus.apache.org by bu...@apache.org on 2015/09/23 04:23:50 UTC

[20/50] [abbrv] yetus git commit: HADOOP-12310. final memory report sometimes generates spurious errors (Kengo Seki via aw)

HADOOP-12310. final memory report sometimes generates spurious errors (Kengo Seki via aw)


Project: http://git-wip-us.apache.org/repos/asf/yetus/repo
Commit: http://git-wip-us.apache.org/repos/asf/yetus/commit/b5713747
Tree: http://git-wip-us.apache.org/repos/asf/yetus/tree/b5713747
Diff: http://git-wip-us.apache.org/repos/asf/yetus/diff/b5713747

Branch: refs/heads/master
Commit: b57137478884281022f6a8c0944611fc630f1b46
Parents: b44bde5
Author: Allen Wittenauer <aw...@apache.org>
Authored: Mon Aug 10 09:46:06 2015 -0700
Committer: Allen Wittenauer <aw...@apache.org>
Committed: Mon Aug 10 09:46:06 2015 -0700

----------------------------------------------------------------------
 dev-support/test-patch.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/yetus/blob/b5713747/dev-support/test-patch.sh
----------------------------------------------------------------------
diff --git a/dev-support/test-patch.sh b/dev-support/test-patch.sh
index 1faf99f..446d5cf 100755
--- a/dev-support/test-patch.sh
+++ b/dev-support/test-patch.sh
@@ -395,9 +395,10 @@ function finish_footer_table
 {
   local maxmem
 
+  # `sort | head` can cause a broken pipe error, but we can ignore it just like compute_gitdiff.
   # shellcheck disable=SC2016,SC2086
   maxmem=$(find "${PATCH_DIR}" -type f -exec ${AWK} 'match($0, /^\[INFO\] Final Memory: [0-9]+/)
-    { print substr($0, 22, RLENGTH-21) }' {} \; | sort -nr | head -n 1)
+    { print substr($0, 22, RLENGTH-21) }' {} \; | sort -nr 2>/dev/null | head -n 1)
 
   if [[ -n ${maxmem} ]]; then
     add_footer_table "Max memory used" "${maxmem}MB"