Skip to content

IGNITE-28404 Fixed the stopping of lease prolongation during lease negotiation#7931

Open
denis-chudov wants to merge 3 commits intoapache:mainfrom
gridgain:ignite-28404
Open

IGNITE-28404 Fixed the stopping of lease prolongation during lease negotiation#7931
denis-chudov wants to merge 3 commits intoapache:mainfrom
gridgain:ignite-28404

Conversation

@denis-chudov
Copy link
Copy Markdown
Contributor

@denis-chudov denis-chudov changed the title IGNITE-28404 IGNITE-28404 Fixed the stopping of lease prolongation during lease negotiation Apr 7, 2026
.deniedLeaseExpirationTimeLong(time)
.build();

clusterService.messagingService().respond(sender, response, correlationId);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if correlationId is null?

integrationTestImplementation project(':ignite-table')
integrationTestImplementation project(':ignite-transactions')
integrationTestImplementation project(':ignite-eventlog')
integrationTestImplementation project(':ignite-placement-driver-api')
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we really need this?

} else {
long time = lease.getLeaseholderId() == null
? clockService.current().longValue()
: NULL_HYBRID_TIMESTAMP;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you please explain why we return NULL_HYBRID_TIMESTAMP? Why not lease.getExpirationTime().longValue()?

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