lakeFS icon indicating copy to clipboard operation
lakeFS copied to clipboard

Read from branch with compaction data

Open idanovo opened this issue 1 year ago • 3 comments
trafficstars

idanovo avatar Apr 25 '24 14:04 idanovo

E2E Test Results - DynamoDB Local - Local Block Adapter

13 passed

github-actions[bot] avatar Apr 25 '24 14:04 github-actions[bot]

Agree with @arielshaqed comments. Is this PR goal to cover all branch reading scenarios? What about functions that are not covered in this PR, like listStagingArea, isStagingEmpty, ..?

itaiad200 avatar Apr 28 '24 06:04 itaiad200

Graveler function Tested Controller function Tested AI
ResetKey X ResetBranch V Add a controller test for ResetKey for a compacted branch when we add the CompactBranch API call
ResetPrefix X ResetBranch V Add a controller test for ResetPrefix for a compacted branch when we add the CompactBranch API call
prepareForCommitIDUpdate X Revert (dirty branch) V Added a test in graveler testsShould also add a test in the controller once we have the CompactBranch API call
prepareForCommitIDUpdate X CherryPick (dirty branch) V Add a controller test for CherryPick for a compacted branch when we add the CompactBranch API call
prepareForCommitIDUpdate X UpdateBranch (dirty branch) X Add a controller test for ResetPrefix for a compacted branch when we add the CompactBranch API call
prepareForCommitIDUpdate X Merge (dirty branch) V Added a test in graveler testsShould also add a test in the controller once we have the CompactBranch API call
prepareForCommitIDUpdate X Import (dirty branch) X Add a controller test for ResetPrefix for a compacted branch when we add the CompactBranch API call

idanovo avatar May 08 '24 12:05 idanovo