sekingme opened a new pull request #8281: Add realization for updating version of derived segments in Materiali… URL: https://github.com/apache/incubator-druid/pull/8281 Some function has not been realized in MaterializedViewSupervisor.checkSegments. Check the comment: **2) version of derived segments isn't the max(created_date) of all base segments** So add realization for it: update version of derived segments if isn't the max (created_date) of all base segments.And prevent exception of "user supplied segments list did not match with segments list obtained from db" This function has been check for 3 months in out druid cluster of 105 nodes. <hr> This PR has: - [x ] been self-reviewed. - [x ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader. - [x ] added integration tests. - [x ] been tested in a test Druid cluster. <hr> ##### Key changed/added classes in this PR * `MaterializedViewSupervisor`
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
