surahman edited a comment on pull request #3786: URL: https://github.com/apache/incubator-heron/pull/3786#issuecomment-1079303754
> 1. Remove the envelope which was part of each returned API call. Not sure if it's a great idea to have. We didn't really use the feature. We never set the timing info. We already have HTTP return codes for status If option 1 simplifies the code then I would agree with going down this route but my fear is that it might break something elsewhere. I think it would likely be better to go down this route and bring things up to current spec than to hack things or work around the issue. Let us plug this issue permanently or it will resurface later. For option 2, how many/which handlers would we need to update to handle the encasing envelopes? I am looking things over and I will try to assist as much as I can on this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
