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.

Sign up for GitHub

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

Null completion field should not throw IAE #33268

Merged
merged 12 commits into from
Sep 3, 2018
Merged

Null completion field should not throw IAE #33268

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
1c5da09
[Issue #33200] null completion values should not throw IAE and be add…
tony-dillon Aug 28, 2018
9d96f19
[Issue #33200] fixed test
tony-dillon Aug 30, 2018
681b868
[Issue #33200] updated test to use Token.NULL_VALUE
tony-dillon Aug 30, 2018
47513b9
[Issue #33200] stopped adding null fields to ignored field list. rest…
tony-dillon Aug 30, 2018
886e18f
Merge branch 'master' into null_completion
tony-dillon Aug 30, 2018
113245b
[Issue #33200] fixed error where wrong document was being asserted on
tony-dillon Aug 30, 2018
94790b1
[Issue #33200] set correct null field value on test document
tony-dillon Aug 30, 2018
576dc7b
[Issue #33200] set nullField on correct document
tony-dillon Aug 30, 2018
563de3f
[Issue #33200] remove unused use statement
tony-dillon Aug 30, 2018
27adc3f
remote IT for null completion field value
jimczi Sep 3, 2018
7e0ad6e
Merge branch 'master' into pr/33268
jimczi Sep 3, 2018
75db758
add local change
jimczi Sep 3, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
add local change
  • Loading branch information
@jimczi
jimczi committed Sep 3, 2018
commit 75db758e1fde46ebcc035dff395617bc091e5c0f
2 changes: 0 additions & 2 deletions server/src/test/java/org/elasticsearch/search/suggest/CompletionSuggestSearchIT.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,10 @@
import org.elasticsearch.action.search.SearchResponse;
import org.elasticsearch.action.support.master.AcknowledgedResponse;
import org.elasticsearch.common.FieldMemoryStats;
import org.elasticsearch.common.Strings;
import org.elasticsearch.common.settings.Settings;
import org.elasticsearch.common.unit.Fuzziness;
import org.elasticsearch.common.xcontent.XContentBuilder;
import org.elasticsearch.common.xcontent.XContentFactory;
import org.elasticsearch.common.xcontent.XContentType;
import org.elasticsearch.index.mapper.MapperParsingException;
import org.elasticsearch.plugins.Plugin;
import org.elasticsearch.search.aggregations.AggregationBuilders;
Expand Down