dolphinscheduler icon indicating copy to clipboard operation
dolphinscheduler copied to clipboard

Published 20 hours ago • verified publisher icon apache

[DSIP-83][datasource-plugin] Add DolphinDB datasource

Open eco8848 opened this issue 10 months ago • 7 comments

Purpose of the pull request

  • This pull request adds DolphinDB datasource.

close #16818

Brief change log

  • Add DolphinDB datasource module.
  • Add DolphinDB documentation.
  • Add DolphinDB constant definitions, connection parameters, data source processors and other components.
  • Add DolphinDB JDBC driver dependency.

Verify this pull request

This change added tests and can be verified as follows:

  • Add DataSourceUtilsTest class to test the datasource utility methods.
  • Add DolphinDBDataSourceChannelFactoryTest class to verify the creation methods of the datasource channel factory.
  • Add DolphinDBDataSourceChannelTest class to validate the creation of the datasource client through the channel.
  • Add DolphinDBDataSourceProcessorTest class to test various functionalities of the datasource processor.

eco8848 avatar Jan 15 '25 06:01 eco8848

Thanks for opening this pull request! Please check out our contributing guidelines. (https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/contribute/join/pull-request.md)

boring-cyborg[bot] avatar Jan 15 '25 06:01 boring-cyborg[bot]

Related DSIP https://github.com/apache/dolphinscheduler/issues/16818

eco8848 avatar Jan 15 '25 06:01 eco8848

Please fix CI errors. @eco8848

SbloodyS avatar Feb 04 '25 09:02 SbloodyS

I have referred to the official license requirements at https://dolphinscheduler.apache.org/zh-cn/docs/3.2.2/contribute/join/DS-License. It is necessary to add a description of the newly introduced third-party library in tools/dependencies/known-dependencies.txt, but it seems to conflict with https://github.com/apache/dolphinscheduler/issues/16838. What should I do now to comply with the requirements? @SbloodyS

eco8848 avatar Feb 11 '25 03:02 eco8848

The CI errors have already been fixed. @SbloodyS

eco8848 avatar Feb 13 '25 01:02 eco8848

I saw that there is a new check requires a milestone label. So is there a general plan for the integration of this PR? @SbloodyS

eco8848 avatar Feb 28 '25 08:02 eco8848

Quality Gate Passed Quality Gate passed

Issues
30 New issues
0 Accepted issues

Measures
0 Security Hotspots
64.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

sonarqubecloud[bot] avatar Mar 17 '25 09:03 sonarqubecloud[bot]

Closing for inactivity.

SbloodyS avatar Aug 06 '25 01:08 SbloodyS