Feedback on Atmosphere and GWT

4 messages

Feedback on Atmosphere and GWT

Reply Threaded

Salut,

Starting playing with GWT and Comet, I tried to make running the sample app of atmosphere "gwt-auction" with the last download of GWT : 1.7.1

Here is my feedback about trying to make it run.

Rework on the sample :

1/ the file atmosphere.xml is missing. Needs to be added to META-INF

2/ in the class StreamingServlet, the line :

AtmosphereResourceEvent e = (AtmosphereResourceEvent) request

.getAttribute("org.atmosphere.cpr.AtmosphereResourceEvent");

returns a null Object.

Instead you must use :

AtmosphereResource vAtmoResource = (AtmosphereResource) request

.getAttribute("org.atmosphere.cpr.AtmosphereResource");

and adapt the code of the following lines.

3/ When using the attribute rename-to (for simplied paths) on the module element of the App.gwt.xml file, you need to rework paths in :

- the App.html file

- web.xml file

- atmosphere file

in order to add the module path. Example :

/gwt-auction/streamingService instead of /streamingService

Google AppEngine :

1/ First problem : sessions must be activated in the appengine-web.xml

<sessions-enabled>true</sessions-enabled>

2/ It seems that Google AppEngine can't be used because of restrictions on a java Security constraint : modifyThreadGroup.

You'll get this error on jetty (running automatically with GAE restrictions, really fine !) :

java.security.AccessControlException: access denied (java.lang.RuntimePermission modifyThreadGroup)

on the method sendMessage of the StreamingServiceImpl object.

I had to deactivate GAE usage on my Eclipse Project to pass through.

Now my sample works very well on my dev box ! ;-)

