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

chore: trigger CI #16752

Closed
wants to merge 125 commits into from
Closed

chore: trigger CI #16752

wants to merge 125 commits into from

Conversation

tinker-michaelj
Copy link
Collaborator

Description:

  • TBD

JivkoKelchev and others added 30 commits October 16, 2024 10:13
@JivkoKelchev
enable long term schedule transactions
2149b34 
Signed-off-by: Zhivko Kelchev <[email protected]>
@vtronkov
nit: add a simple comment
25fefc0 
Signed-off-by: Valentin Tronkov <[email protected]>
@ibankov
fix schedule sign longterm
6d70cfc 
Signed-off-by: ibankov <[email protected]>
@ibankov
additional tests
8a46546 
Signed-off-by: ibankov <[email protected]>
@JivkoKelchev
Merge branch 'refs/heads/develop' into 15055-POC-long-term-scheduled-…
0bbe343 
…transactions

# Conflicts:
#	hedera-node/hedera-app/src/main/java/com/hedera/node/app/workflows/handle/HandleWorkflow.java
@JivkoKelchev
fix merge conflicts
ae99cba 
Signed-off-by: Zhivko Kelchev <[email protected]>
@JivkoKelchev
revert SchedulingConfig
a560341 
refactor hapi tests

Signed-off-by: Zhivko Kelchev <[email protected]>
@tinker-michaelj
Add signingCryptoKeys() to AppKeyVerifier
99060db 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
spotless
0cc535d 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Standarize some stylistic things
a0e1ad6 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
spotless
8782b71 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
fix unit test
af02776 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Merge upstream
cf39c77 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
fix checkModuleInfo
89d550c 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
fix unit test
68149f8 
Signed-off-by: Michael Tinker <[email protected]>
@ibankov
LongTerm feature flag tests, and fixes
be2d792 
Signed-off-by: ibankov <[email protected]>
@JivkoKelchev
fix test
753e11c 
Signed-off-by: Zhivko Kelchev <[email protected]>
@JivkoKelchev
fix unit test
cffc354 
Signed-off-by: Zhivko Kelchev <[email protected]>
@JivkoKelchev
fix unit test
064d33e 
Signed-off-by: Zhivko Kelchev <[email protected]>
@ibankov
fix unit test ScheduleCreateHandlerTest
1cf47a8 
Signed-off-by: ibankov <[email protected]>
@ibankov
longer time to expiry for ScheduleLongTermExecutionSpecs
3299b2a 
Signed-off-by: ibankov <[email protected]>
@JivkoKelchev
fix fake upgrade error.
e0d80b6 
fix long term schedule throttle warning.

Signed-off-by: Zhivko Kelchev <[email protected]>
@JivkoKelchev
Merge remote-tracking branch 'refs/remotes/origin/16044-simplify-sig-…
251cc69 
…persistence' into 15055-POC-long-term-scheduled-transactions

# Conflicts:
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/HandlerUtility.java
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleCreateHandler.java
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleDeleteHandler.java
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleManager.java
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleSignHandler.java
#	hedera-node/hedera-schedule-service-impl/src/test/java/com/hedera/node/app/service/schedule/impl/handlers/HandlerUtilityTest.java
#	hedera-node/hedera-schedule-service-impl/src/test/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleCreateHandlerTest.java
#	hedera-node/hedera-schedule-service-impl/src/test/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleManagerTest.java
@JivkoKelchev
fix merge conflicts
b64f9d0 
Signed-off-by: Zhivko Kelchev <[email protected]>
@ibankov
spotless and add fees for schedule_create_contract_call
b43b9a7 
Signed-off-by: ibankov <[email protected]>
@ibankov
port ScheduleLongTermSignSuite from mono
913b3e8 
Signed-off-by: ibankov <[email protected]>
@JivkoKelchev
Merge branch 'refs/heads/develop' into 15055-POC-long-term-scheduled-…
3e96af2 
…transactions

# Conflicts:
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/AbstractScheduleHandler.java
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/HandlerUtility.java
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleCreateHandler.java
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleDeleteHandler.java
#	hedera-node/hedera-schedule-service-impl/src/main/java/com/hedera/node/app/service/schedule/impl/handlers/ScheduleSignHandler.java
@JivkoKelchev
Merge branch 'refs/heads/develop' into 15055-POC-long-term-scheduled-…
4b54916 
…transactions

# Conflicts:
#	hedera-node/hedera-app/src/test/java/com/hedera/node/app/workflows/handle/HandleWorkflowTest.java
#	hedera-node/test-clients/src/main/java/com/hedera/services/bdd/spec/queries/schedule/HapiGetScheduleInfo.java
@JivkoKelchev
revert unnecessary changes
48cc80f 
Signed-off-by: Zhivko Kelchev <[email protected]>
@JivkoKelchev
revert unnecessary changes
2730cec 
Signed-off-by: Zhivko Kelchev <[email protected]>
@codecov Codecov
Copy link

codecov bot commented Nov 22, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 83.33333% with 88 lines in your changes missing coverage. Please review.

Project coverage is 63.59%. Comparing base (11f5f17) to head (7923878).

