[
https://issues.apache.org/jira/browse/MESOS-1194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14359142#comment-14359142
]
Akanksha Agrawal edited comment on MESOS-1194 at 3/13/15 7:36 PM:
------------------------------------------------------------------
Added the required validation in the constructor:
was (Author: akanksha08):
Added the required validatiion in the constructor:
{quote}
if (!message.IsInitialized()) {
std::cerr << "Protobuf not initialized: " <<
message.InitializationErrorString() << std::endl;
abort();
}
{quote}
> protobuf-JSON rendering doesnt validate
> ---------------------------------------
>
> Key: MESOS-1194
> URL: https://issues.apache.org/jira/browse/MESOS-1194
> Project: Mesos
> Issue Type: Bug
> Components: stout
> Affects Versions: 0.19.0
> Reporter: Till Toenshoff
> Assignee: Akanksha Agrawal
> Priority: Minor
> Labels: json, newbie, protobuf, stout
>
> When using JSON::Protobuf(Message&), the supplied protobuf is not checked for
> being properly initialized, hence e.g. required fields could be missing.
> It would be desirable to have a feedback mechanism in place for this
> constructor - maybe this would do:
> {noformat}
> if (!message.IsInitialized()) {
> std::cerr << "Protobuf not initialized: " <<
> message.InitializationErrorString() << std::endl;
> abort();
> }
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
