hiero-ledger / hiero-ledger/hiero-consensus-node
Unify HandleException and PreCheckException
Open
HapiThursday
- Dominant language
- Java
- Stars
- 407
- Forks
- 226
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 210
Description
As it turned out, having two distinct exceptions, `HandleException` and `PreCheckException`, does not really provide any value. On the contrary, it makes some code more complex because both Exceptions can occur. Therefore, we should unify them.
Contributor guide
Research direction
Locate the definitions and usages of HandleException and PreCheckException, then trace the code paths where either exception can occur. Consolidate the exception handling and run the relevant existing tests; done means callers no longer need to account for two distinct exceptions and the test suite passes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100