davidradl commented on code in PR #25815:
URL: https://github.com/apache/flink/pull/25815#discussion_r1894022766
##########
flink-runtime/src/test/java/org/apache/flink/streaming/util/KeyedTwoInputStreamOperatorTestHarness.java:
##########
@@ -64,8 +67,24 @@ public KeyedTwoInputStreamOperatorTestHarness(
}
public int numKeyedStateEntries() {
- AbstractStreamOperator<?> abstractStreamOperator =
(AbstractStreamOperator<?>) operator;
- KeyedStateBackend<Object> keyedStateBackend =
abstractStreamOperator.getKeyedStateBackend();
+ KeyedStateBackend<?> keyedStateBackend;
+ if (operator instanceof AbstractAsyncStateStreamOperator) {
Review Comment:
as reflection is quite costly and the operator is an instance variable, can
we organize the code so the reflection is done once and caches rather than on
every access. I know this is test code - but it could save some time.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
