commit ca8b51e7f08d497fb947ff545be187b4f4a14fef
author Varun Sharma <varunsha@partner.ciena.com> Fri Jun 17 17:11:02 2016 +0530
committer Varun Sharma <varunsha@partner.ciena.com> Sat Jun 18 16:26:27 2016 +0530
tree 2c2560e98887e7bbaef5f9f263b9635cffc1773d
parent 0cf9e346d693db75136e1069afe348d2272cef68

ONOS-4721 fix

Change-Id: Idb0ba949293451488b350c7cdcfc50094a3c9abc

core/common/src/main/java/org/onosproject/codec/impl/GroupCodec.java

diff --git a/core/common/src/main/java/org/onosproject/codec/impl/GroupCodec.java b/core/common/src/main/java/org/onosproject/codec/impl/GroupCodec.java
index f6e3e9d..0afd71c 100644
--- a/core/common/src/main/java/org/onosproject/codec/impl/GroupCodec.java
+++ b/core/common/src/main/java/org/onosproject/codec/impl/GroupCodec.java
@@ -146,8 +146,7 @@
                 groupType = Group.Type.FAILOVER;
                 break;
             default:
-                log.warn("The requested type {} is not defined for group.", type);
-                return null;
+                nullIsIllegal(groupType, "The requested group type " + type + " is not valid");
         }
 
         // parse group buckets