commit b711fbf42423c513206eec67474e73e6d354130c
author Varun Sharma <varunsha@partner.ciena.com> Sat Aug 06 04:25:07 2016 +0530
committer Varun Sharma <varunsha@partner.ciena.com> Mon Aug 08 13:21:29 2016 +0000
tree 1e0e5995cfd158641839822dbc3dad1b7f845445
parent 3fdb9dd1b55e7958963b5988f9e1237dcf0aff27

Cherry pick 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 fa98d66..385f399 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
@@ -151,8 +151,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