[ 
https://issues.apache.org/jira/browse/IGNITE-28920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18103986#comment-18103986
 ] 

Ignite TC Bot commented on IGNITE-28920:
----------------------------------------

{panel:title=Branch: [pull/13447/head] Base: [master] : No blockers 
found!|borderStyle=dashed|borderColor=#ccc|titleBGColor=#D6F7C1}{panel}
{panel:title=Branch: [pull/13447/head] Base: [master] : No new tests 
found!|borderStyle=dashed|borderColor=#ccc|titleBGColor=#F7D6C1}{panel}
[TeamCity *--> Run :: All* 
Results|https://ci2.ignite.apache.org/viewLog.html?buildId=9274888&buildTypeId=IgniteTests24Java8_RunAll]
{color:#ffffff}tcbot-analysis-comment chainBuildId=9274888 
rerunBuildIds=none{color}

> Reduce MarshallableMessage scope: GridCacheEntryInfo uses marshal hooks only 
> to rebase expireTime
> -------------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-28920
>                 URL: https://issues.apache.org/jira/browse/IGNITE-28920
>             Project: Ignite
>          Issue Type: Task
>            Reporter: Anton Vinogradov
>            Assignee: Vladimir Steshin
>            Priority: Major
>              Labels: IEP-132, ise, wire-format
>             Fix For: 2.19
>
>          Time Spent: 9h 10m
>  Remaining Estimate: 0h
>
> GridCacheEntryInfo implements MarshallableMessage, but its 
> marshal()/unmarshal() hooks never use the Marshaller: marshal() converts the 
> absolute expireTime into a remaining TTL before send, and unmarshal() 
> restores the absolute value on the receiver, compensating clock differences 
> between nodes. MarshallableMessage should stay reserved for messages that 
> actually carry marshalled payload. Proposal: move the expireTime rebase out 
> of the marshalling hooks and drop the MarshallableMessage interface from 
> GridCacheEntryInfo. Same cleanup direction as IGNITE-28914 
> (GridDhtPartitionsFullMessage).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to