Skip to content

Commit ef2edba

Browse files
committed
Conversions: resolveExecutionProfile only when needed
As this can be expensive
1 parent 1b455ff commit ef2edba

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

core/src/main/java/com/datastax/oss/driver/internal/core/cql/Conversions.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -535,17 +535,18 @@ public static CoordinatorException toThrowable(
535535

536536
public static boolean resolveIdempotence(Request request, InternalDriverContext context) {
537537
Boolean requestIsIdempotent = request.isIdempotent();
538-
DriverExecutionProfile executionProfile = resolveExecutionProfile(request, context);
539538
return (requestIsIdempotent == null)
540-
? executionProfile.getBoolean(DefaultDriverOption.REQUEST_DEFAULT_IDEMPOTENCE)
539+
? resolveExecutionProfile(request, context)
540+
.getBoolean(DefaultDriverOption.REQUEST_DEFAULT_IDEMPOTENCE)
541541
: requestIsIdempotent;
542542
}
543543

544544
public static Duration resolveRequestTimeout(Request request, InternalDriverContext context) {
545-
DriverExecutionProfile executionProfile = resolveExecutionProfile(request, context);
546-
return request.getTimeout() != null
547-
? request.getTimeout()
548-
: executionProfile.getDuration(DefaultDriverOption.REQUEST_TIMEOUT);
545+
Duration timeout = request.getTimeout();
546+
return timeout != null
547+
? timeout
548+
: resolveExecutionProfile(request, context)
549+
.getDuration(DefaultDriverOption.REQUEST_TIMEOUT);
549550
}
550551

551552
public static RetryPolicy resolveRetryPolicy(Request request, InternalDriverContext context) {

0 commit comments

Comments
 (0)