Skip to content

feat: Introduce metaclass to convert JavaObject and refactor datasource#75

Merged
zhongjiajie merged 4 commits into
apache:mainfrom
zhongjiajie:refactor-modle-datasource
Feb 27, 2023
Merged

feat: Introduce metaclass to convert JavaObject and refactor datasource#75
zhongjiajie merged 4 commits into
apache:mainfrom
zhongjiajie:refactor-modle-datasource

Conversation

@zhongjiajie

Copy link
Copy Markdown
Member
  • Introduce metaclass to convert JavaObject to python easier
  • Add more detail for datasource including host, port, schema etc
  • Change else using datasource

Brief Summary of The Change

Pull Request checklist

I confirm that the following checklist has been completed.

  • Add/Change test cases for the changes.
  • Add/Change the related documentation, should also change docs/source/config.rst when you change file default_config.yaml.
  • (Optional) Add your change to UPDATING.md when it is an incompatible change.

* Introduce metaclass to convert JavaObject to python easier
* Add more detail for datasource including host, port, schema etc
* Change else using datasource
@zhongjiajie zhongjiajie added enhancement New feature or request dep-main-repo Dependent on repository apache/dolphinscheduler labels Feb 24, 2023
zhongjiajie added a commit to zhongjiajie/dolphinscheduler that referenced this pull request Feb 24, 2023
@codecov-commenter

codecov-commenter commented Feb 24, 2023

Copy link
Copy Markdown

Codecov Report

Merging #75 (06ece1f) into main (428d717) will decrease coverage by 0.19%.
The diff coverage is 84.96%.

@@            Coverage Diff             @@
##             main      #75      +/-   ##
==========================================
- Coverage   90.93%   90.75%   -0.19%     
==========================================
  Files          61       63       +2     
  Lines        2218     2314      +96     
==========================================
+ Hits         2017     2100      +83     
- Misses        201      214      +13     
Flag Coverage Δ
unittests 90.75% <84.96%> (-0.19%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/pydolphinscheduler/core/__init__.py 100.00% <ø> (ø)
src/pydolphinscheduler/models/meta.py 66.66% <66.66%> (ø)
src/pydolphinscheduler/models/datasource.py 85.96% <85.96%> (ø)
src/pydolphinscheduler/constants.py 100.00% <100.00%> (ø)
src/pydolphinscheduler/models/connection.py 100.00% <100.00%> (ø)
src/pydolphinscheduler/tasks/datax.py 100.00% <100.00%> (ø)
src/pydolphinscheduler/tasks/procedure.py 100.00% <100.00%> (ø)
src/pydolphinscheduler/tasks/sql.py 100.00% <100.00%> (ø)
src/pydolphinscheduler/core/workflow.py 81.76% <0.00%> (+0.30%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@zhongjiajie zhongjiajie marked this pull request as ready for review February 27, 2023 06:39
@zhongjiajie zhongjiajie merged commit 7baac8f into apache:main Feb 27, 2023
@zhongjiajie zhongjiajie deleted the refactor-modle-datasource branch February 27, 2023 06:47
@zhongjiajie

Copy link
Copy Markdown
Member Author

ref: apache/dolphinscheduler#13616

@zhongjiajie

Copy link
Copy Markdown
Member Author

ref: apache/dolphinscheduler#13616

may release in version apache/dolphinscheduler:3.1.5

@zhongjiajie zhongjiajie removed the dep-main-repo Dependent on repository apache/dolphinscheduler label Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants