gavinchou opened a new pull request, #66669:
URL: https://github.com/apache/doris/pull/66669

   ## Proposed changes
   
   Fix `CloudSchemaChangeJobTest.FillVersionHolesBeforeNewTabletRunning` by 
making the test fixture match the current cloud schema change flow:
   
   - inject the base tablet `[2-2]` rowset returned by the mocked schema change 
`alter_version=2`
   - provide a local-backed remote filesystem for cloud rowset writer 
initialization
   - mock `prepare_rowset`/`commit_rowset` for historical rowset conversion
   - make zero-segment fixture rowsets internally consistent (`empty=true`, 
`num_rows=0`, `newest_write_timestamp` set)
   
   ## Problem
   
   The test mocked `prepare_tablet_job` with `alter_version=2`, but only 
injected rowsets into the new tablet. Current master refreshes and reads the 
base tablet historical rowsets for `[2, alter_version]`, so the test failed 
with `failed to find path in version_graph. spec_version: 2-2`.
   
   ## Validation
   
   - `sh format_code.sh be/test/cloud/cloud_schema_change_job_test.cpp`
   - `sh run-be-ut.sh --run --filter 
"CloudSchemaChangeJobTest.FillVersionHolesBeforeNewTabletRunning"`
   - `sh run-be-ut.sh --run --filter "CloudSchemaChangeJobTest.*"`
   
   Jira: DORIS-27883
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to