----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12134/#review25889 -----------------------------------------------------------
Ship it! Ship It: Not committed, Pending review from alexh - Darren Shepherd On Sept. 4, 2013, 4:55 p.m., Laszlo Hornyak wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12134/ > ----------------------------------------------------------- > > (Updated Sept. 4, 2013, 4:55 p.m.) > > > Review request for cloudstack, Alex Huang and Darren Shepherd. > > > Repository: cloudstack-git > > > Description > ------- > > The txn attribute was used often with the DB annotation, but it does not have > an effect on the behavior of the code since it is never read. > - All of this references to txn removed > - Attribute removed from annotation > - Documentation references removed > > > Diffs > ----- > > > engine/orchestration/src/org/apache/cloudstack/engine/datacenter/entity/api/db/dao/EngineHostDaoImpl.java > 5ba65cc > engine/schema/src/com/cloud/certificate/dao/CertificateDaoImpl.java f071cea > engine/schema/src/com/cloud/cluster/agentlb/dao/HostTransferMapDaoImpl.java > cff4cfc > engine/schema/src/com/cloud/dc/dao/ClusterVSMMapDaoImpl.java b12fa9d > engine/schema/src/com/cloud/dc/dao/DataCenterIpAddressDaoImpl.java 353402d > engine/schema/src/com/cloud/dc/dao/DataCenterLinkLocalIpAddressDaoImpl.java > b52e373 > engine/schema/src/com/cloud/dc/dao/DataCenterVnetDaoImpl.java d3a2409 > engine/schema/src/com/cloud/dc/dao/StorageNetworkIpAddressDaoImpl.java > 782ee0d > engine/schema/src/com/cloud/dc/dao/StorageNetworkIpRangeDaoImpl.java > d732e6f > engine/schema/src/com/cloud/host/dao/HostDaoImpl.java 41ee7a7 > engine/schema/src/com/cloud/network/dao/AccountGuestVlanMapDaoImpl.java > e7a7b34 > engine/schema/src/com/cloud/network/dao/ExternalFirewallDeviceDaoImpl.java > 01f8861 > > engine/schema/src/com/cloud/network/dao/ExternalLoadBalancerDeviceDaoImpl.java > ea6437d > engine/schema/src/com/cloud/network/dao/FirewallRulesDaoImpl.java 41f911c > engine/schema/src/com/cloud/network/dao/NetworkDaoImpl.java 0f83815 > engine/schema/src/com/cloud/network/dao/NetworkDomainDaoImpl.java bbb9203 > engine/schema/src/com/cloud/network/dao/NetworkExternalFirewallDaoImpl.java > b176760 > > engine/schema/src/com/cloud/network/dao/NetworkExternalLoadBalancerDaoImpl.java > c29c164 > engine/schema/src/com/cloud/network/dao/NetworkServiceMapDaoImpl.java > 3cdd738 > engine/schema/src/com/cloud/network/dao/PhysicalNetworkDaoImpl.java 1e26a51 > > engine/schema/src/com/cloud/network/dao/PhysicalNetworkServiceProviderDaoImpl.java > 16a23dd > > engine/schema/src/com/cloud/network/dao/PhysicalNetworkTrafficTypeDaoImpl.java > 7e47239 > engine/schema/src/com/cloud/network/dao/PortProfileDaoImpl.java 61fe52a > engine/schema/src/com/cloud/network/dao/VirtualRouterProviderDaoImpl.java > dba835f > engine/schema/src/com/cloud/network/vpc/dao/PrivateIpDaoImpl.java fe435c0 > engine/schema/src/com/cloud/network/vpc/dao/StaticRouteDaoImpl.java 518237d > engine/schema/src/com/cloud/network/vpc/dao/VpcDaoImpl.java 6560b90 > engine/schema/src/com/cloud/network/vpc/dao/VpcGatewayDaoImpl.java 13c37c4 > engine/schema/src/com/cloud/network/vpc/dao/VpcOfferingDaoImpl.java 2cda547 > > engine/schema/src/com/cloud/network/vpc/dao/VpcOfferingServiceMapDaoImpl.java > 4b5f1b9 > engine/schema/src/com/cloud/network/vpc/dao/VpcServiceMapDaoImpl.java > a992181 > engine/schema/src/com/cloud/offerings/dao/NetworkOfferingDaoImpl.java > ef8237a > > engine/schema/src/com/cloud/offerings/dao/NetworkOfferingServiceMapDaoImpl.java > 7282443 > engine/schema/src/com/cloud/service/dao/ServiceOfferingDaoImpl.java 14b2abf > engine/schema/src/com/cloud/storage/dao/StoragePoolWorkDaoImpl.java 052bae4 > engine/schema/src/com/cloud/storage/dao/VolumeDaoImpl.java bf28410 > engine/schema/src/com/cloud/upgrade/dao/VersionDaoImpl.java 7c5c9cc > engine/schema/src/com/cloud/vm/dao/UserVmCloneSettingDaoImpl.java 174f283 > > engine/schema/src/org/apache/cloudstack/region/gslb/GlobalLoadBalancerLbRuleMapDaoImpl.java > 2a6e72b > > engine/schema/src/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java > b39f844 > framework/db/src/com/cloud/utils/db/DB.java f83a7ea > framework/db/src/com/cloud/utils/db/GenericDaoBase.java 94e5374 > > plugins/hypervisors/baremetal/src/com/cloud/baremetal/database/BaremetalDhcpDaoImpl.java > 8123ee0 > > plugins/hypervisors/baremetal/src/com/cloud/baremetal/database/BaremetalPxeDaoImpl.java > acd7f13 > plugins/hypervisors/ucs/src/com/cloud/ucs/database/UcsBladeDaoImpl.java > 5cb9b02 > plugins/hypervisors/ucs/src/com/cloud/ucs/database/UcsManagerDaoImpl.java > 93c0885 > > plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/dao/LegacyZoneDaoImpl.java > 7d2d128 > > plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/dao/VmwareDatacenterDaoImpl.java > 9f5796a > > plugins/hypervisors/vmware/src/com/cloud/network/dao/CiscoNexusVSMDeviceDaoImpl.java > cc25573 > > plugins/network-elements/netscaler/src/com/cloud/network/dao/NetScalerPodDaoImpl.java > 30dd06d > server/src/com/cloud/dc/dao/DedicatedResourceDaoImpl.java 266e65f > server/src/com/cloud/network/vpc/dao/NetworkACLDaoImpl.java fd3308d > server/src/com/cloud/network/vpc/dao/NetworkACLItemDaoImpl.java 8162ce8 > server/test/com/cloud/vpc/dao/MockNetworkDaoImpl.java ec1a017 > server/test/com/cloud/vpc/dao/MockNetworkOfferingDaoImpl.java a8208dd > server/test/com/cloud/vpc/dao/MockNetworkOfferingServiceMapDaoImpl.java > d1e8354 > server/test/com/cloud/vpc/dao/MockNetworkServiceMapDaoImpl.java 103f04e > server/test/com/cloud/vpc/dao/MockVpcDaoImpl.java 562d67d > server/test/com/cloud/vpc/dao/MockVpcOfferingDaoImpl.java 329931e > server/test/com/cloud/vpc/dao/MockVpcOfferingServiceMapDaoImpl.java 3357686 > > Diff: https://reviews.apache.org/r/12134/diff/ > > > Testing > ------- > > > Thanks, > > Laszlo Hornyak > >
