From dceaa7cceff5c9c7db1b3df2ba6bad95e0f86edd Mon Sep 17 00:00:00 2001 From: Alena Prokharchyk Date: Tue, 22 Apr 2014 18:06:33 -0700 Subject: [PATCH] Revert "Return isolation methods as a part of listPhysicalNetworks call" This reverts commit eb877a093c7e7f0f28029aedf6bd3061867a01b3. --- .../PhysicalNetworkIsolationMethodDao.java | 39 ------------------- ...PhysicalNetworkIsolationMethodDaoImpl.java | 7 ++-- server/src/com/cloud/api/ApiDBUtils.java | 10 ----- .../src/com/cloud/api/ApiResponseHelper.java | 2 +- 4 files changed, 4 insertions(+), 54 deletions(-) delete mode 100644 engine/schema/src/com/cloud/network/dao/PhysicalNetworkIsolationMethodDao.java diff --git a/engine/schema/src/com/cloud/network/dao/PhysicalNetworkIsolationMethodDao.java b/engine/schema/src/com/cloud/network/dao/PhysicalNetworkIsolationMethodDao.java deleted file mode 100644 index a4e1bd588c8..00000000000 --- a/engine/schema/src/com/cloud/network/dao/PhysicalNetworkIsolationMethodDao.java +++ /dev/null @@ -1,39 +0,0 @@ -// Copyright 2012 Citrix Systems, Inc. Licensed under the -// Apache License, Version 2.0 (the "License"); you may not use this -// file except in compliance with the License. Citrix Systems, Inc. -// reserves all rights not expressly granted by the License. -// You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0 -// Unless required by applicable law or agreed to in writing, software -// distributed under the License is distributed on an "AS IS" BASIS, -// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -// See the License for the specific language governing permissions and -// limitations under the License. -// -// Automatically generated by addcopyright.py at 04/03/2012 -package com.cloud.network.dao; - -import java.util.List; - -import com.cloud.utils.db.GenericDao; - -public interface PhysicalNetworkIsolationMethodDao extends GenericDao{ - - /** - * @param physicalNetworkId - * @return - */ - List getAllIsolationMethod(long physicalNetworkId); - - /** - * @param physicalNetworkId - * @return - */ - String getIsolationMethod(long physicalNetworkId); - - /** - * @param physicalNetworkId - * @return - */ - int clearIsolationMethods(long physicalNetworkId); - -} diff --git a/engine/schema/src/com/cloud/network/dao/PhysicalNetworkIsolationMethodDaoImpl.java b/engine/schema/src/com/cloud/network/dao/PhysicalNetworkIsolationMethodDaoImpl.java index a22548a239e..11b4399fcca 100644 --- a/engine/schema/src/com/cloud/network/dao/PhysicalNetworkIsolationMethodDaoImpl.java +++ b/engine/schema/src/com/cloud/network/dao/PhysicalNetworkIsolationMethodDaoImpl.java @@ -20,6 +20,7 @@ import java.util.List; import org.springframework.stereotype.Component; +import com.cloud.utils.db.GenericDao; import com.cloud.utils.db.GenericDaoBase; import com.cloud.utils.db.GenericSearchBuilder; import com.cloud.utils.db.SearchBuilder; @@ -27,7 +28,8 @@ import com.cloud.utils.db.SearchCriteria; import com.cloud.utils.db.SearchCriteria.Op; @Component -public class PhysicalNetworkIsolationMethodDaoImpl extends GenericDaoBase implements PhysicalNetworkIsolationMethodDao { +public class PhysicalNetworkIsolationMethodDaoImpl extends GenericDaoBase implements + GenericDao { private final GenericSearchBuilder IsolationMethodSearch; private final SearchBuilder AllFieldsSearch; @@ -45,7 +47,6 @@ public class PhysicalNetworkIsolationMethodDaoImpl extends GenericDaoBase getAllIsolationMethod(long physicalNetworkId) { SearchCriteria sc = IsolationMethodSearch.create(); sc.setParameters("physicalNetworkId", physicalNetworkId); @@ -53,7 +54,6 @@ public class PhysicalNetworkIsolationMethodDaoImpl extends GenericDaoBase sc = IsolationMethodSearch.create(); sc.setParameters("physicalNetworkId", physicalNetworkId); @@ -61,7 +61,6 @@ public class PhysicalNetworkIsolationMethodDaoImpl extends GenericDaoBase sc = AllFieldsSearch.create(); sc.setParameters("physicalNetworkId", physicalNetworkId); diff --git a/server/src/com/cloud/api/ApiDBUtils.java b/server/src/com/cloud/api/ApiDBUtils.java index ca25d66977c..2da107f24f6 100755 --- a/server/src/com/cloud/api/ApiDBUtils.java +++ b/server/src/com/cloud/api/ApiDBUtils.java @@ -181,7 +181,6 @@ import com.cloud.network.dao.NetworkRuleConfigDao; import com.cloud.network.dao.NetworkRuleConfigVO; import com.cloud.network.dao.NetworkVO; import com.cloud.network.dao.PhysicalNetworkDao; -import com.cloud.network.dao.PhysicalNetworkIsolationMethodDao; import com.cloud.network.dao.PhysicalNetworkServiceProviderDao; import com.cloud.network.dao.PhysicalNetworkServiceProviderVO; import com.cloud.network.dao.PhysicalNetworkTrafficTypeDao; @@ -410,7 +409,6 @@ public class ApiDBUtils { static ResourceMetaDataService s_resourceDetailsService; static HostGpuGroupsDao s_hostGpuGroupsDao; static VGPUTypesDao s_vgpuTypesDao; - static PhysicalNetworkIsolationMethodDao s_pNtwkIsolatinoMethodsDao; @Inject private ManagementServer ms; @@ -633,8 +631,6 @@ public class ApiDBUtils { private HostGpuGroupsDao hostGpuGroupsDao; @Inject private VGPUTypesDao vgpuTypesDao; - @Inject - private PhysicalNetworkIsolationMethodDao pNtwkIsolatinoMethodsDao; @PostConstruct void init() { @@ -749,7 +745,6 @@ public class ApiDBUtils { s_resourceDetailsService = resourceDetailsService; s_hostGpuGroupsDao = hostGpuGroupsDao; s_vgpuTypesDao = vgpuTypesDao; - s_pNtwkIsolatinoMethodsDao = pNtwkIsolatinoMethodsDao; } // /////////////////////////////////////////////////////////// @@ -1840,9 +1835,4 @@ public class ApiDBUtils { public static List listResourceTagViewByResourceUUID(String resourceUUID, ResourceObjectType resourceType) { return s_tagJoinDao.listBy(resourceUUID, resourceType); } - - public static List getIsolationMethodsForPhysicalNtwk(long pNtwkId) { - return s_pNtwkIsolatinoMethodsDao.getAllIsolationMethod(pNtwkId); - } - } diff --git a/server/src/com/cloud/api/ApiResponseHelper.java b/server/src/com/cloud/api/ApiResponseHelper.java index 74ccf29b7cc..1155d1ffc17 100755 --- a/server/src/com/cloud/api/ApiResponseHelper.java +++ b/server/src/com/cloud/api/ApiResponseHelper.java @@ -2465,7 +2465,7 @@ public class ApiResponseHelper implements ResponseGenerator { if (result.getBroadcastDomainRange() != null) { response.setBroadcastDomainRange(result.getBroadcastDomainRange().toString()); } - response.setIsolationMethods(ApiDBUtils.getIsolationMethodsForPhysicalNtwk(result.getId())); + response.setIsolationMethods(result.getIsolationMethods()); response.setTags(result.getTags()); if (result.getState() != null) { response.setState(result.getState().toString());