[GitHub] [doris] zhangguoqiang666 commented on a diff in pull request #17576: [Improvement](regresstion-test)Enhance regression strategy and Allows load and query to run simultaneously in different directories

["zhangguoqiang666 (via GitHub)" <gi...@apache.org>]

zhangguoqiang666 commented on code in PR #17576:
URL: https://github.com/apache/doris/pull/17576#discussion_r1130642368


##########
regression-test/framework/src/main/groovy/org/apache/doris/regression/RegressionTest.groovy:
##########
@@ -180,13 +180,43 @@ class RegressionTest {
         def recorder = new Recorder()
         def directoryFilter = config.getDirectoryFilter()
         if (!config.withOutLoadData) {
-            log.info('Start to run load scripts')
+            List<String> load_sources = new ArrayList<>()
+            List<String> other_sources = new ArrayList<>()
+            new File(config.suitePath).eachDir { dir ->
+                {
+                    def load_flag = 0
+                    dir.eachFileRecurse { f_01 ->
+                        if (f_01.name.contains("load")) {
+                            load_flag = 1
+                            load_sources.add(f_01.name)
+                        }

Review Comment:
   Do you mean in the following way?
   
   List<String> load_sources = new ArrayList<>()
   List<String> other_sources = new ArrayList<>()
   
   new File(config.suitePath).eachDir { dir ->
                   {
                       List<String> load_temp_sources = new ArrayList<>()
                       List<String> other_temp_sources = new ArrayList<>()
                       load_temp_sources.clear()
                       other_temp_sources.clear()
                       dir.eachFileRecurse { f ->
                           if (f.name.contains("load")) {
                               load_temp_sources.add(f.name)
                           }else{
                              other_temp_sources.add(f.name)
                           }
                       }
                       if (load_temp_sources) {
                          load_sources.addAll(load_temp_sources)
                          other_sources.addAll(other_temp_sources)
                       }else {
                          load_sources.addAll(other_temp_sources)
                       }
                   }
               }



-- 
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@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org