Skip to content

Commit 01ffed6

Browse files
committed
Fix NPE due to missing params set during serialization
1 parent 44b1a67 commit 01ffed6

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

server/src/main/java/org/elasticsearch/action/bulk/BulkRequest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@
4848
import java.util.Objects;
4949
import java.util.Set;
5050

51+
import static java.util.Collections.emptySet;
5152
import static org.elasticsearch.action.ValidateActions.addValidationError;
5253

5354
/**
@@ -86,7 +87,7 @@ public class BulkRequest extends LegacyActionRequest
8687
private Boolean globalRequireAlias;
8788
private Boolean globalRequireDatsStream;
8889
private boolean includeSourceOnError = true;
89-
private Set<String> paramsUsed;
90+
private Set<String> paramsUsed = emptySet();
9091

9192
private long sizeInBytes = 0;
9293

server/src/main/java/org/elasticsearch/action/bulk/TransportAbstractBulkAction.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,6 @@
4848
import org.elasticsearch.transport.TransportService;
4949

5050
import java.io.IOException;
51-
import java.util.Collections;
5251
import java.util.HashMap;
5352
import java.util.HashSet;
5453
import java.util.Map;
@@ -486,10 +485,7 @@ private void doStreamsChecks(
486485
}
487486

488487
private boolean streamsRestrictedParamsUsed(BulkRequest bulkRequest) {
489-
return Sets.difference(
490-
bulkRequest.requestParamsUsed() == null ? Collections.emptySet() : bulkRequest.requestParamsUsed(),
491-
STREAMS_ALLOWED_PARAMS
492-
).isEmpty() == false;
488+
return Sets.difference(bulkRequest.requestParamsUsed(), STREAMS_ALLOWED_PARAMS).isEmpty() == false;
493489
}
494490

495491
/**

0 commit comments

Comments
 (0)