You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2021/11/12 07:25:46 UTC

[GitHub] [dolphinscheduler] codecov-commenter commented on pull request #6786: [Improvement][Datasource]refactor datasource parsing other parameter

codecov-commenter commented on pull request #6786:
URL: https://github.com/apache/dolphinscheduler/pull/6786#issuecomment-966882794


   # [Codecov](https://codecov.io/gh/apache/dolphinscheduler/pull/6786?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#6786](https://codecov.io/gh/apache/dolphinscheduler/pull/6786?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (fcafc4b) into [dev](https://codecov.io/gh/apache/dolphinscheduler/commit/5741c758b3521885e4ea11fdcb3c66d275537739?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (5741c75) will **increase** coverage by `0.13%`.
   > The diff coverage is `76.08%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/graphs/tree.svg?width=650&height=150&src=pr&token=bv9iXXRLi9&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/dolphinscheduler/pull/6786?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   ```diff
   @@             Coverage Diff              @@
   ##                dev    #6786      +/-   ##
   ============================================
   + Coverage     41.80%   41.93%   +0.13%     
   + Complexity     3616     3613       -3     
   ============================================
     Files           641      641              
     Lines         25890    25814      -76     
     Branches       2795     2796       +1     
   ============================================
   + Hits          10824    10826       +2     
   + Misses        14087    14014      -73     
   + Partials        979      974       -5     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/dolphinscheduler/pull/6786?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [...g/apache/dolphinscheduler/spi/utils/Constants.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1zcGkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvc3BpL3V0aWxzL0NvbnN0YW50cy5qYXZh) | `0.00% <ø> (ø)` | |
   | [...urce/clickhouse/ClickHouseDatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL2NsaWNraG91c2UvQ2xpY2tIb3VzZURhdGFzb3VyY2VQcm9jZXNzb3IuamF2YQ==) | `60.52% <50.00%> (+8.52%)` | :arrow_up: |
   | [...rce/api/datasource/db2/Db2DatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL2RiMi9EYjJEYXRhc291cmNlUHJvY2Vzc29yLmphdmE=) | `56.75% <50.00%> (+5.73%)` | :arrow_up: |
   | [...e/api/datasource/hive/HiveDatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL2hpdmUvSGl2ZURhdGFzb3VyY2VQcm9jZXNzb3IuamF2YQ==) | `38.88% <50.00%> (+1.98%)` | :arrow_up: |
   | [...i/datasource/oracle/OracleDatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL29yYWNsZS9PcmFjbGVEYXRhc291cmNlUHJvY2Vzc29yLmphdmE=) | `51.11% <50.00%> (+5.49%)` | :arrow_up: |
   | [...urce/postgresql/PostgreSqlDatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL3Bvc3RncmVzcWwvUG9zdGdyZVNxbERhdGFzb3VyY2VQcm9jZXNzb3IuamF2YQ==) | `55.26% <50.00%> (+7.26%)` | :arrow_up: |
   | [...i/datasource/presto/PrestoDatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL3ByZXN0by9QcmVzdG9EYXRhc291cmNlUHJvY2Vzc29yLmphdmE=) | `56.75% <50.00%> (+7.77%)` | :arrow_up: |
   | [...api/datasource/spark/SparkDatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL3NwYXJrL1NwYXJrRGF0YXNvdXJjZVByb2Nlc3Nvci5qYXZh) | `44.64% <50.00%> (+3.46%)` | :arrow_up: |
   | [...source/sqlserver/SqlServerDatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL3NxbHNlcnZlci9TcWxTZXJ2ZXJEYXRhc291cmNlUHJvY2Vzc29yLmphdmE=) | `54.05% <50.00%> (+6.13%)` | :arrow_up: |
   | [...ce/api/datasource/AbstractDatasourceProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-ZG9scGhpbnNjaGVkdWxlci1kYXRhc291cmNlLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLWRhdGFzb3VyY2UtYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi9kYXRhc291cmNlL2FwaS9kYXRhc291cmNlL0Fic3RyYWN0RGF0YXNvdXJjZVByb2Nlc3Nvci5qYXZh) | `77.08% <88.00%> (+11.86%)` | :arrow_up: |
   | ... and [3 more](https://codecov.io/gh/apache/dolphinscheduler/pull/6786/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/dolphinscheduler/pull/6786?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/dolphinscheduler/pull/6786?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [5741c75...fcafc4b](https://codecov.io/gh/apache/dolphinscheduler/pull/6786?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org