You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@storm.apache.org by bo...@apache.org on 2016/01/11 16:34:54 UTC

[6/8] storm git commit: per cr, answering mutable empty list instead of immutable in Utils.getStrings()

per cr, answering mutable empty list instead of immutable in Utils.getStrings()


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

Branch: refs/heads/master
Commit: ab769cba028b5db4eb00540da6254bb67be216d6
Parents: c9cc96a
Author: Aaron Dixon <at...@gmail.com>
Authored: Tue Jan 5 16:59:45 2016 -0600
Committer: Aaron Dixon <at...@gmail.com>
Committed: Tue Jan 5 16:59:45 2016 -0600

----------------------------------------------------------------------
 storm-core/src/jvm/backtype/storm/utils/Utils.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/storm/blob/ab769cba/storm-core/src/jvm/backtype/storm/utils/Utils.java
----------------------------------------------------------------------
diff --git a/storm-core/src/jvm/backtype/storm/utils/Utils.java b/storm-core/src/jvm/backtype/storm/utils/Utils.java
index eaa6e85..890451e 100644
--- a/storm-core/src/jvm/backtype/storm/utils/Utils.java
+++ b/storm-core/src/jvm/backtype/storm/utils/Utils.java
@@ -634,7 +634,7 @@ public class Utils {
 
     public static List<String> getStrings(final Object o) {
         if (o == null) {
-            return Collections.emptyList();
+            return new ArrayList<String>();
         } else if (o instanceof String) {
             return new ArrayList<String>() {{ add((String) o); }};
         } else if (o instanceof Collection) {