portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark.Easpa...@Lazard.com
Subject Re: svn commit: r566920 - /portals/jetspeed-2/trunk/components/jetspeed-portal/src/main/java/org/apache/jetspeed/container/invoker/ServletPortletInvoker.java
Date Fri, 17 Aug 2007 13:34:23 GMT

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/apache/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-portal/src/main/java/org/apache/jetspeed/container/invoker/ServletPortletInvoker.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


Mime
View raw message