Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Date speedup #159

Merged
merged 2 commits into from Sep 4, 2012
Merged

Date speedup #159

merged 2 commits into from Sep 4, 2012

Conversation

pcl
Copy link
Contributor

@pcl pcl commented Aug 29, 2012

Apache httpclient 4.1.x uses { RFC1036, RFC1123, ASCTIME }, and uses java.text.SimpleDateFormat#parse(String), which raises an exception on failure. Since the Riak server uses RFC1123, this ordering is faster -- we avoid the creation of a ParseException during the failed RFC1036 parse.

See https://issues.apache.org/jira/browse/HTTPCLIENT-1227 for details.

Apache httpclient 4.1.x uses { RFC1036, RFC1123, ASCTIME }, and uses
java.text.SimpleDateFormat#parse(String), which raises an exception
on failure. Since the Riak server uses RFC1123, this ordering
is faster -- we avoid the creation of a ParseException during the
failed RFC1036 parse.

See https://issues.apache.org/jira/browse/HTTPCLIENT-1227 for details.
@travisbot
Copy link

This pull request fails (merged 109f967 into d2478b6).

@travisbot
Copy link

This pull request passes (merged 683b8677 into d2478b6).

@pcl
Copy link
Contributor Author

pcl commented Aug 30, 2012

Looks like this will be resolved in HttpClient 4.2.2, and in the 4.3.x branch. So upgrading your dependency to 4.2.2 once it becomes available will also resolve the issue. I'm not sure what their schedule is.

https://issues.apache.org/jira/browse/HTTPCLIENT-1227

@ghost ghost assigned broach Sep 4, 2012
broach pushed a commit that referenced this pull request Sep 4, 2012
Looks good, thanks. Merging this into 1.0.6 which I'll cut as a public release this week.
@broach broach merged commit bf415bd into basho:master Sep 4, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants