Skip to content

feat: add pathReconstructionTotalTime to control the entire Path Reconsturction phase (an additional time-control way)#752

Merged
StevenArzt merged 1 commit intosecure-software-engineering:developfrom
Westwood-Lin:main
Jul 25, 2024
Merged

feat: add pathReconstructionTotalTime to control the entire Path Reconsturction phase (an additional time-control way)#752
StevenArzt merged 1 commit intosecure-software-engineering:developfrom
Westwood-Lin:main

Conversation

@Westwood-Lin
Copy link
Contributor

Commit Message:

  • The field pathReconstructionTimeout only controls the time for reconstructing a specific path, not the entire phase.
  • Added a pathReconstructionTotalTime field to the PathConfiguration to control the total time for the path reconstruction process.
  • Added corresponding control logic in BatchPathBuilder.java, ConcurrentAbstractionPathBuilder.java, and RecursivePathBuilder.java. These code pieces are written with comments.
  • Also supplemented the logic for the new field in XMLConstants and XMLConfigurationWriter.java for conceptual integrity.
pr_code_tested_snipaste

…consturction phase (an additional time-control way)

Commit Message:
- The pathReconstructionTimeout only controls the time for reconstructing a specific path, not the entire phase.
- Added a `pathReconstructionTotalTime` field to the `PathConfiguration` to control the total time for the path reconstruction process.
- Added corresponding control logic in `BatchPathBuilder.java`, `ConcurrentAbstractionPathBuilder.java`, and `RecursivePathBuilder.java`.
- Also supplemented the logic for the new field in `XMLConstants` and `XMLConfigurationWriter.java` for conceptual integrity.
@StevenArzt
Copy link
Member

Thanks for spotting the issue. Your extension makes a lot of sense.

@StevenArzt StevenArzt merged commit 496b7f3 into secure-software-engineering:develop Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants