jvsingh opened a new pull request, #14008:
URL: https://github.com/apache/camel/pull/14008

   # Description
   
   Test KafkaBreakOnFirstErrorSeekIssueIT consistently took over 460 seconds 
(most other integration tests in camel-kafka took around 50 seconds on an 
average). 
   (These tests have a dependency on docker, so if its test images are not 
present on the build machine, the first time these are downloaded in about 200 
seconds. i.e. the first time this test took 700 seconds to execute. But 
subsequent runs took 460 secons.)
   In this change, I removed the unnecessary loop to push messages to partition 
1. Each of 7 iterations of this loop was taking 60 seconds
   
   Note that times reported above are for my development machine and could vary 
for other build machines. However, the proportional reduction in test execution 
time should be similar (around 80% reduction in execution time for repeat tests 
when docker images are already present locally) 
   
   
   # Target
   
   - [ ] I checked that the commit is targeting the correct branch (note that 
Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch)
   
   # Tracking
   - [ ] If this is a large change, bug fix, or code improvement, I checked 
there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for 
the change (usually before you start working on it).
   
   <!--
   # *Note*: trivial changes like, typos, minor documentation fixes and other 
small items do not require a JIRA issue. In this case your pull request should 
address just this issue, without pulling in other changes.
   -->
   
   # Apache Camel coding standards and style
   
   - [ ] I checked that each commit in the pull request has a meaningful 
subject line and body.
   
   <!--
   If you're unsure, you can format the pull request title like `[CAMEL-XXX] 
Fixes bug in camel-file component`, where you replace `CAMEL-XXX` with the 
appropriate JIRA issue.
   -->
   
   - [ ] I have run `mvn clean install -DskipTests` locally and I have 
committed all auto-generated changes
   
   <!--
   You can run the aforementioned command in your module so that the build 
auto-formats your code. This will also be verified as part of the checks and 
your PR may be rejected if if there are uncommited changes after running `mvn 
clean install -DskipTests`.
   
   You can learn more about the contribution guidelines at 
https://github.com/apache/camel/blob/main/CONTRIBUTING.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-unsubscr...@camel.apache.org

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

Reply via email to