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]

Reply via email to