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/06/07 01:29:45 UTC

[GitHub] [dolphinscheduler] CalvinKirs commented on a change in pull request #5562: [Feature-#3961][Registry]Registry-SPI

CalvinKirs commented on a change in pull request #5562:
URL: https://github.com/apache/dolphinscheduler/pull/5562#discussion_r646220918



##########
File path: dolphinscheduler-alert-plugin/dolphinscheduler-alert-dingtalk/pom.xml
##########
@@ -21,7 +21,7 @@
     <parent>
         <artifactId>dolphinscheduler-alert-plugin</artifactId>
         <groupId>org.apache.dolphinscheduler</groupId>
-        <version>${revision}</version>
+        <version>1.3.6-SNAPSHOT</version>

Review comment:
       The original method will cause the release plug-in to not work normally, you can try it.
   If you must do this, you must introduce the flatten plug-in, but I think it is completely unnecessary to introduce a new plug-in.




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

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