[Bug][Dinky] Documentation bug or improvement

[wangjc0216]

Search before asking

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

Description

dinky plugin should be added specific version

I recently used dolphin scheduler with dinky v0.7.3, but failed. log below:

[LOG-PATH]: /opt/dolphinscheduler/logs/20230720/10280823947712_10-53-59.log, [HOST]:  Host{address='dolphinscheduler-worker-0.dolphinscheduler-worker-headless:1234', ip='dolphinscheduler-worker-0.dolphinscheduler-worker-headless', port=1234}
...
[INFO] 2023-07-20 14:48:59.012 +0800 - access url: http://172.16.97.197:8888/openapi/getJobInstance?id=null
[INFO] 2023-07-20 14:48:59.018 +0800 - dinky task succeed with results: {"datas":null,"code":1,"msg":"Failed to convert value of type 'java.lang.String' to required type 'java.lang.Integer'; nested exception is java.lang.NumberFormatException: For input string: \"null\"","time":"2023-07-20 14:48:59"}
...

I found dinky has changed its api uri

curl http://127.0.0.1:8888/openapi/getJobInstanceByTaskId?id=5
uri be changed from /openapi/getJobInstance to /openapi/getJobInstanceByTaskId

Documentation Links

No response

Are you willing to submit a PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[CallMeKingsley97]

Why the jobInstanceId is null

[wangjc0216]

Why the jobInstanceId is null
I use mysql as datasource in dinky not flink ,I don't know if it's related @CallMeKingsley97

[CallMeKingsley97]

Anyway, jobInstanceId should't be "null". Could you post more logs ?
By the way, i found nothing from dinky(v0.7.3) about "/openapi/getJobInstanceByTaskId" .😂

[wangjc0216]

I found /openapi/getJobInstanceByTaskId in dinky :

But I notice /openapi/getJobInstance and /openapi/getJobInstanceByTaskId both exist.

I submit task in dinky and log is below , it doesn't contain JobId

[CallMeKingsley97]

Could you submit a new task for using different sql?, and watch the return data, take a closer look at the fields called "jobInstanceId".
After you submit a task, the program will try to get jobInstanceInfo.

[CallMeKingsley97]

Are there any questions? @wangjc0216

[wangjc0216]

Are there any questions? @wangjc0216

@CallMeKingsley97 I use flink as dinky param , and I will get expected value(different from dinky-mysql):

I think there should be something wrong in dinky plugin. I am not familiar about Java and Dolphin Scheduler, so I may spend some times to find out what is the real reason.

[CallMeKingsley97]

What's the meaning of "get expected value(different from dinky-mysql)",I think the data returned from dinky which in your screenshot is right. I‘m not familiar about dinky, Can you describe it in detail?