zeroflag commented on a change in pull request #1733:
URL: https://github.com/apache/hive/pull/1733#discussion_r579069892
##########
File path: hplsql/src/main/java/org/apache/hive/hplsql/Exec.java
##########
@@ -328,16 +334,16 @@ public String callStackPop() {
*/
public Var findVariable(String name) {
Var var;
- String name1 = name;
+ String name1 = name.toUpperCase();
String name1a = null;
String name2 = null;
Scope cur = exec.currentScope;
Package pack;
Package packCallContext = exec.getPackageCallContext();
ArrayList<String> qualified = exec.meta.splitIdentifier(name);
if (qualified != null) {
- name1 = qualified.get(0);
- name2 = qualified.get(1);
+ name1 = qualified.get(0).toUpperCase();
Review comment:
splitIdentifier() is also used with table names and column names, I
think it's better to leave column and table names alone as it would modify the
HiveQL queries and it's rather Hive's territory than HPLSQL.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
