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

[Improvement] [Task-Seatunnel] config file suffix parse error #16956

Open
3 tasks done
Xuzhengz opened this issue Jan 13, 2025 · 0 comments
Open
3 tasks done

[Improvement] [Task-Seatunnel] config file suffix parse error #16956

Xuzhengz opened this issue Jan 13, 2025 · 0 comments
Assignees
Labels
improvement make more easy to user or prompt friendly

Comments

@Xuzhengz
Copy link

Search before asking

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

What happened

The content of the seatunnel task file uses the result passed upstream, which is in JSON format. However, the file is created with a conf suffix because the variable values were not replaced with the JSON result passed upstream during the validation process of JSONUtils.checkJsonValid

What you expected to happen

When passing the JSON Utils. checkJsonValid checksum, parse the variable value into the result passed upstream

How to reproduce

  1. Set a config name for the global parameter and pass in a JSON format seatunnel task configuration information
  2. Use ${config} as task configuration in the seatunnel task

Anything else

No response

Version

dev

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@Xuzhengz Xuzhengz added bug Something isn't working Waiting for reply Waiting for reply labels Jan 13, 2025
@SbloodyS SbloodyS added improvement make more easy to user or prompt friendly and removed bug Something isn't working Waiting for reply Waiting for reply labels Jan 14, 2025
@SbloodyS SbloodyS changed the title [Bug] [Task-Seatunnel] config file suffix parse error [Improvement] [Task-Seatunnel] config file suffix parse error Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement make more easy to user or prompt friendly
Projects
None yet
Development

No branches or pull requests

2 participants