[incubator-pinot] branch master updated: [TE] aaronucsd/fix for the live alerts not showing bug (#3710)

[ap...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

apucher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 17c85c5  [TE] aaronucsd/fix for the live alerts not showing bug (#3710)
17c85c5 is described below

commit 17c85c5c39df1a93bf2510d53d78b4ffa0ddc40e
Author: Long Huynh <lo...@linkedin.com>
AuthorDate: Thu Jan 17 14:10:25 2019 -0800

    [TE] aaronucsd/fix for the live alerts not showing bug (#3710)
    
    prod hot fix
---
 thirdeye/thirdeye-frontend/app/utils/manage-alert-utils.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/thirdeye/thirdeye-frontend/app/utils/manage-alert-utils.js b/thirdeye/thirdeye-frontend/app/utils/manage-alert-utils.js
index 3171387..64f892a 100644
--- a/thirdeye/thirdeye-frontend/app/utils/manage-alert-utils.js
+++ b/thirdeye/thirdeye-frontend/app/utils/manage-alert-utils.js
@@ -52,7 +52,7 @@ export function powerSort(targetArray, targetProperty) {
       a = a[targetProperty];
       b = b[targetProperty];
     }
-    return a.toLowerCase().trim().localeCompare(b.toLowerCase().trim());
+    return (a || '').toLowerCase().trim().localeCompare((b || '').toLowerCase().trim());
   });
   return sortedArray;
 }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org