Skip to content

Commit 54c5914

Browse files
committed
JAVA-1126: Factored expression into a final field
1 parent 9fb2935 commit 54c5914

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/main/com/mongodb/DBCollectionImpl.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -537,13 +537,15 @@ private class OrderedRunGenerator implements Iterable<Run> {
537537
private final DBPort port;
538538
private final WriteConcern writeConcern;
539539
private final DBEncoder encoder;
540+
private final int maxBatchWriteSize;
540541

541542
public OrderedRunGenerator(final List<WriteRequest> writeRequests, final WriteConcern writeConcern, final DBEncoder encoder,
542543
final DBPort port) {
543544
this.writeRequests = writeRequests;
544545
this.port = port;
545546
this.writeConcern = writeConcern.continueOnError(false);
546547
this.encoder = encoder;
548+
this.maxBatchWriteSize = db.getConnector().getServerDescription(port.getAddress()).getMaxWriteBatchSize();
547549
}
548550

549551
@Override
@@ -570,8 +572,7 @@ public Run next() {
570572
private int getStartIndexOfNextRun() {
571573
WriteRequest.Type type = writeRequests.get(curIndex).getType();
572574
for (int i = curIndex; i < writeRequests.size(); i++) {
573-
if (i == curIndex + db.getConnector().getServerDescription(port.getAddress()).getMaxWriteBatchSize()
574-
|| writeRequests.get(i).getType() != type) {
575+
if (i == curIndex + maxBatchWriteSize || writeRequests.get(i).getType() != type) {
575576
return i;
576577
}
577578
}
@@ -592,13 +593,15 @@ private class UnorderedRunGenerator implements Iterable<Run> {
592593
private final DBPort port;
593594
private final WriteConcern writeConcern;
594595
private final DBEncoder encoder;
596+
private final int maxBatchWriteSize;
595597

596598
public UnorderedRunGenerator(final List<WriteRequest> writeRequests, final WriteConcern writeConcern,
597599
final DBEncoder encoder, final DBPort port) {
598600
this.writeRequests = writeRequests;
599601
this.port = port;
600602
this.writeConcern = writeConcern.continueOnError(true);
601603
this.encoder = encoder;
604+
this.maxBatchWriteSize = db.getConnector().getServerDescription(port.getAddress()).getMaxWriteBatchSize();
602605
}
603606

604607
@Override
@@ -629,7 +632,7 @@ public Run next() {
629632
}
630633
run.add(writeRequest, curIndex);
631634
curIndex++;
632-
if (run.size() > db.getConnector().getServerDescription(port.getAddress()).getMaxWriteBatchSize()) {
635+
if (run.size() > maxBatchWriteSize) {
633636
return runs.remove(run.type);
634637
}
635638
}

0 commit comments

Comments
 (0)