Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 7ebc381

Browse files
committed
Avoid NPE warning.
1 parent e2cdb74 commit 7ebc381

File tree

1 file changed

+5
-4
lines changed
  • sdks/java/io/jdbc/src/main/java/org/apache/beam/sdk/io/jdbc

1 file changed

+5
-4
lines changed

sdks/java/io/jdbc/src/main/java/org/apache/beam/sdk/io/jdbc/JdbcIO.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2196,9 +2196,10 @@ public PCollection<V> expand(PCollection<T> input) {
21962196
autoSharding == null || (autoSharding && input.isBounded() != IsBounded.UNBOUNDED),
21972197
"Autosharding is only supported for streaming pipelines.");
21982198

2199-
long batchSize = getBatchSize() == null ? DEFAULT_BATCH_SIZE : getBatchSize();
2200-
long maxBufferingDuration =
2201-
getMaxBatchBufferingDuration() == null
2199+
Long batchSizeAsLong = getBatchSize();
2200+
long batchSize = batchSizeAsLong == null ? DEFAULT_BATCH_SIZE : batchSizeAsLong;
2201+
Long maxBufferingDurationAsLong = getMaxBatchBufferingDuration();
2202+
long maxBufferingDuration = maxBufferingDurationAsLong == null
22022203
? DEFAULT_MAX_BATCH_BUFFERING_DURATION
22032204
: getMaxBatchBufferingDuration();
22042205

@@ -2215,7 +2216,7 @@ public PCollection<V> expand(PCollection<T> input) {
22152216
.setStatement(getStatement())
22162217
.setRetryConfiguration(getRetryConfiguration())
22172218
.setReturnResults(true)
2218-
.setBatchSize(1L) // We are writing iterables 1 at a time.
2219+
.setBatchSize(1L) // We are writing iterables 1 at a time.
22192220
.setMaxBatchBufferingDuration(maxBufferingDuration)
22202221
.build())));
22212222
}

0 commit comments

Comments
 (0)