Skip to content

Commit df2309f

Browse files
committed
make fields final
1 parent e3f70ab commit df2309f

File tree

1 file changed

+5
-7
lines changed
  • instrumentation/aws-sdk/aws-sdk-1.11/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v1_11

1 file changed

+5
-7
lines changed

instrumentation/aws-sdk/aws-sdk-1.11/library/src/main/java/io/opentelemetry/instrumentation/awssdk/v1_11/RequestAccess.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -101,8 +101,8 @@ private static String invokeOrNull(@Nullable MethodHandle method, Object obj) {
101101
@Nullable private final MethodHandle getTableName;
102102
@Nullable private final MethodHandle getTopicArn;
103103
@Nullable private final MethodHandle getTargetArn;
104-
@Nullable private MethodHandle getStateMachineArn;
105-
@Nullable private MethodHandle getStepFunctionsActivityArn;
104+
@Nullable private final MethodHandle getStateMachineArn;
105+
@Nullable private final MethodHandle getStepFunctionsActivityArn;
106106

107107
private RequestAccess(Class<?> clz) {
108108
getBucketName = findAccessorOrNull(clz, "getBucketName");
@@ -112,11 +112,9 @@ private RequestAccess(Class<?> clz) {
112112
getTableName = findAccessorOrNull(clz, "getTableName");
113113
getTopicArn = findAccessorOrNull(clz, "getTopicArn");
114114
getTargetArn = findAccessorOrNull(clz, "getTargetArn");
115-
String className = clz.getName();
116-
if (className.startsWith(STEP_FUNCTIONS_REQUEST_CLASS_PREFIX)) {
117-
getStateMachineArn = findAccessorOrNull(clz, "getStateMachineArn");
118-
getStepFunctionsActivityArn = findAccessorOrNull(clz, "getActivityArn");
119-
}
115+
boolean isStepFunction = clz.getName().startsWith(STEP_FUNCTIONS_REQUEST_CLASS_PREFIX);
116+
getStateMachineArn = isStepFunction ? findAccessorOrNull(clz, "getStateMachineArn") : null;
117+
getStepFunctionsActivityArn = isStepFunction ? findAccessorOrNull(clz, "getActivityArn") : null;
120118
}
121119

122120
@Nullable

0 commit comments

Comments
 (0)