Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Fix "undefinex index type" when indexing #140

Merged
merged 2 commits into from Dec 18, 2019
Merged

Fix "undefinex index type" when indexing #140

merged 2 commits into from Dec 18, 2019

Conversation

Jalle19
Copy link
Contributor

@Jalle19 Jalle19 commented Dec 18, 2019

Fixes #138

hugovk
hugovk reviewed Dec 18, 2019
View reviewed changes
Copy link
Contributor

@hugovk hugovk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of copy-pastos.

tests/Documents/Bulk/BulkResponseAggregatorTest.php Outdated Show resolved Hide resolved
tests/Documents/Bulk/BulkResponseAggregatorTest.php Outdated Show resolved Hide resolved
Fix a regression from 78b9728
9b5ce74 
Fixes "undefined index type" during normal bulk indexing operations. Fixes #138.
@Jalle19 Jalle19 merged commit dd64a60 into master Dec 18, 2019
@Jalle19 Jalle19 deleted the hotfix/bulk-response-errors branch December 18, 2019 11:01
@hugovk hugovk mentioned this pull request Dec 18, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@hugovk hugovk hugovk left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

BulkResponseAggregator Undefined index
2 participants
@Jalle19 @hugovk