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] ApplicationOperation uses conf instead of sessionConf #6385

Open
2 of 4 tasks
ic4y opened this issue May 11, 2024 · 2 comments
Open
2 of 4 tasks

[Bug] ApplicationOperation uses conf instead of sessionConf #6385

ic4y opened this issue May 11, 2024 · 2 comments
Labels
kind:bug This is a clearly a bug priority:major

Comments

@ic4y
Copy link

ic4y commented May 11, 2024

Code of Conduct

Search before asking

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

Describe the bug

When using a Kyuubi cluster to proxy multiple big data environments, we use the kyuubi.engineEnv.HADOOP_CONF_DIR in sessionConf to specify the configuration related to the big data environment. However, after creating the engine, the yarnClient used in KyuubiApplicationManager.getApplicationInfo is not created using sessionConf, which will result in the connection not being the cluster specified by kyuubi.engineEnv.HADOOP_CONF_DIR.

Related classes:
org.apache.kyuubi.engine.KyuubiApplicationManager#getApplicationInfo
org.apache.kyuubi.engine.ApplicationOperation#getApplicationInfoByTag
org.apache.kyuubi.engine.ApplicationOperation#killApplicationByTag
org.apache.kyuubi.engine.YarnApplicationOperation#initialize
org.apache.kyuubi.util.KyuubiHadoopUtils#newYarnConfiguration

The same problem will also occur in BatchSession, KubernetesApplicationOperation, HiveEngine YARN mode, and so on.

Affects Version(s)

1.9.0

Kyuubi Server Log Output

None

Kyuubi Engine Log Output

None

Kyuubi Server Configurations

None

Kyuubi Engine Configurations

None

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 May 11, 2024
Copy link

Hello @ic4y,
Thanks for finding the time to report the issue!
We really appreciate the community's efforts to improve Apache Kyuubi.

@wForget
Copy link
Member

wForget commented May 11, 2024

@ic4y Thank you for reporting this issue. YarnApplicationOperation should probably consider kyuubi.engineEnv.HADOOP_CONF_DIR/YARN_CONF_DIR of session conf. and cc @yaooqinn

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

2 participants