[
https://issues.apache.org/jira/browse/SHINDIG-257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12595662#action_12595662
]
Brian Eaton commented on SHINDIG-257:
-------------------------------------
SHINDIG-226 doesn't look thread-safe either, and Kevin Brown had a few comments
on it. Would you be willing to address Kevin's comments and move your code to
HttpUtil.java?
About related issue patches vs global patches: in general I agree with you, but
I also like to see build warnings get cleaned up. It's one thing to send in a
giant patch for several unrelated issues, but adding a few small bits of code
cleanup seems within reason. (I defer to the shindig team on which they
prefer, of course...)
> shindig handling of HTTP dates is not thread safe
> -------------------------------------------------
>
> Key: SHINDIG-257
> URL: https://issues.apache.org/jira/browse/SHINDIG-257
> Project: Shindig
> Issue Type: Bug
> Reporter: Brian Eaton
> Attachments: dateformat.patch
>
>
> I noticed this during a bit of load testing, spurious exceptions from
> SimpleDateFormat because it isn't thread safe.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
