morningman commented on pull request #3713: URL: https://github.com/apache/incubator-doris/pull/3713#issuecomment-637932239
> The status mainly reflect BE's status, if some error happends, it should be FE's problem?Our mainly purpose is to monitoring BE's status, so maybe we can leave it to make sure that BE's status is OK? I remember this PR is to solve some problems that BE cannot report successfully because of the version out of data. If there are other problems in the reporting process, it is equivalent to this BE not reporting successfully. SO I think it should also be able to be reflected in this status field. We should focus on the final result, not the success or failure of a certain part of the entire reporting process. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
