cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Darren Shepherd" <darren.s.sheph...@gmail.com>
Subject Re: Review Request 12134: removed txn attribute from DB annotation
Date Wed, 04 Sep 2013 17:13:31 GMT

-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message