{"push_id":1477500403,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"2f7ad4463e7c283627d75a44096d65763635226c","before":"6a79cdd01450ab3608e6810053276959ce1b61ce","commits":[{"sha":"2f7ad4463e7c283627d75a44096d65763635226c","author":{"name":"MikeGoldsmith","email":"f193bbb68a08aa17764473fa0e35f2beba56cb5f@gmail.com"},"message":"NCBC-1142: Query does not recover after service restart\n\nMOTIVATION\n----------\nWhen a N1QL query is submitted to a cluster that is restarting, the\navailable Query URIs can be exhausted through retries until none\nare left. However, when this happens an empty URI string is returned\nwhich causes a InvalidOperationsException when trying to send to an\ninvalid URI with the HttpClient. Also, no logging is available to\nindicate what has happened.\n\nMODIFICATIONS\n-------------\n- In the case the QueryClient doesn't retrieve a valid URI when trying\n  to submit a query, log out an error and return a failure result\n- Add log message to ExceptionUtils for empty N1QL URIs\n- Refactor GetRandom in ArrayExtensions to remove a duplicate and make\n  the main implementation more clear\n\nRESULT\n------\nIf the QueryClient does not retrieve a valid URI when trying to submit a\nquery, it logs out what has happened and returns a failed request to\nprevent further retries.\n\nChange-Id: Ifce8731190814bd2784b4822c1a5176ef7aa4a79\nReviewed-on: http://review.couchbase.org/71189\nReviewed-by: Jeffry Morris <jeffrymorris@gmail.com>\nTested-by: Jeffry Morris <jeffrymorris@gmail.com>","distinct":true,"url":"https://api.github.com/repos/couchbase/couchbase-net-client/commits/2f7ad4463e7c283627d75a44096d65763635226c"}]}
{"actor":{"display_login":"ns-codereview"}}