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

Handle - More informative error handling for HTTP requests [JIRA: RIAK-1845] #532

Merged
merged 1 commit into from Aug 5, 2015

Conversation

zeeshanlakhani
Copy link
Contributor

  • Catchall for error'ing from yz_solr:dist_search, making certain errors
    more informative, as errors were getting caught by the scrub_headers
    function and blowing-up on a benign lists:keydelete.
  • Little Indentation Clean-Up on yz_pb_search.erl as well
  • Handles More informative error handling for HTTP requests [JIRA: RIAK-1845] #498 (RIAK-1845) (RIAK-1845) / RIAK-1845

  more informative, as errors were getting caught by the scrub_headers
  function and blowing-up on a benign lists:keydelete.

* Little Indentation Clean-Up on yz_pb_search.erl as well

* Handles 498/RIAK-1845
@jvoegele
Copy link
Contributor

jvoegele commented Aug 4, 2015

👍

@zeeshanlakhani
Copy link
Contributor Author

@jvoegele gotta read about +1's and bors https://github.com/basho/internal_wiki/wiki/Bors-automated-integration-CI-bot for reviewing :)

@jvoegele
Copy link
Contributor

jvoegele commented Aug 5, 2015

👍 095c125

borshop added a commit that referenced this pull request Aug 5, 2015
…rors-yz_wm_search

Handle - More informative error handling for HTTP requests [JIRA: RIAK-1845]

Reviewed-by: jvoegele
@zeeshanlakhani
Copy link
Contributor Author

@borshop merge

@borshop borshop merged commit 095c125 into 2.0 Aug 5, 2015
@zeeshanlakhani zeeshanlakhani deleted the handling/zl/better-handle-of-wm-errors-yz_wm_search branch August 7, 2015 22:36
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