[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wink-dev
Subject:    [jira] Commented: (WINK-114) Cache parsed Accept header content where possible
From:       "Hudson (JIRA)" <jira () apache ! org>
Date:       2009-08-30 10:37:32
Message-ID: 2029703698.1251628652746.JavaMail.jira () brutus
[Download RAW message or body]


    [ https://issues.apache.org/jira/browse/WINK-114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12749245#action_12749245 \
] 

Hudson commented on WINK-114:
-----------------------------

Integrated in Wink-Trunk-JDK1.5 #123 (See \
[http://hudson.zones.apache.org/hudson/job/Wink-Trunk-JDK1.5/123/])  Cache parsed \
Accept header content where possible See []


> Cache parsed Accept header content where possible
> -------------------------------------------------
> 
> Key: WINK-114
> URL: https://issues.apache.org/jira/browse/WINK-114
> Project: Wink
> Issue Type: Improvement
> Components: Common
> Affects Versions: 0.1
> Reporter: Nick Gallardo
> Assignee: Michael Elman
> Fix For: 0.2
> 
> Attachments: WINK-114-generic-sync.patch, WINK-114-generic-sync2.patch, \
> WINK-114-generic.patch, WINK-114-ThreadLocal.patch, WINK-114-use-pattern.patch, \
> WINK-114.patch 
> 
> AcceptHeaderDelete.fromString() is called for every invocation.  This drives calls \
> to string splits and some regex tools that start to show up in profiles as taking \
> up a notable amount of time.  Since the content for the Accept headers will remain \
> within a fairly bounded set, we can cache the parsed content and improve \
> performance. Here's a patch with a simple solution.  Really I think there's a \
> better way to do this.  Presumably, there are other headers where the same thing \
> applies.  We can handle those on an individual basis or we can come up with \
> something more generic that can be used by all HeaderDelegates.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic