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 2022/04/21 13:56:02 UTC

[GitHub] [dolphinscheduler] EricGao888 opened a new pull request, #9659: [Bug][Doc]Update database init instruction docs (#9598)

EricGao888 opened a new pull request, #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659

   ## Purpose of the pull request
   * Update database initialization instructions in docs
   
   ## Brief change log
   1. Add instructions on how to create db, user and grant permissions for dolphin users who use latest versions of mysql.
   2. Update instructions on how to init db since related directory structure has changed.
   
   ## Verify this pull request
   
   * Verified by manual test.
   


-- 
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

[GitHub] [dolphinscheduler] zhongjiajie commented on pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
zhongjiajie commented on PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#issuecomment-1106099322

   I find out keyword `create-dolphinscheduler.sh` also in https://raw.githubusercontent.com/apache/dolphinscheduler/9f84dbbda00956d5bf198ce7eb4c47f2ea335173/docs/docs/en/architecture/configuration.md . and the binary package structure already changed, I think you could change it too. But you could also just delete `<create|upgarde>-dolphinscheduler.sh` in this PR too


-- 
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

[GitHub] [dolphinscheduler] EricGao888 commented on pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
EricGao888 commented on PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#issuecomment-1106282578

   > I find out keyword `create-dolphinscheduler.sh` also in https://raw.githubusercontent.com/apache/dolphinscheduler/9f84dbbda00956d5bf198ce7eb4c47f2ea335173/docs/docs/en/architecture/configuration.md . and the binary package structure already changed, I think you could change it too. But you could also just delete `<create|upgarde>-dolphinscheduler.sh` in this PR too
   
   @SbloodyS @zhongjiajie  I've fixed `upgrade-schema.sh` and updated db upgrade related docs. I just checked the `configuration.md`, it seems that we need to update the whole `Directory Structure` section. I will submit a new PR to this issue: #9670 


-- 
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

[GitHub] [dolphinscheduler] SbloodyS commented on a diff in pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
SbloodyS commented on code in PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#discussion_r856042023


##########
docs/docs/zh/guide/upgrade.md:
##########
@@ -13,24 +13,17 @@
 - 以下升级操作都需要在新版本的目录进行
 
 ## 4. 数据库升级
-- 修改conf/datasource.properties中的下列属性
-
-- 如果选择 MySQL，请注释掉 PostgreSQL 相关配置(反之同理), 还需要手动添加 [[ mysql-connector-java 驱动 jar ](https://downloads.MySQL.com/archives/c-j/)] 包到 lib 目录下，这里下载的是mysql-connector-java-8.0.16.jar，然后正确配置数据库连接相关信息
-
-    ```properties
-      # postgre
-      #spring.datasource.driver-class-name=org.postgresql.Driver
-      #spring.datasource.url=jdbc:postgresql://localhost:5432/dolphinscheduler
-      # mysql
-      spring.datasource.driver-class-name=com.mysql.jdbc.Driver
-      spring.datasource.url=jdbc:mysql://xxx:3306/dolphinscheduler?useUnicode=true&characterEncoding=UTF-8&allowMultiQueries=true     需要修改ip，本机localhost即可
-      spring.datasource.username=xxx						需要修改为上面的{user}值
-      spring.datasource.password=xxx						需要修改为上面的{password}值
+- 将`./tools/conf/application.yaml`中的username和password改成你设定数据库用户名和密码

Review Comment:
   We should also modify the english version.



-- 
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

[GitHub] [dolphinscheduler] EricGao888 commented on a diff in pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
EricGao888 commented on code in PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#discussion_r856058525


##########
docs/docs/zh/guide/upgrade.md:
##########
@@ -13,24 +13,17 @@
 - 以下升级操作都需要在新版本的目录进行
 
 ## 4. 数据库升级
-- 修改conf/datasource.properties中的下列属性
-
-- 如果选择 MySQL，请注释掉 PostgreSQL 相关配置(反之同理), 还需要手动添加 [[ mysql-connector-java 驱动 jar ](https://downloads.MySQL.com/archives/c-j/)] 包到 lib 目录下，这里下载的是mysql-connector-java-8.0.16.jar，然后正确配置数据库连接相关信息
-
-    ```properties
-      # postgre
-      #spring.datasource.driver-class-name=org.postgresql.Driver
-      #spring.datasource.url=jdbc:postgresql://localhost:5432/dolphinscheduler
-      # mysql
-      spring.datasource.driver-class-name=com.mysql.jdbc.Driver
-      spring.datasource.url=jdbc:mysql://xxx:3306/dolphinscheduler?useUnicode=true&characterEncoding=UTF-8&allowMultiQueries=true     需要修改ip，本机localhost即可
-      spring.datasource.username=xxx						需要修改为上面的{user}值
-      spring.datasource.password=xxx						需要修改为上面的{password}值
+- 将`./tools/conf/application.yaml`中的username和password改成你设定数据库用户名和密码

Review Comment:
   I think I've already updated the English version? 
   ![image](https://user-images.githubusercontent.com/34905992/164687186-2d72a12d-feb6-4c38-8bac-493e1be283e2.png)
   



-- 
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

[GitHub] [dolphinscheduler] SbloodyS commented on a diff in pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
SbloodyS commented on code in PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#discussion_r856073335


##########
docs/docs/zh/guide/upgrade.md:
##########
@@ -13,24 +13,17 @@
 - 以下升级操作都需要在新版本的目录进行
 
 ## 4. 数据库升级
-- 修改conf/datasource.properties中的下列属性
-
-- 如果选择 MySQL，请注释掉 PostgreSQL 相关配置(反之同理), 还需要手动添加 [[ mysql-connector-java 驱动 jar ](https://downloads.MySQL.com/archives/c-j/)] 包到 lib 目录下，这里下载的是mysql-connector-java-8.0.16.jar，然后正确配置数据库连接相关信息
-
-    ```properties
-      # postgre
-      #spring.datasource.driver-class-name=org.postgresql.Driver
-      #spring.datasource.url=jdbc:postgresql://localhost:5432/dolphinscheduler
-      # mysql
-      spring.datasource.driver-class-name=com.mysql.jdbc.Driver
-      spring.datasource.url=jdbc:mysql://xxx:3306/dolphinscheduler?useUnicode=true&characterEncoding=UTF-8&allowMultiQueries=true     需要修改ip，本机localhost即可
-      spring.datasource.username=xxx						需要修改为上面的{user}值
-      spring.datasource.password=xxx						需要修改为上面的{password}值
+- 将`./tools/conf/application.yaml`中的username和password改成你设定数据库用户名和密码

Review Comment:
   Sorry. I missed it. @EricGao888 



-- 
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

[GitHub] [dolphinscheduler] EricGao888 commented on a diff in pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
EricGao888 commented on code in PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#discussion_r855766086


##########
docs/docs/en/guide/installation/pseudo-cluster.md:
##########
@@ -146,10 +146,29 @@ mysql> GRANT ALL PRIVILEGES ON dolphinscheduler.* TO '{user}'@'localhost' IDENTI
 mysql> flush privileges;
 ```
 
+The script above does not work for latest verisons (e.g. 8.0.27) of mysql, use the script below instead:

Review Comment:
   Sure, I will fix it.



-- 
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

[GitHub] [dolphinscheduler] SbloodyS commented on pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
SbloodyS commented on PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#issuecomment-1106082855

   May be we should modifiy this docs too. Because init script and upgrade script are similar. @EricGao888 
   
   https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/guide/upgrade.md


-- 
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

[GitHub] [dolphinscheduler] sonarcloud[bot] commented on pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
sonarcloud[bot] commented on PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#issuecomment-1106334025

   Kudos, SonarCloud Quality Gate passed!&nbsp; &nbsp; [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Quality Gate passed')](https://sonarcloud.io/dashboard?id=apache-dolphinscheduler&pullRequest=9659)
   
   [![Bug](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/bug-16px.png 'Bug')](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=BUG) [![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png 'A')](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=BUG) [0 Bugs](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=BUG)  
   [![Vulnerability](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/vulnerability-16px.png 'Vulnerability')](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=VULNERABILITY) [![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png 'A')](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=VULNERABILITY) [0 Vulnerabilities](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=VULNERABILITY)  
   [![Security Hotspot](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/security_hotspot-16px.png 'Security Hotspot')](https://sonarcloud.io/project/security_hotspots?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=SECURITY_HOTSPOT) [![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png 'A')](https://sonarcloud.io/project/security_hotspots?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=SECURITY_HOTSPOT) [0 Security Hotspots](https://sonarcloud.io/project/security_hotspots?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=SECURITY_HOTSPOT)  
   [![Code Smell](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/code_smell-16px.png 'Code Smell')](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=CODE_SMELL) [![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png 'A')](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=CODE_SMELL) [0 Code Smells](https://sonarcloud.io/project/issues?id=apache-dolphinscheduler&pullRequest=9659&resolved=false&types=CODE_SMELL)
   
   [![No Coverage information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/CoverageChart/NoCoverageInfo-16px.png 'No Coverage information')](https://sonarcloud.io/component_measures?id=apache-dolphinscheduler&pullRequest=9659&metric=coverage&view=list) No Coverage information  
   [![No Duplication information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/Duplications/NoDuplicationInfo-16px.png 'No Duplication information')](https://sonarcloud.io/component_measures?id=apache-dolphinscheduler&pullRequest=9659&metric=duplicated_lines_density&view=list) No Duplication information
   
   


-- 
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

[GitHub] [dolphinscheduler] zhongjiajie merged pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
zhongjiajie merged PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659


-- 
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

[GitHub] [dolphinscheduler] SbloodyS commented on a diff in pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
SbloodyS commented on code in PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#discussion_r855743768


##########
docs/docs/en/guide/installation/pseudo-cluster.md:
##########
@@ -146,10 +146,29 @@ mysql> GRANT ALL PRIVILEGES ON dolphinscheduler.* TO '{user}'@'localhost' IDENTI
 mysql> flush privileges;
 ```
 
+The script above does not work for latest verisons (e.g. 8.0.27) of mysql, use the script below instead:

Review Comment:
   I think we should not use this description. Just put ```Mysql 8``` in this area. And ```Mysql 5.6/5.7``` above.



-- 
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

[GitHub] [dolphinscheduler] EricGao888 commented on pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
EricGao888 commented on PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#issuecomment-1106085199

   > May be we should modifiy this docs too. Because init script and upgrade script are similar. @EricGao888
   > 
   > https://github.com/apache/dolphinscheduler/blob/dev/docs/docs/en/guide/upgrade.md
   
   Sure, I will update it.


-- 
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

[GitHub] [dolphinscheduler] EricGao888 commented on pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
EricGao888 commented on PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#issuecomment-1106104767

   > I find out keyword `create-dolphinscheduler.sh` also in https://raw.githubusercontent.com/apache/dolphinscheduler/9f84dbbda00956d5bf198ce7eb4c47f2ea335173/docs/docs/en/architecture/configuration.md . and the binary package structure already changed, I think you could change it too. But you could also just delete `<create|upgarde>-dolphinscheduler.sh` in this PR too
   
   @zhongjiajie Sure. I just figured out there's also a bug in upgrade-schema.sh caused by incorrect file path. I will also fix it in this PR. https://github.com/apache/dolphinscheduler/blob/1de7f154a38566f4d7989c8c7ebe8bda16a8dc6e/dolphinscheduler-tools/src/main/bin/upgrade-schema.sh#L22-27


-- 
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

[GitHub] [dolphinscheduler] codecov-commenter commented on pull request #9659: [Bug][Doc]Update database init instruction docs (#9598)

Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #9659:
URL: https://github.com/apache/dolphinscheduler/pull/9659#issuecomment-1106138346

   # [Codecov](https://codecov.io/gh/apache/dolphinscheduler/pull/9659?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 [#9659](https://codecov.io/gh/apache/dolphinscheduler/pull/9659?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (1de7f15) into [dev](https://codecov.io/gh/apache/dolphinscheduler/commit/7b1c316d9e436e895fc2fd3cb133bcbe58d9f879?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (7b1c316) will **increase** coverage by `0.08%`.
   > The diff coverage is `51.11%`.
   
   > :exclamation: Current head 1de7f15 differs from pull request most recent head 52c22f8. Consider uploading reports for the commit 52c22f8 to get more accurate results
   
   ```diff
   @@             Coverage Diff              @@
   ##                dev    #9659      +/-   ##
   ============================================
   + Coverage     39.92%   40.01%   +0.08%     
   - Complexity     4463     4475      +12     
   ============================================
     Files           835      835              
     Lines         33541    33549       +8     
     Branches       3707     3709       +2     
   ============================================
   + Hits          13391    13424      +33     
   + Misses        18906    18877      -29     
   - Partials       1244     1248       +4     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/dolphinscheduler/pull/9659?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [...hinscheduler/common/config/StoreConfiguration.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvY29tbW9uL2NvbmZpZy9TdG9yZUNvbmZpZ3VyYXRpb24uamF2YQ==) | `0.00% <ø> (ø)` | |
   | [...che/dolphinscheduler/common/utils/HadoopUtils.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvY29tbW9uL3V0aWxzL0hhZG9vcFV0aWxzLmphdmE=) | `30.39% <ø> (ø)` | |
   | [...e/dolphinscheduler/common/utils/PropertyUtils.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvY29tbW9uL3V0aWxzL1Byb3BlcnR5VXRpbHMuamF2YQ==) | `37.34% <0.00%> (+1.20%)` | :arrow_up: |
   | [.../apache/dolphinscheduler/common/utils/S3Utils.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvY29tbW9uL3V0aWxzL1MzVXRpbHMuamF2YQ==) | `0.00% <ø> (ø)` | |
   | [...duler/plugin/task/api/AbstractCommandExecutor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci10YXNrLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLXRhc2stYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi90YXNrL2FwaS9BYnN0cmFjdENvbW1hbmRFeGVjdXRvci5qYXZh) | `0.00% <0.00%> (ø)` | |
   | [...olphinscheduler/plugin/task/api/TaskConstants.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci10YXNrLXBsdWdpbi9kb2xwaGluc2NoZWR1bGVyLXRhc2stYXBpL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9kb2xwaGluc2NoZWR1bGVyL3BsdWdpbi90YXNrL2FwaS9UYXNrQ29uc3RhbnRzLmphdmE=) | `0.00% <ø> (ø)` | |
   | [.../server/worker/processor/TaskExecuteProcessor.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci13b3JrZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvc2VydmVyL3dvcmtlci9wcm9jZXNzb3IvVGFza0V4ZWN1dGVQcm9jZXNzb3IuamF2YQ==) | `0.00% <0.00%> (ø)` | |
   | [...heduler/api/service/impl/ResourcesServiceImpl.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci1hcGkvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvYXBpL3NlcnZpY2UvaW1wbC9SZXNvdXJjZXNTZXJ2aWNlSW1wbC5qYXZh) | `52.68% <20.00%> (-0.09%)` | :arrow_down: |
   | [...eduler/server/worker/runner/TaskExecuteThread.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci13b3JrZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvc2VydmVyL3dvcmtlci9ydW5uZXIvVGFza0V4ZWN1dGVUaHJlYWQuamF2YQ==) | `12.58% <44.44%> (+12.58%)` | :arrow_up: |
   | [...exception/StorageOperateNoConfiguredException.java](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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-ZG9scGhpbnNjaGVkdWxlci1jb21tb24vc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2RvbHBoaW5zY2hlZHVsZXIvY29tbW9uL2V4Y2VwdGlvbi9TdG9yYWdlT3BlcmF0ZU5vQ29uZmlndXJlZEV4Y2VwdGlvbi5qYXZh) | `100.00% <100.00%> (ø)` | |
   | ... and [5 more](https://codecov.io/gh/apache/dolphinscheduler/pull/9659/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/9659?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/9659?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 [7b1c316...52c22f8](https://codecov.io/gh/apache/dolphinscheduler/pull/9659?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