dolphinscheduler icon indicating copy to clipboard operation
dolphinscheduler copied to clipboard

[Bug] [dolphinscheduler-task-plugin/dolphinscheduler-task-api] ParameterUtils set TIME TYPE parameter is String, should be java.sql.Time

Open yinxiaolog opened this issue 1 year ago • 1 comments

Search before asking

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

What happened

image image

What you expected to happen

TIME TYPE is setted String,execute sql as String,not Time,there wiil be an error when execute this sql.

前端SQL自定义参数为TIME类型时,传到后端给SQL执行设置参数类型时,会把Time类型设置为String,后续sql执行时会导致类型不匹配而出错。

How to reproduce

It‘’s easy to reproduce when setting self-define param type TIME.

设置SQL执行时自定义参数类型为TIME时,当后端执行引擎特殊适配这种类型时就会出错。

Anything else

No response

Version

dev

Are you willing to submit PR?

  • [X] Yes I am willing to submit a PR!

Code of Conduct

yinxiaolog avatar May 21 '24 08:05 yinxiaolog

Search before asking

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

What happened

image image

What you expected to happen

TIME TYPE is setted String, execute sql as String, not Time, there will be an error when execute this sql.

When the front-end SQL custom parameter is of type TIME, when it is passed to the back-end to set the parameter type for SQL execution, the Time type will be set to String, which will cause a type mismatch and an error when subsequent SQL is executed.

How to reproduce

It’s easy to reproduce when setting self-define param type TIME.

When setting the custom parameter type during SQL execution to TIME, an error will occur when the back-end execution engine specially adapts to this type.

Anything else

No response

Version

dev

Are you willing to submit PR?

  • [X] Yes I am willing to submit a PR!

Code of Conduct

github-actions[bot] avatar May 21 '24 08:05 github-actions[bot]

I have submitted pr to fix this bug,it‘’s here https://github.com/apache/dolphinscheduler/pull/16050

yinxiaolog avatar May 23 '24 02:05 yinxiaolog