-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Nicer exception when no scroll ID is provided #10813
Conversation
@@ -62,7 +63,7 @@ public static String buildScrollId(SearchType searchType, AtomicArray<? extends | |||
} else if (searchType == SearchType.SCAN) { | |||
return buildScrollId(ParsedScrollId.SCAN, searchPhaseResults, attributes); | |||
} else { | |||
throw new ElasticsearchIllegalStateException("search_type [" + searchType + "] not supported"); | |||
throw new ElasticsearchIllegalStateException(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should have an empty exception message?
@Myll I left some comments. |
@rjernst okay, fixed |
@@ -39,6 +39,7 @@ | |||
import org.elasticsearch.search.internal.ShardSearchTransportRequest; | |||
|
|||
import java.io.IOException; | |||
import java.util.Arrays; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leftover import
@Myll Can you remove the leftover import and rebase? |
Hi @stephenfournier Any chance you want to pick this up again, or should we take it over? |
No further feedback. Closing |
#9469