Skip to content

focus logging eventLog creation on local usage#316

Merged
kevin-mcgoldrick merged 1 commit intomasterfrom
logging_local
Apr 3, 2024
Merged

focus logging eventLog creation on local usage#316
kevin-mcgoldrick merged 1 commit intomasterfrom
logging_local

Conversation

@kevin-mcgoldrick
Copy link
Collaborator

focus logging eventLog creation on local usage

Agent debugger takes a very long time to prepare a script to run. The majority of that time is just setting up the logevent. The PR reduces that time.

Best measure with the TEST APITestHarnessTest.testRunConcurrentTestPlan()

before: 22 sec (46 sec on work network)
after: 10 sec

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@kevin-mcgoldrick kevin-mcgoldrick merged commit 710dfe8 into master Apr 3, 2024
@kevin-mcgoldrick kevin-mcgoldrick deleted the logging_local branch April 3, 2024 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants