Skip to content

Fix NoSuchElementException in Hystrix.endCurrentThreadExecutingCommand#133

Merged
benjchristensen merged 1 commit intoNetflix:masterfrom
benjchristensen:fix-error-handling-endCurrentThreadExecutingCommand
Mar 31, 2013
Merged

Fix NoSuchElementException in Hystrix.endCurrentThreadExecutingCommand#133
benjchristensen merged 1 commit intoNetflix:masterfrom
benjchristensen:fix-error-handling-endCurrentThreadExecutingCommand

Conversation

@benjchristensen
Copy link
Copy Markdown
Contributor

If a semaphore isolated HystrixCommand gets a response from a request cache this exception will be thrown:

Caused by: java.util.NoSuchElementException
at java.util.LinkedList.removeFirst(LinkedList.java:268)
at java.util.LinkedList.pop(LinkedList.java:799)
at com.netflix.hystrix.Hystrix.endCurrentThreadExecutingCommand(Hystrix.java:88)
at com.netflix.hystrix.HystrixCommand.queueInSemaphore(HystrixCommand.java:647)
at com.netflix.hystrix.HystrixCommand.queue(HystrixCommand.java:551)
at com.netflix.platform.dependencycommand.NFDependencyCommand.queue(NFDependencyCommand.java:346)

benjchristensen added a commit that referenced this pull request Mar 31, 2013
…rrentThreadExecutingCommand

Fix NoSuchElementException in Hystrix.endCurrentThreadExecutingCommand
@benjchristensen benjchristensen merged commit 366f11a into Netflix:master Mar 31, 2013
@cloudbees-pull-request-builder
Copy link
Copy Markdown

Hystrix-pull-requests #6 SUCCESS
This pull request looks good

If a semaphore isolated HystrixCommand gets a response from a request cache this exception will be thrown:

Caused by: java.util.NoSuchElementException
        at java.util.LinkedList.removeFirst(LinkedList.java:268)
        at java.util.LinkedList.pop(LinkedList.java:799)
        at com.netflix.hystrix.Hystrix.endCurrentThreadExecutingCommand(Hystrix.java:88)
        at com.netflix.hystrix.HystrixCommand.queueInSemaphore(HystrixCommand.java:647)
        at com.netflix.hystrix.HystrixCommand.queue(HystrixCommand.java:551)
        at com.netflix.platform.dependencycommand.NFDependencyCommand.queue(NFDependencyCommand.java:346)
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.

2 participants