Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

AKI-603 Redesigned AionPendingStateImpl and Transaction pool #1120

Merged
merged 23 commits into from
Feb 28, 2020
Merged

AKI-603 Redesigned AionPendingStateImpl and Transaction pool #1120

merged 23 commits into from
Feb 28, 2020

Conversation

AionJayT
Copy link
Collaborator

Description

Fixes Issue # .

Type of change

  • Bug fix.
  • New feature.
  • Enhancement.
  • Unit test.
  • Breaking change (a fix or feature that causes existing functionality to not work as expected).
  • Requires documentation update.

Testing

@AionJayT AionJayT added bug Something isn't working enhancement New feature or request labels Feb 24, 2020
@AionJayT AionJayT added this to the 1.4 milestone Feb 24, 2020
@AionJayT AionJayT self-assigned this Feb 24, 2020
AionJayT
AionJayT commented Feb 25, 2020
View reviewed changes
modTxPool/src/org/aion/txpool/v1/TxPoolV1.java Outdated Show resolved Hide resolved
modTxPool/src/org/aion/txpool/v1/TxPoolV1.java Outdated Show resolved Hide resolved
modTxPool/src/org/aion/txpool/v1/TxPoolV1.java Outdated Show resolved Hide resolved
modTxPool/src/org/aion/txpool/v1/TxPoolV1.java Outdated Show resolved Hide resolved
modTxPool/src/org/aion/txpool/v1/TxPoolV1.java Outdated Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/v1/PendingTxCacheV1.java Outdated Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/v1/PendingTxCacheV1.java Outdated Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/v1/PendingTxCacheV1.java Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/v1/PendingTxCacheV1.java Outdated Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/v1/PendingTxCacheV1.java Outdated Show resolved Hide resolved
@AionJayT AionJayT added the wip Indicates that a PR or issue is Work In Progress, issuer should be notified before any actions label Feb 26, 2020
AlexandraRoatis
AlexandraRoatis reviewed Feb 26, 2020
View reviewed changes
modAionImpl/src/org/aion/zero/impl/pendingState/AionPendingStateImpl.java Outdated Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/AionPendingStateImpl.java Outdated Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/AionPendingStateImpl.java Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/AionPendingStateImpl.java Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/AionPendingStateImpl.java Outdated Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/AionPendingStateImpl.java Outdated Show resolved Hide resolved
modAionImpl/src/org/aion/zero/impl/pendingState/AionPendingStateImpl.java Outdated Show resolved Hide resolved
modApiServer/src/org/aion/api/server/ApiTxResponse.java Outdated Show resolved Hide resolved
@AionJayT
AKI-605 TxPoolV1 implementation
b56ccd7
@AionJayT
AKI-605 added tests for TxPoolV1
9a4f608
@AionJayT
AKI-660 implemented pendingTxCacheV1
b163903
@AionJayT
AKI-660 added tests for PendingTxCacheV1
be1e3cc
@AionJayT
AKI-604 revised transaction life cycle in the AionPendingState with p…
9ffb03b 
…endingTxCacheV1 and TxPoolV1
@AionJayT
revised PendingTxCacheV1 methods
7975018
@AionJayT
move magic numbers and the property define to the Constant class
83caa53
@AionJayT
revised the poolBackup recovery in the pendingStateImpl
ab4b61f
@AionJayT
revised cachePoolBackup feature
752ca8c
@AionJayT
removed&refactored the transaction backup methods in the repository l…
aaa2f7c 
…ayers
@AionJayT
refactor the TxResponse
baf60f8
@AionJayT
remove unnecessary check and the revised Exception message
b8d3590
@AionJayT
Added TX and TXPOOL default settings in the config file
368c139
@AionJayT
refactoring addPendingTransactions to small code pieces
0651ff3
@AionJayT
revised blockEnergyLimit object in TxPoolV1 and exception message
ba54185
@AionJayT
Added more unit tests in TxPoolV1
5896fc0
@AionJayT
Added more unit tests in PendingTxCacheV1
4ccfec5
@AionJayT
removed unused argument in the AionPendingStateImpl
8716902
@AionJayT AionJayT removed the wip Indicates that a PR or issue is Work In Progress, issuer should be notified before any actions label Feb 28, 2020
@AionJayT AionJayT merged commit bd727f1 into master Feb 28, 2020
@AionJayT AionJayT deleted the AKI-603 branch February 28, 2020 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlexandraRoatis AlexandraRoatis AlexandraRoatis left review comments

Assignees

@AionJayT AionJayT

Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
1.4
Development

Successfully merging this pull request may close these issues.
2 participants
@AionJayT @AlexandraRoatis