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

[BUG] incorrectly recognizes --hivevar parameters as hiveconf when using HIVE_SQL #6897

Open
3 of 4 tasks
ic4y opened this issue Jan 17, 2025 · 0 comments
Open
3 of 4 tasks
Labels
kind:bug This is a clearly a bug priority:major

Comments

@ic4y
Copy link
Contributor

ic4y commented Jan 17, 2025

Code of Conduct

Search before asking

  • I have searched in the issues and found no similar issues.

Describe the bug

The version in use is 1.9.1, and kyuubi.engine.type=HIVE_SQL.

I use the following command to connect to Kyuubi:
./beeline -u 'jdbc:hive2://192.28.16.177:10009' --hivevar table_dt=20241113 -n xxxxx -p yyyyyyy

From the logs, we can see that the value of table_dt is passed via -conf:

Image

Image

When using the set command to print all variables, everything is displayed, and we can see that table_dt’s type is conf instead of hivevar:

Image

Image

However, if we connect directly to HiveServer2 for testing:

./beeline -u 'jdbc:hive2://192.28.18.38:7001' --hivevar tabel_dt=20241113 -n xxxxxx -p yyyyyyy

Using the set command, we can see that the variable tabel_dt is of type hivevar:

Image

In Hive:
• hiveconf is typically used for configuration or system-level parameters. It can also be referenced in queries within scripts.
• hivevar is mainly used for user-defined variables that can be referenced in queries but does not directly affect Hive’s own configuration.

If we treat a hivevar as hiveconf directly, problems can easily occur.

Affects Version(s)

1.9.1

Kyuubi Server Log Output

Kyuubi Engine Log Output

Kyuubi Server Configurations

Kyuubi Engine Configurations

Additional context

No response

Are you willing to submit PR?

  • Yes. I would be willing to submit a PR with guidance from the Kyuubi community to fix.
  • No. I cannot submit a PR at this time.
@ic4y ic4y added kind:bug This is a clearly a bug priority:major labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug This is a clearly a bug priority:major
Projects
None yet
Development

No branches or pull requests

1 participant