Return-Path: X-Original-To: apmail-cloudstack-issues-archive@www.apache.org Delivered-To: apmail-cloudstack-issues-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id CEC3A104AD for ; Wed, 16 Oct 2013 14:07:54 +0000 (UTC) Received: (qmail 97848 invoked by uid 500); 16 Oct 2013 14:07:51 -0000 Delivered-To: apmail-cloudstack-issues-archive@cloudstack.apache.org Received: (qmail 97796 invoked by uid 500); 16 Oct 2013 14:07:51 -0000 Mailing-List: contact issues-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list issues@cloudstack.apache.org Received: (qmail 97712 invoked by uid 500); 16 Oct 2013 14:07:50 -0000 Delivered-To: apmail-incubator-cloudstack-issues@incubator.apache.org Received: (qmail 97572 invoked by uid 99); 16 Oct 2013 14:07:47 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 16 Oct 2013 14:07:47 +0000 Date: Wed, 16 Oct 2013 14:07:47 +0000 (UTC) From: "ASF subversion and git services (JIRA)" To: cloudstack-issues@incubator.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (CLOUDSTACK-4095) Region id within the Database Transaction code... MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/CLOUDSTACK-4095?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13796818#comment-13796818 ] ASF subversion and git services commented on CLOUDSTACK-4095: ------------------------------------------------------------- Commit e88377a5e50e6dcf296049a8c12517aeb6b2cac8 in branch refs/heads/4.2 from [~kishan] [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=e88377a ] CLOUDSTACK-4095 : Remove region_id from Transaction. Read from db.properties whenever required > Region id within the Database Transaction code... > ------------------------------------------------- > > Key: CLOUDSTACK-4095 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-4095 > Project: CloudStack > Issue Type: Bug > Security Level: Public(Anyone can view this level - this is the default.) > Components: Management Server > Affects Versions: 4.2.0 > Reporter: Alex Huang > Assignee: Kishan Kavala > Priority: Critical > Fix For: 4.2.1 > > > Why should the region id be in Transaction Id and then propagated through GenericDao. This is should not happen at all. We need to fix this. GenericDao is a generic framework. It should not understand regions at all. Even if you're using the db.properties to expose region id, you can read it yourself in your code, rather than pushing it into Transaction. -- This message was sent by Atlassian JIRA (v6.1#6144)