----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12134/#review25742 -----------------------------------------------------------
By now this patch is kind of outdated and needs an update, just wondering what developers think about the idea. Is the txn attribute being planned to use in the future? - Laszlo Hornyak On June 27, 2013, 4:34 p.m., Laszlo Hornyak wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12134/ > ----------------------------------------------------------- > > (Updated June 27, 2013, 4:34 p.m.) > > > Review request for cloudstack. > > > 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/api/src/org/apache/cloudstack/storage/datastore/db/PrimaryDataStoreDaoImpl.java > 8f7826f > > engine/orchestration/src/org/apache/cloudstack/engine/datacenter/entity/api/db/dao/EngineHostDaoImpl.java > 3c34023 > 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 e97f2c6 > 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 810b973 > 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 45a8068 > engine/schema/src/com/cloud/network/dao/NetworkDaoImpl.java c55cf28 > 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 fac35a9 > engine/schema/src/com/cloud/network/vpc/dao/VpcDaoImpl.java 6560b90 > engine/schema/src/com/cloud/network/vpc/dao/VpcGatewayDaoImpl.java a8cb2b3 > 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 f82b511 > 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 > > 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 2a3b469 > 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 > utils/src/com/cloud/utils/db/DB.java f83a7ea > utils/src/com/cloud/utils/db/GenericDaoBase.java f0fc700 > > Diff: https://reviews.apache.org/r/12134/diff/ > > > Testing > ------- > > > Thanks, > > Laszlo Hornyak > >