commit 097ab9cab6fbf010fe72c78d5770f7a13d689436
author Julien Desprez <jdesprez@google.com> Fri Apr 17 17:03:16 2020 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Fri Apr 17 17:03:16 2020 +0000
tree cf856daeb3b47a4a65393e005728769ae522d018
parent a4105b9c34bdfb6f041b46f5143ef081dfcb4a44
parent c6707f851e260de30ad9855f68fe087c9ffe256e

Merge "Ensure the parser fallback to string if cannot add"

src/com/android/tradefed/util/SubprocessTestResultsParser.java

diff --git a/src/com/android/tradefed/util/SubprocessTestResultsParser.java b/src/com/android/tradefed/util/SubprocessTestResultsParser.java
index db9f774..bc9d88f 100644
--- a/src/com/android/tradefed/util/SubprocessTestResultsParser.java
+++ b/src/com/android/tradefed/util/SubprocessTestResultsParser.java
@@ -576,8 +576,9 @@
                         try {
                             InvocationMetricLogger.addInvocationMetrics(key, Long.parseLong(val));
                         } catch (NumberFormatException e) {
-                            CLog.e("Key %s should have a number value, instead was: %s", key, val);
-                            CLog.e(e);
+                            CLog.d("Key %s doesn't have a number value, was: %s.", key, val);
+                            // If it's not a number then, let the string concatenate
+                            InvocationMetricLogger.addInvocationMetrics(key, val);
                         }
                     } else {
                         InvocationMetricLogger.addInvocationMetrics(key, val);