[GitHub] [doris] dataroaring commented on pull request #10405: Add MigrationHandler

[GitBox <gi...@apache.org>]

dataroaring commented on PR #10405:
URL: https://github.com/apache/doris/pull/10405#issuecomment-1166939289

   It seems that the patch move cold data to s3 by creating new tablet on s3, then users can not insert new data to the cold tablet?
   
   We implement the feature by just moving cold rowset to s3, then users can continue insert new data to all tablets.  IMHO, the later solution is more friendly to users. The pr is here https://github.com/apache/doris/pull/10280.  
   
   BTW, https://github.com/apache/doris/pull/10280 is now being tested, and our regression tests has passed. We also did performance testing, and I will add performance data later in the 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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org