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/07/30 13:47:33 UTC

[GitHub] [dolphinscheduler] zhuxt2015 commented on a diff in pull request #10621: [bug][Api]Appending the copy or import suffix indefinitely causes the name to exceed the lenght limit

zhuxt2015 commented on code in PR #10621:
URL: https://github.com/apache/dolphinscheduler/pull/10621#discussion_r933808618


##########
dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ProcessDefinitionServiceImpl.java:
##########
@@ -1876,6 +1878,27 @@ protected void doBatchOperateProcessDefinition(User loginUser,
         }
     }
 
+    /**
+     * get new Task name or Process name when copy or import operate
+     * @param originalName Task or Process original name
+     * @param suffix "_copy_" or "_import_"
+     * @return
+     */
+    private String getNewName(String originalName, String suffix) {

Review Comment:
   use regex match



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