You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by an...@apache.org on 2014/03/20 17:25:43 UTC
svn commit: r1579678 - in /tomee/tomee/trunk/server: openejb-cxf-rs/
openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
Author: andygumbrecht
Date: Thu Mar 20 16:25:42 2014
New Revision: 1579678
URL: http://svn.apache.org/r1579678
Log:
https://issues.apache.org/jira/browse/OPENEJB-2082
Added:
tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
Modified:
tomee/tomee/trunk/server/openejb-cxf-rs/ (props changed)
tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
Propchange: tomee/tomee/trunk/server/openejb-cxf-rs/
------------------------------------------------------------------------------
--- svn:ignore (original)
+++ svn:ignore Thu Mar 20 16:25:42 2014
@@ -7,3 +7,4 @@
.settings
out
target
+.idea
Added: tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java?rev=1579678&view=auto
==============================================================================
--- tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java (added)
+++ tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java Thu Mar 20 16:25:42 2014
@@ -0,0 +1,37 @@
+/**
+ * Tomitribe Confidential
+ *
+ * Copyright(c) Tomitribe Corporation. 2014
+ *
+ * The source code for this program is not published or otherwise divested
+ * of its trade secrets, irrespective of what has been deposited with the
+ * U.S. Copyright Office.
+ *
+ * Author: agumbrecht@tomitribe.com
+ * Date: 20.03.14
+ * Time: 16:27
+ */
+package org.apache.openejb.server.rest;
+
+import org.apache.openejb.assembler.classic.AppInfo;
+import org.apache.openejb.observer.Event;
+
+@Event
+public class AfterServicesDeployed {
+ private final AppInfo app;
+ private final String s;
+
+ public AfterServicesDeployed(final AppInfo appInfo) {
+ app = appInfo;
+ this.s = "AfterServicesDeployed{appId=" + app.appId + "}";
+ }
+
+ public AppInfo getApp() {
+ return app;
+ }
+
+ @Override
+ public String toString() {
+ return s;
+ }
+}
Modified: tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java?rev=1579678&r1=1579677&r2=1579678&view=diff
==============================================================================
--- tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java (original)
+++ tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java Thu Mar 20 16:25:42 2014
@@ -619,6 +619,8 @@ public abstract class RESTService implem
}
}
}
+
+ SystemInstance.get().fireEvent(new AfterServicesDeployed(appInfo));
}
private void quickCheckIfOldDeploymentShouldBeUsedFromEjbConfig(final AppInfo appInfo) {
Re: svn commit: r1579678 - in /tomee/tomee/trunk/server: openejb-cxf-rs/ openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
Posted by David Blevins <da...@gmail.com>.
FYI, definitely the wrong license header on that file :)
-David
On Mar 20, 2014, at 9:35 AM, Romain Manni-Bucau <rm...@gmail.com> wrote:
> Hi
>
> maybe last event should be AfterServiceCall(service, appInfo) for all
> services or renamed to AfterRESTService
>
> Side note: please don't add a bunch of events or we should really
> rework our current handling since it is not efficient at all ATM
>
> Romain Manni-Bucau
> Twitter: @rmannibucau
> Blog: http://rmannibucau.wordpress.com/
> LinkedIn: http://fr.linkedin.com/in/rmannibucau
> Github: https://github.com/rmannibucau
>
>
>
>
> ---------- Forwarded message ----------
> From: <an...@apache.org>
> Date: 2014-03-20 17:25 GMT+01:00
> Subject: svn commit: r1579678 - in /tomee/tomee/trunk/server:
> openejb-cxf-rs/
> openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
> openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
> To: commits@tomee.apache.org
>
>
> Author: andygumbrecht
> Date: Thu Mar 20 16:25:42 2014
> New Revision: 1579678
>
> URL: http://svn.apache.org/r1579678
> Log:
> https://issues.apache.org/jira/browse/OPENEJB-2082
>
> Added:
> tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
> Modified:
> tomee/tomee/trunk/server/openejb-cxf-rs/ (props changed)
> tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
>
> Propchange: tomee/tomee/trunk/server/openejb-cxf-rs/
> ------------------------------------------------------------------------------
> --- svn:ignore (original)
> +++ svn:ignore Thu Mar 20 16:25:42 2014
> @@ -7,3 +7,4 @@
> .settings
> out
> target
> +.idea
>
> Added: tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
> URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java?rev=1579678&view=auto
> ==============================================================================
> --- tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
> (added)
> +++ tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
> Thu Mar 20 16:25:42 2014
> @@ -0,0 +1,37 @@
> +/**
> + * Tomitribe Confidential
> + *
> + * Copyright(c) Tomitribe Corporation. 2014
> + *
> + * The source code for this program is not published or otherwise divested
> + * of its trade secrets, irrespective of what has been deposited with the
> + * U.S. Copyright Office.
> + *
> + * Author: agumbrecht@tomitribe.com
> + * Date: 20.03.14
> + * Time: 16:27
> + */
> +package org.apache.openejb.server.rest;
> +
> +import org.apache.openejb.assembler.classic.AppInfo;
> +import org.apache.openejb.observer.Event;
> +
> +@Event
> +public class AfterServicesDeployed {
> + private final AppInfo app;
> + private final String s;
> +
> + public AfterServicesDeployed(final AppInfo appInfo) {
> + app = appInfo;
> + this.s = "AfterServicesDeployed{appId=" + app.appId + "}";
> + }
> +
> + public AppInfo getApp() {
> + return app;
> + }
> +
> + @Override
> + public String toString() {
> + return s;
> + }
> +}
>
> Modified: tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
> URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java?rev=1579678&r1=1579677&r2=1579678&view=diff
> ==============================================================================
> --- tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
> (original)
> +++ tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
> Thu Mar 20 16:25:42 2014
> @@ -619,6 +619,8 @@ public abstract class RESTService implem
> }
> }
> }
> +
> + SystemInstance.get().fireEvent(new AfterServicesDeployed(appInfo));
> }
>
> private void
> quickCheckIfOldDeploymentShouldBeUsedFromEjbConfig(final AppInfo
> appInfo) {
Fwd: svn commit: r1579678 - in /tomee/tomee/trunk/server:
openejb-cxf-rs/ openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
Posted by Romain Manni-Bucau <rm...@gmail.com>.
Hi
maybe last event should be AfterServiceCall(service, appInfo) for all
services or renamed to AfterRESTService
Side note: please don't add a bunch of events or we should really
rework our current handling since it is not efficient at all ATM
Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau
---------- Forwarded message ----------
From: <an...@apache.org>
Date: 2014-03-20 17:25 GMT+01:00
Subject: svn commit: r1579678 - in /tomee/tomee/trunk/server:
openejb-cxf-rs/
openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
To: commits@tomee.apache.org
Author: andygumbrecht
Date: Thu Mar 20 16:25:42 2014
New Revision: 1579678
URL: http://svn.apache.org/r1579678
Log:
https://issues.apache.org/jira/browse/OPENEJB-2082
Added:
tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
Modified:
tomee/tomee/trunk/server/openejb-cxf-rs/ (props changed)
tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
Propchange: tomee/tomee/trunk/server/openejb-cxf-rs/
------------------------------------------------------------------------------
--- svn:ignore (original)
+++ svn:ignore Thu Mar 20 16:25:42 2014
@@ -7,3 +7,4 @@
.settings
out
target
+.idea
Added: tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java?rev=1579678&view=auto
==============================================================================
--- tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
(added)
+++ tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/AfterServicesDeployed.java
Thu Mar 20 16:25:42 2014
@@ -0,0 +1,37 @@
+/**
+ * Tomitribe Confidential
+ *
+ * Copyright(c) Tomitribe Corporation. 2014
+ *
+ * The source code for this program is not published or otherwise divested
+ * of its trade secrets, irrespective of what has been deposited with the
+ * U.S. Copyright Office.
+ *
+ * Author: agumbrecht@tomitribe.com
+ * Date: 20.03.14
+ * Time: 16:27
+ */
+package org.apache.openejb.server.rest;
+
+import org.apache.openejb.assembler.classic.AppInfo;
+import org.apache.openejb.observer.Event;
+
+@Event
+public class AfterServicesDeployed {
+ private final AppInfo app;
+ private final String s;
+
+ public AfterServicesDeployed(final AppInfo appInfo) {
+ app = appInfo;
+ this.s = "AfterServicesDeployed{appId=" + app.appId + "}";
+ }
+
+ public AppInfo getApp() {
+ return app;
+ }
+
+ @Override
+ public String toString() {
+ return s;
+ }
+}
Modified: tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java?rev=1579678&r1=1579677&r2=1579678&view=diff
==============================================================================
--- tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
(original)
+++ tomee/tomee/trunk/server/openejb-rest/src/main/java/org/apache/openejb/server/rest/RESTService.java
Thu Mar 20 16:25:42 2014
@@ -619,6 +619,8 @@ public abstract class RESTService implem
}
}
}
+
+ SystemInstance.get().fireEvent(new AfterServicesDeployed(appInfo));
}
private void
quickCheckIfOldDeploymentShouldBeUsedFromEjbConfig(final AppInfo
appInfo) {