refactor: remove duplicate getRightItem call in TablesNamesFinder (#2370)

1. Mark Join.getRightItem() and setRightItem() as @deprecated since they return the same value as getFromItem().
2. Remove the duplicate call to join.getRightItem().accept(this, context) in TablesNamesFinder.visit() method, as join.getFromItem() already processes the same FromItem object.

Co-authored-by: zhezzz <taylorlzz@outlook.com>

Author: zhezzz

src/main/java/net/sf/jsqlparser/statement/select/Join.java

@@ -277,10 +277,12 @@ public Join withCross(boolean cross) {
     /**
      * Returns the right item of the join
      */
+    @Deprecated
     public FromItem getRightItem() {
         return fromItem;
     }
 
+    @Deprecated
     public void setRightItem(FromItem item) {
         fromItem = item;
     }

src/main/java/net/sf/jsqlparser/util/TablesNamesFinder.java

@@ -1379,7 +1379,6 @@ private <S> void visitJoins(List<Join> joins, S context) {
         }
         for (Join join : joins) {
             join.getFromItem().accept(this, context);
-            join.getRightItem().accept(this, context);
             for (Expression expression : join.getOnExpressions()) {
                 expression.accept(this, context);
             }