portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Software Engineer" <rsamb...@crosslink.net>
Subject Re: svn commit: r566920 - /portals/jetspeed-2/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/invoker/ServletPortletInvoker.java
Date Sat, 18 Aug 2007 08:12:58 GMT

To unsubscribe you must undergo the JetSpeed de-indoctination ritual at the
Temple of Pollux, administered by the Druid High Priestess of Jargon.


----- Original Message -----
From: <Mark.Easparro@Lazard.com>
To: "Jetspeed Developers List" <jetspeed-dev@portals.apache.org>
Cc: <jetspeed-dev@portals.apache.org>
Sent: Friday, August 17, 2007 9:34 AM
Subject: Re: svn commit: r566920 -
/portals/jetspeed-2/trunk/components/jetspeed-portal/src/main/java/org/apach
e/jetspeed/container/invoker/ServletPortletInvoker.java


>
> I have attempted numerous times to unsubscribe from this list without
success.
> I apologize for this email but please advise on how to unsubscribe.
>
> Thanks,
> Mark
>
>
>
>
>
>
>
> |---------+---------------------------->
> |         |                            |
> |         |                            |
> |         |   woonsan@apache.org       |
> |         |   17 Aug 2007 12:56 AM     |
> |         |   Please respond to        |
> |         |   "Jetspeed Developers     |
> |         |   List"                    |
> |         |                            |
> |---------+---------------------------->
>
>---------------------------------------------------------------------------
--------------------------------------------------------|
>   |
|
>   |       To:      jetspeed-dev@portals.apache.org
|
>   |       cc:
|
>   |       Subject: svn commit: r566920 -
|
>   |
/portals/jetspeed-2/trunk/components/jetspeed-portal/src/main/java/org/apach
e/jetspeed/container/invoker/ServletPortletInvok|
>   |       er.java
|
>
>---------------------------------------------------------------------------
--------------------------------------------------------|
>
>
>
> Author: woonsan
> Date: Thu Aug 16 21:56:37 2007
> New Revision: 566920
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=566920
> Log:
> I missed one line which invoked requestContext.getRequest() in parallel
mode in
> ServletPortletInvoker.java. The line can throw an NPE because
> requestContext.getRequest() can be null in parallel mode with very high
load.
> I fixed this problem and tested again parallel rendering with 300
concurrent
> users.
>
> Modified:
>
>
portals/jetspeed-2/trunk/components/jetspeed-portal/src/main/java/org/apache
/jetspeed/container/invoker/ServletPortletInvoker.java
>
>
> Modified:
>
portals/jetspeed-2/trunk/components/jetspeed-portal/src/main/java/org/apache
/jetspeed/container/invoker/ServletPortletInvoker.java
>
> URL:
>
http://svn.apache.org/viewvc/portals/jetspeed-2/trunk/components/jetspeed-po
rtal/src/main/java/org/apache/jetspeed/container/invoker/ServletPortletInvok
er.java?view=diff&rev=566920&r1=566919&r2=566920
>
>
============================================================================
==
> ---
>
portals/jetspeed-2/trunk/components/jetspeed-portal/src/main/java/org/apache
/jetspeed/container/invoker/ServletPortletInvoker.java
>  (original)
> +++
>
portals/jetspeed-2/trunk/components/jetspeed-portal/src/main/java/org/apache
/jetspeed/container/invoker/ServletPortletInvoker.java
>  Thu Aug 16 21:56:37 2007
> @@ -256,7 +256,7 @@
>
> CurrentWorkerContext.setAttribute(ContainerConstants.PORTLET_RESPONSE,
> portletResponse);
>
CurrentWorkerContext.setAttribute(ContainerConstants.METHOD_ID,
> methodID);
>
> CurrentWorkerContext.setAttribute(ContainerConstants.PORTLET_NAME,
> app.getName()+"::"+portletDefinition.getName());
> -
> CurrentWorkerContext.setAttribute(ContainerConstants.PORTAL_CONTEXT,
> requestContext.getRequest().getContextPath());
> +
> CurrentWorkerContext.setAttribute(ContainerConstants.PORTAL_CONTEXT,
> ((HttpServletRequest) servletRequest).getContextPath());
>              }
>
>              PortletRequestContext.createContext(portletDefinition,
> portletInstance, portletRequest, portletResponse);
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>
>
>
>
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
> For additional commands, e-mail: jetspeed-dev-help@portals.apache.org
>


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org


Mime
View raw message