Laffini / Laffini/Java-Matching-Engine-Core
Questions about order modification
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 23
- Forks
- 15
- PR merge metrics
- No merged PRs in 30d
Description
Hi Laffini. When examining the Order class code in the core, I noticed that the side attribute was not marked as final. Upon further investigation, I found that the change order interface provides the ability to modify the side. However, generally speaking, exchanges do not allow modifications to the side. Therefore, I am somewhat puzzled about this. @Laffini
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the Order class in the core and the change order interface mentioned in the issue. Determine whether changing the side is intended for this matching engine and, if not, define the required design change and its completion criteria. The issue is currently a question and names no tests.
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
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100