Mailing-List: contact cloudstack-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cloudstack-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of Murali.Reddy@citrix.com
 designates 203.166.19.134 as permitted sender)
From: Murali Reddy <Murali.Reddy@citrix.com>
To: Kelven Yang <kelven.yang@citrix.com>,
	"cloudstack-dev@incubator.apache.org" <cloudstack-dev@incubator.apache.org>
Subject: CLOUDSTACK-1568, CLOUDSTACK-1664: Fix ActionEvent interception and
 optimize @DB lookup with caching
Thread-Topic: CLOUDSTACK-1568, CLOUDSTACK-1664: Fix ActionEvent interception
 and optimize @DB lookup with caching
Thread-Index: AQHOJXW5s1KIMShAbU6ABqw+uuYzMw==
Date: Wed, 20 Mar 2013 14:17:58 +0000
Message-ID: <CD6FC0D2.37B5F%murali.reddy@citrix.com>
In-Reply-To: <20130319011420.140F73936D@tyr.zones.apache.org>
Accept-Language: en-US
Content-Language: en-US
user-agent: Microsoft-MacOutlook/14.2.5.121010
Content-Type: text/plain; charset="us-ascii"
Content-ID: <0CB2F6CB81E981479E44A63DAB571CAF@citrix.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0


Kelven, I see that you fixed action event issue in below commit. But I am
still running into some issues. While working on CLOUDSTACK-1673 I noticed
that in case of AccountManagerImpl, there are multiple methods with same
name. So in that case getAnnotation on the method is returning null even
though there is ActionEvent annotation. If I rename method then action
event annotation works fine. You can repro it by creating user (there are
two createUser methods in AccountManagerImpl). I looked at
ComponentMethodProxyCache to see if I can figure the problem but with no
luck. If you get a chance please try once.

On 19/03/13 6:44 AM, "kelveny@apache.org" <kelveny@apache.org> wrote:

>Updated Branches:
>  refs/heads/master d9b85e397 -> fb94b7221
>
>
>CLOUDSTACK-1568,CLOUDSTACK-1664: Fix ActionEvent interception and
>optimize @DB lookup with caching
>
>
>Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo
>Commit:=20
>http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/fb94b72
>2
>Tree:=20
>http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/fb94b722
>Diff:=20
>http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/fb94b722
>
>Branch: refs/heads/master
>Commit: fb94b72213bf96f2878b90260067f61629c6a956
>Parents: d9b85e3
>Author: Kelven Yang <kelveny@gmail.com>
>Authored: Mon Mar 18 18:05:09 2013 -0700
>Committer: Kelven Yang <kelveny@gmail.com>
>Committed: Mon Mar 18 18:07:52 2013 -0700
>
>----------------------------------------------------------------------
> .../com/cloud/event/ActionEventInterceptor.java    |   15 ++-
> .../ConsoleProxyHttpHandlerHelper.java             |   14 ++-
> .../utils/component/ComponentMethodProxyCache.java |   90 +++++++++++++++
> .../utils/component/SpringComponentScanUtils.java  |    1 -
> .../cloud/utils/db/TransactionContextBuilder.java  |   14 +--
> 5 files changed, 119 insertions(+), 15 deletions(-)
>----------------------------------------------------------------------
>