Files with missing lines Patch % Lines
...dera/node/app/workflows/handle/HandleWorkflow.java 78.37% 15 Missing and 9 partials ⚠️
...rvice/schedule/impl/WritableScheduleStoreImpl.java 79.68% 12 Missing and 1 partial ⚠️
...ra/node/app/blocks/impl/KVStateChangeListener.java 0.00% 12 Missing ⚠️
.../hedera/node/app/throttle/ThrottleAccumulator.java 76.00% 8 Missing and 4 partials ⚠️
...app/service/schedule/impl/ScheduleServiceImpl.java 87.01% 5 Missing and 5 partials ⚠️
...ice/token/impl/handlers/FinalizeRecordHandler.java 14.28% 6 Missing ⚠️
...-app/src/main/java/com/hedera/node/app/Hedera.java 78.57% 2 Missing and 1 partial ⚠️
...app/workflows/standalone/TransactionExecutors.java 76.92% 3 Missing ⚠️
.../node/app/workflows/prehandle/PreHandleResult.java 33.33% 1 Missing and 1 partial ⚠️
...ce/schedule/impl/handlers/ScheduleSignHandler.java 60.00% 0 Missing and 2 partials ⚠️
... and 1 more
Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             develop   #16752      +/-   ##
=============================================
+ Coverage      63.45%   63.59%   +0.14%     
- Complexity     20313    20401      +88     
=============================================
  Files           2545     2547       +2     
  Lines          94604    94885     +281     
  Branches        9878     9916      +38     
=============================================
+ Hits           60027    60342     +315     
+ Misses         30958    30923      -35     
- Partials        3619     3620       +1
Files with missing lines Coverage Δ
.../utils/sysfiles/domain/throttling/ScaleFactor.java 100.00% <100.00%> (ø)
...ce/addressbook/impl/helpers/AddressBookHelper.java 55.17% <ø> (ø)
.../main/java/com/hedera/node/app/spi/AppContext.java 33.33% <ø> (ø)
...com/hedera/node/app/blocks/BlockStreamManager.java 100.00% <ø> (ø)
...om/hedera/node/app/blocks/impl/BlockImplUtils.java 79.46% <100.00%> (+0.37%) ⬆️
...edera/node/app/blocks/impl/BlockStreamBuilder.java 73.17% <ø> (+3.98%) ⬆️
...a/node/app/blocks/impl/BlockStreamManagerImpl.java 94.88% <100.00%> (+0.10%) ⬆️
...e/app/blocks/impl/BoundaryStateChangeListener.java 63.95% <100.00%> (+1.30%) ⬆️
...ode/app/blocks/schemas/V0560BlockStreamSchema.java 100.00% <100.00%> (ø)
...a/com/hedera/node/app/services/AppContextImpl.java 100.00% <ø> (ø)
... and 30 more

... and 30 files with indirect coverage changes

Impacted file tree graph

---- 🚨 Try these New Features:

JivkoKelchev and others added 25 commits November 22, 2024 17:46
@JivkoKelchev
fix: Dispatch only executable schedules (#16739)
edf7cb3 
Signed-off-by: Zhivko Kelchev <[email protected]>
@tinker-michaelj
intermezzo
219e2ad 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
intermezzo
01a984d 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Add RepeatableHip423Tests
c243460 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Add maxExecutionsPerUserTxn
e5d5b13 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
intermezzo
90f901c 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
fix unit test
b881367 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Add executionPurgesScheduleStateAsWhenRunningOutOfConsensusTimes()
81132b5 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
intermezzo
a2ee990 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
intermezzo
8428f21 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
fix unit test
a5bc487 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Merge upstream
cee359c 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Merge branch 'develop' into 15055-POC-long-term-scheduled-transactions
dac374c
@tinker-michaelj
Merge branch '15055-POC-long-term-scheduled-transactions' into hip-42…
2412fd6 
…3-loose-ends
@tinker-michaelj
intermezzo
7069a1b 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
intermezzo
24c5d61 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Add BlockStreamInfo#lastHandleTime
9ca2c23 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Merge branch 'lolol' into hip-423-loose-ends
f73c413
@tinker-michaelj @jsync-swirlds
Update hapi/hedera-protobufs/block/stream/output/state_changes.proto
1a41718 
Co-authored-by: Joseph S. <[email protected]>
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Add executionResultsAreStreamedAsExpected()
4bfdb37 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Merge branch 'antant' into hip-423-loose-ends
540e5ad
@tinker-michaelj
Add newline
79601f1 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Wrap remove() and purgeToNext() in K/V change streaming
9ee8753 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Fix BaseTranslator scheduleRef inferrence
54bdb02 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
scheduleRef is not a side effect
7923878 
Signed-off-by: Michael Tinker <[email protected]>
@tinker-michaelj
Copy link
Collaborator Author

Will be merged via #16017 (review)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsync-swirlds jsync-swirlds jsync-swirlds left review comments

@andrewb1269hg andrewb1269hg Awaiting requested review from andrewb1269hg andrewb1269hg is a code owner automatically assigned from hashgraph/devops-ci

@kimbor kimbor Awaiting requested review from kimbor kimbor is a code owner automatically assigned from hashgraph/hedera-services

@timo0 timo0 Awaiting requested review from timo0 timo0 is a code owner automatically assigned from hashgraph/platform-hashgraph

@artemananiev artemananiev Awaiting requested review from artemananiev artemananiev is a code owner automatically assigned from hashgraph/platform-data

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tinker-michaelj @jsync-swirlds @JivkoKelchev @vtronkov @ibankov