[
https://issues.apache.org/jira/browse/MESOS-8375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16321431#comment-16321431
]
Michael Park commented on MESOS-8375:
-------------------------------------
{noformat}
commit 3685c011bb71da0ba2af75691101ea383eeb2ccd
Author: Michael Park <[email protected]>
Date: Sat Jan 6 00:48:58 2018 -0800
Replaced `os::read` with `state::read`.
The `state::checkpoint` utility was updated to checkpoint
the automatically downgraded resources. This was done to mitigate
the need to manually invoke `downgradeResources` prior to
checkpointing. `state::read` was introduced to provide a symmetric
functionality for `state::checkpoint`. Specifically, it will perform
upgrade resources upon reading the checkpointed resources state.
This patch updates the previous uses of `os::read` which was used to
read the state that was written by `state::checkpoint(path, string)`.
While there is no functional change, it completes the picture where
`state::read` is used to read state written by `state::checkpoint`.
Review: https://reviews.apache.org/r/65025
{noformat}
{noformat}
commit 80f66061e343fa26dd7e3b9613f0fa8e0b9b4a36
Author: Michael Park <[email protected]>
Date: Fri Jan 5 18:03:42 2018 -0800
Replaced `protobuf::read` with `state::read`.
The `state::checkpoint` utility was updated to checkpoint
the automatically downgraded resources. This was done to mitigate
the need to manually invoke `downgradeResources` prior to
checkpointing. `state::read` was introduced to provide a symmetric
functionality for `state::checkpoint`. Specifically, it will perform
upgrade resources upon reading the checkpointed resources state.
This patch updates the previous uses of `protobuf::read` accompanied
by calls to `convertResourceFormat`. Rather than reading the protobufs
then upgrading the resources, we know simply call `state::read` which
performs the reverse operation of `state::checkpoint`.
Review: https://reviews.apache.org/r/65024
{noformat}
{noformat}
commit ef303905e0be96f28d79a569afd004ec75f98296
Author: Michael Park <[email protected]>
Date: Fri Jan 5 09:52:51 2018 -0800
Added `state::read` to complement `state::checkpoint`.
Review: https://reviews.apache.org/r/65023
{noformat}
{noformat}
commit fda054b50ff7cdd2d7a60d31cfe24ce42bfbfaa5
Author: Michael Park <[email protected]>
Date: Fri Jan 5 17:44:41 2018 -0800
Updated uses of `protobuf::read(path)` which now returns `Try<T>`.
Since the path version of `protobuf::read` now returns `Try<T>`,
many of the existing code is removed and/or simplified.
Review: https://reviews.apache.org/r/65022
{noformat}
> Use protobuf reflection to simplify upgrading of resources.
> -----------------------------------------------------------
>
> Key: MESOS-8375
> URL: https://issues.apache.org/jira/browse/MESOS-8375
> Project: Mesos
> Issue Type: Task
> Reporter: Michael Park
> Assignee: Michael Park
> Priority: Blocker
>
> This is the {{upgradeResources}} half of the protobuf-reflection-based
> upgrade/downgrade of resources:
> https://issues.apache.org/jira/browse/MESOS-8221
> We will also add {{state::read}} to complement {{state::checkpoint}} which
> will be used to read protobufs from disk rather than {{protobuf::read}}.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