But unfortunately will not on GAE :-(

-- Thanks Jeanfrancois for this promising framework.

-- Sorry for the typo in my precedent subject

Jeanfrancois Arcand

Re: Feedback on Atmosphere and GWT

Reply Threaded

Salut,

Julien Lamandé wrote:
> Salut,
>
> Starting playing with GWT and Comet, I tried to make running the sample
> app of atmosphere "gwt-auction" with the last download of GWT : 1.7.1
> Here is my feedback about trying to make it run.
>
> Rework on the sample :
> 1/ the file atmosphere.xml is missing. Needs to be added to META-INF

Grrr my mistake. Just added it:

Committed revision 611.

>
> 2/ in the class StreamingServlet, the line :
> AtmosphereResourceEvent e = (AtmosphereResourceEvent) request
> .getAttribute("org.atmosphere.cpr.AtmosphereResourceEvent");
> returns a null Object.
> Instead you must use :
> AtmosphereResource vAtmoResource = (AtmosphereResource) request
> .getAttribute("org.atmosphere.cpr.AtmosphereResource");
>
> and adapt the code of the following lines.
>

Nice catch. I've used IntelJ for refactoring and guess what it missed
that one. And worse, I forgot to test that simple :-(. Commited.

> 3/ When using the attribute rename-to (for simplied paths) on the module
> element of the App.gwt.xml file, you need to rework paths in :
> - the App.html file
> - web.xml file
> - atmosphere file
> in order to add the module path. Example :
> /gwt-auction/streamingService instead of /streamingService

OK I'm sure sure about that one? Can you attach your file?

>
> Google AppEngine :
>
> 1/ First problem : sessions must be activated in the appengine-web.xml
> <sessions-enabled>true</sessions-enabled>

OK That one I will keep it in mine as this week I will start working on GAE.

>
> 2/ It seems that Google AppEngine can't be used because
> of restrictions on a java Security constraint : modifyThreadGroup.
> You'll get this error on jetty (running automatically with GAE
> restrictions, really fine !) :
> /java/./security/./AccessControlException/: /access
> denied/ (/java/./lang/./RuntimePermission modifyThreadGroup/)
> on the method sendMessage of the StreamingServiceImpl object.

Can you paste the entire stack trace?

>
> I had to deactivate GAE usage on my Eclipse Project to pass through.
>
>
> Now my sample works very well on my dev box ! ;-)
> But unfortunately will not on GAE :-(

I think I need to write a special CometSupport SPI for GAE:

* http://is.gd/4L7l8

so when the thread times out after 30 seconds, the Resume behavior happens.

If you have cycle this week to help, let me know as it will help being
more than one people testing this.

Merci!

-- Jeanfrancois

>
> -- Thanks Jeanfrancois for this promising framework.
> -- Sorry for the typo in my precedent subject

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Jeanfrancois Arcand

Re: Feedback on Atmosphere and GWT

Reply Threaded

Salut,

I've added support for GAE:

* http://is.gd/4LAZX

Try it and let me know. At least the Chat samples works fine.

A+

-- Jeanfrancois

Jeanfrancois Arcand wrote:

> Salut,
>
> Julien Lamandé wrote:
>> Salut,
>>
>> Starting playing with GWT and Comet, I tried to make running the
>> sample app of atmosphere "gwt-auction" with the last download of GWT :
>> 1.7.1
>> Here is my feedback about trying to make it run.
>>
>> Rework on the sample :
>> 1/ the file atmosphere.xml is missing. Needs to be added to META-INF
>
> Grrr my mistake. Just added it:
>
> Committed revision 611.
>
>
>>
>> 2/ in the class StreamingServlet, the line :
>> AtmosphereResourceEvent e = (AtmosphereResourceEvent) request
>> .getAttribute("org.atmosphere.cpr.AtmosphereResourceEvent");
>> returns a null Object.
>> Instead you must use :
>> AtmosphereResource vAtmoResource = (AtmosphereResource) request
>> .getAttribute("org.atmosphere.cpr.AtmosphereResource");
>>
>> and adapt the code of the following lines.
>>
>
> Nice catch. I've used IntelJ for refactoring and guess what it missed
> that one. And worse, I forgot to test that simple :-(. Commited.
>
>
>> 3/ When using the attribute rename-to (for simplied paths) on the
>> module element of the App.gwt.xml file, you need to rework paths in :
>> - the App.html file
>> - web.xml file
>> - atmosphere file
>> in order to add the module path. Example :
>> /gwt-auction/streamingService instead of /streamingService
>
> OK I'm sure sure about that one? Can you attach your file?
>
>>
>> Google AppEngine :
>>
>> 1/ First problem : sessions must be activated in the appengine-web.xml
>> <sessions-enabled>true</sessions-enabled>
>
> OK That one I will keep it in mine as this week I will start working on
> GAE.
>
>>
>> 2/ It seems that Google AppEngine can't be used because of
>> restrictions on a java Security constraint : modifyThreadGroup.
>> You'll get this error on jetty (running automatically with GAE
>> restrictions, really fine !) :
>> /java/./security/./AccessControlException/: /access denied/
>> (/java/./lang/./RuntimePermission modifyThreadGroup/)
>> on the method sendMessage of the StreamingServiceImpl object.
>
> Can you paste the entire stack trace?
>
>
>>
>> I had to deactivate GAE usage on my Eclipse Project to pass through.
>>
>>
>> Now my sample works very well on my dev box ! ;-)
>> But unfortunately will not on GAE :-(
>
> I think I need to write a special CometSupport SPI for GAE:
>
> * http://is.gd/4L7l8
>
> so when the thread times out after 30 seconds, the Resume behavior happens.
>
> If you have cycle this week to help, let me know as it will help being
> more than one people testing this.
>
> Merci!
>
> -- Jeanfrancois
>
>
>>
>> -- Thanks Jeanfrancois for this promising framework.
>> -- Sorry for the typo in my precedent subject
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Jeanfrancois Arcand

Re: Feedback on Atmosphere and GWT

Reply Threaded

In reply to this post by Jeanfrancois Arcand

Salut,

Jeanfrancois Arcand wrote:

OK investigating GAE yesterday I understood your question. Only if you
deploy to GAE the above change is required as the application gets
deployed at / instead of /atmosphere-gwt-auction

A+

- jeanfrancois

>
> OK I'm sure sure about that one? Can you attach your file?
>
>>
>> Google AppEngine :
>>
>> 1/ First problem : sessions must be activated in the appengine-web.xml
>> <sessions-enabled>true</sessions-enabled>
>
> OK That one I will keep it in mine as this week I will start working on
> GAE.
>
>>
>> 2/ It seems that Google AppEngine can't be used because of
>> restrictions on a java Security constraint : modifyThreadGroup.
>> You'll get this error on jetty (running automatically with GAE
>> restrictions, really fine !) :
>> /java/./security/./AccessControlException/: /access denied/
>> (/java/./lang/./RuntimePermission modifyThreadGroup/)
>> on the method sendMessage of the StreamingServiceImpl object.
>
> Can you paste the entire stack trace?
>
>
>>
>> I had to deactivate GAE usage on my Eclipse Project to pass through.
>>
>>
>> Now my sample works very well on my dev box ! ;-)
>> But unfortunately will not on GAE :-(
>
> I think I need to write a special CometSupport SPI for GAE:
>
> * http://is.gd/4L7l8
>
> so when the thread times out after 30 seconds, the Resume behavior happens.
>
> If you have cycle this week to help, let me know as it will help being
> more than one people testing this.
>
> Merci!
>
> -- Jeanfrancois
>
>
>>
>> -- Thanks Jeanfrancois for this promising framework.
>> -- Sorry for the typo in my precedent subject
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]