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
Next Next commit
Merge branch 'master' into pr/33268
  • Loading branch information
@jimczi
jimczi committed Sep 3, 2018
commit 7e0ad6ea3c7cd92d056f16a6537f8a8d95e7aeb7

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.