Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search Index equals method has bug #111

Closed
gurminder71 opened this issue May 26, 2024 · 0 comments · Fixed by #114 or #115
Closed

Search Index equals method has bug #111

gurminder71 opened this issue May 26, 2024 · 0 comments · Fixed by #114 or #115

Comments

@gurminder71
Copy link
Contributor

I see couple of equal methods are missing. But the following is a bug:

diff --git a/src/main/java/com/google/cloud/solutions/spannerddl/parser/ASTcreate_search_index_statement.java b/src/main/java/com/google/cloud/solutions/spannerddl/parser/ASTcreate_search_index_statement.java
index 8e10e35..24ce51b 100644
--- a/src/main/java/com/google/cloud/solutions/spannerddl/parser/ASTcreate_search_index_statement.java
+++ b/src/main/java/com/google/cloud/solutions/spannerddl/parser/ASTcreate_search_index_statement.java
@@ -99,7 +99,7 @@ public class ASTcreate_search_index_statement extends SimpleNode
 
   @Override
   public boolean equals(Object other) {
-    if (other instanceof ASTcreate_index_statement) {
+    if (other instanceof ASTcreate_search_index_statement) {
       return this.toString().equals(other.toString());
     }
     return false;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant