Skip to content

Commit a08ffc8

Browse files
authored
fix passing null routing to Elasticsearch's builder.
Closes #3278 Signed-off-by: Peter-Josef Meisch <pj.meisch@sothawo.com>
1 parent c3eb590 commit a08ffc8

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

src/main/java/org/springframework/data/elasticsearch/client/elc/RequestConverter.java

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -589,10 +589,13 @@ public IndexRequest<?> documentIndexRequest(IndexQuery query, IndexCoordinates i
589589
builder.version(query.getVersion()).versionType(versionType);
590590
}
591591

592-
builder //
593-
.ifSeqNo(query.getSeqNo()) //
594-
.ifPrimaryTerm(query.getPrimaryTerm()) //
595-
.routing(query.getRouting()); //
592+
builder
593+
.ifSeqNo(query.getSeqNo())
594+
.ifPrimaryTerm(query.getPrimaryTerm());
595+
596+
if (query.getRouting() != null) {
597+
builder.routing(query.getRouting()); //
598+
}
596599

597600
if (query.getOpType() != null) {
598601
switch (query.getOpType()) {

0 commit comments

Comments
 (0)