Repository: reef
Updated Branches:
  refs/heads/master 8ef0c4e2a -> 8e628faeb


[REEF-1646] Adding ArgumentNullException in config initialization

JIRA: [REEF-1646](https://issues.apache.org/jira/browse/REEF-1646)
This closes #1173


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

Branch: refs/heads/master
Commit: 8e628faeb2c1d2a1dade06ad39629f8269e70d81
Parents: 8ef0c4e
Author: andrey-me <[email protected]>
Authored: Mon Oct 31 11:36:04 2016 -0700
Committer: Julia Wang <[email protected]>
Committed: Tue Nov 1 11:03:40 2016 -0700

----------------------------------------------------------------------
 .../Implementations/Configuration/ConfigurationBuilderImpl.cs   | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/reef/blob/8e628fae/lang/cs/Org.Apache.REEF.Tang/Implementations/Configuration/ConfigurationBuilderImpl.cs
----------------------------------------------------------------------
diff --git 
a/lang/cs/Org.Apache.REEF.Tang/Implementations/Configuration/ConfigurationBuilderImpl.cs
 
b/lang/cs/Org.Apache.REEF.Tang/Implementations/Configuration/ConfigurationBuilderImpl.cs
index fe3a099..4eab9a1 100644
--- 
a/lang/cs/Org.Apache.REEF.Tang/Implementations/Configuration/ConfigurationBuilderImpl.cs
+++ 
b/lang/cs/Org.Apache.REEF.Tang/Implementations/Configuration/ConfigurationBuilderImpl.cs
@@ -59,6 +59,11 @@ namespace Org.Apache.REEF.Tang.Implementations.Configuration
             this.ClassHierarchy = 
TangFactory.GetTang().GetDefaultClassHierarchy(assemblies, parsers);
             foreach (IConfiguration tc in confs) 
             {
+                if (tc == null)
+                {
+                    throw new ArgumentNullException("One of specified 
configurations is null");
+                } 
+                
                 AddConfiguration((ConfigurationImpl)tc);
             }
         }

Reply via email to