You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@helix.apache.org by GitBox <gi...@apache.org> on 2020/08/15 23:24:15 UTC

[GitHub] [helix] kaisun2000 opened a new issue #1278: Fix TestEnqueueJobs.testQueueJobsMaxCapacity

kaisun2000 opened a new issue #1278:
URL: https://github.com/apache/helix/issues/1278


   github log:
   >2020-08-14T23:50:43.0367342Z [ERROR]   TestClusterInMaintenanceModeWhenReachingMaxPartition.testDisableCluster:119 expected:<true> but was:<false>
   
   >2020-08-14T23:50:42.7831939Z [ERROR] testQueueJobsMaxCapacity(org.apache.helix.integration.task.TestEnqueueJobs)  Time elapsed: 302.201 s  <<< FAILURE!
   2020-08-14T23:50:42.7833487Z org.apache.helix.HelixException: Workflow "testQueueJobsMaxCapacity" context is null or job "testQueueJobsMaxCapacity_JOB2" is not in states: [COMPLETED]
   2020-08-14T23:50:42.7833641Z 	at org.apache.helix.integration.task.TestEnqueueJobs.testQueueJobsMaxCapacity(TestEnqueueJobs.java:205)


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org


[GitHub] [helix] kaisun2000 commented on issue #1278: Fix TestEnqueueJobs.testQueueJobsMaxCapacity

Posted by GitBox <gi...@apache.org>.
kaisun2000 commented on issue #1278:
URL: https://github.com/apache/helix/issues/1278#issuecomment-674458093


   ```
       for (int i = 0; i < numberOfJobsAddedInitially; i++) {
         _driver.pollForJobState(queueName, TaskUtil.getNamespacedJobName(queueName, "JOB" + i),
             TaskState.COMPLETED);
       }
   ```
   pollForJobState is following, timeout is 5 minutes. Enable to 15 minutes with test case time to 10 minute to have a try. Also, add more about if ctx is null or the other case.
   
   ```
      long st = System.currentTimeMillis();
       do {
         Thread.sleep(timeToSleep);
         ctx = getWorkflowContext(workflowName);
       } while ((ctx == null || ctx.getJobState(jobName) == null
           || !allowedStates.contains(ctx.getJobState(jobName)))
           && System.currentTimeMillis() < st + timeout);
   
       if (ctx == null || !allowedStates.contains(ctx.getJobState(jobName))) {
         throw new HelixException(
             String.format("Workflow \"%s\" context is null or job \"%s\" is not in states: %s",
                 workflowName, jobName, allowedStates));
       }
   ```


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org


[GitHub] [helix] jiajunwang commented on issue #1278: Fix TestEnqueueJobs.testQueueJobsMaxCapacity

Posted by GitBox <gi...@apache.org>.
jiajunwang commented on issue #1278:
URL: https://github.com/apache/helix/issues/1278#issuecomment-849103205


   Close test unstable tickets since we have an automatic tracking mechanism https://github.com/apache/helix/pull/1757 now for tracking the most recent test issues.


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org


[GitHub] [helix] kaisun2000 commented on issue #1278: Fix TestEnqueueJobs.testQueueJobsMaxCapacity

Posted by GitBox <gi...@apache.org>.
kaisun2000 commented on issue #1278:
URL: https://github.com/apache/helix/issues/1278#issuecomment-692922804


   #1346 


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org


[GitHub] [helix] jiajunwang closed issue #1278: Fix TestEnqueueJobs.testQueueJobsMaxCapacity

Posted by GitBox <gi...@apache.org>.
jiajunwang closed issue #1278:
URL: https://github.com/apache/helix/issues/1278


   


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org