[Feature-15953][datasource-plugin] Add dolphindb datasource #16116
Conversation
yinxiaolog
requested review from
EricGao888,
songjianet,
Amy0104 and
caishunfeng
as code owners
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| String[] hostSeperator = connectionParams.getAddress().split(Constants.DOUBLE_SLASH); | ||
| String[] hostPortArray = hostSeperator[hostSeperator.length - 1].split(Constants.COMMA); | ||
| dataSourceParamDTO.setPort(Integer.parseInt(hostPortArray[0].split(Constants.COLON)[1])); |
Check notice
Code scanning / CodeQL
Missing catch of NumberFormatException Note
|
Like #16127, please find a PMC/Committer that is willing to maintain this datasource plugin on an ongoing basis. Otherwise we won't accept it. @yinxiaolog |
I will maintain this datasource. |
ruanwenjun
force-pushed
the
Feature-15953
branch
from
6b5464f to
4d8205d
Compare
|
@yinxiaolog Please create a DSIP issue firstly. #14102 |
|
Hello! I am a member of the DolphinDB team, currently responsible for advancing the integration of the DolphinDB datasource into the DolphinScheduler project. Here, I would like to gain a detailed understanding of the issues encountered in the previously submitted code regarding the issue. If you could provide detailed information about the situation, I would be very grateful. Looking forward to your reply, thank you! @ruanwenjun @yinxiaolog |
Hi @yinxiaolog, Please create a DSIP issue firstly. For more info of DSIP you can refer to #14102 |
|
Hi @ruanwenjun @SbloodyS We have created a new DolphinDB DSIP #16818 |
Purpose of the pull request
Add dolphindb datasource.
增加dolphindb数据源。