mdayakar commented on code in PR #6422:
URL: https://github.com/apache/hive/pull/6422#discussion_r3153609191
##########
hplsql/src/main/java/org/apache/hive/hplsql/Stmt.java:
##########
@@ -1082,7 +1085,7 @@ else if (trace) {
trace(ctx, "Variable not found: " + ctx.L_ID(i).getText());
}
}
- exec.setSqlCode(SqlCodes.SUCCESS);
+ exec.setSqlSuccess();
Review Comment:
It may be redundant here but it won't override anything as the same
statement we are calling above in case of success. Anyway I will remove this
statement as a part of some other PR as currently I could see some random
failures everytime, now its green :).
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
