You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2021/11/26 08:56:08 UTC

[flink] 01/03: [hotfix][tests] Extract originOwner variable

This is an automated email from the ASF dual-hosted git repository.

chesnay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git

commit bca54f3311d95ff90c2e1dfb17e9e5a539adc879
Author: Chesnay Schepler <ch...@apache.org>
AuthorDate: Thu Nov 25 15:46:30 2021 +0100

    [hotfix][tests] Extract originOwner variable
---
 .../org/apache/flink/architecture/rules/ApiAnnotationRules.java    | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/flink-architecture-tests/src/test/java/org/apache/flink/architecture/rules/ApiAnnotationRules.java b/flink-architecture-tests/src/test/java/org/apache/flink/architecture/rules/ApiAnnotationRules.java
index e1cc723..9f7078a 100644
--- a/flink-architecture-tests/src/test/java/org/apache/flink/architecture/rules/ApiAnnotationRules.java
+++ b/flink-architecture-tests/src/test/java/org/apache/flink/architecture/rules/ApiAnnotationRules.java
@@ -137,11 +137,12 @@ public class ApiAnnotationRules {
                                         @Override
                                         public boolean apply(JavaMethodCall call) {
                                             final JavaClass targetOwner = call.getTargetOwner();
-                                            if (call.getOriginOwner().equals(targetOwner)) {
+                                            final JavaClass originOwner = call.getOriginOwner();
+                                            if (originOwner.equals(targetOwner)) {
                                                 return false;
                                             }
-                                            if (call.getOriginOwner().isInnerClass()
-                                                    && call.getOriginOwner()
+                                            if (originOwner.isInnerClass()
+                                                    && originOwner
                                                             .getEnclosingClass()
                                                             .map(targetOwner::equals)
                                                             .orElse(false)) {