----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65844/ -----------------------------------------------------------
Review request for mesos, Benjamin Mahler and Michael Park. Repository: mesos Description ------- Currently only `Resources::parse()` calls `upgradeResource()`, but the actual parsing functions `Resources::fromJSON()` and `Resources::fromSimpleString()` which are also public utilities do not, this may produce invalid resource objects. This patch moves the `upgradeResource()` to the two internal parsing functions, so that all resource objects parsed are valid. Diffs ----- src/common/resources.cpp a0c29246b3e3b377aabd0eb00ba16aa6bbd804e3 Diff: https://reviews.apache.org/r/65844/diff/1/ Testing ------- make check Thanks, Meng Zhu
