-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[objectstore] Fix checksums not being updated on modifying shared file #32337
Conversation
Codecov Report
@@ Coverage Diff @@
## master #32337 +/- ##
============================================
+ Coverage 64.01% 64.01% +<.01%
- Complexity 18564 18567 +3
============================================
Files 1171 1171
Lines 69851 69861 +10
Branches 1267 1267
============================================
+ Hits 44714 44722 +8
- Misses 24767 24769 +2
Partials 370 370
Continue to review full report at Codecov.
|
f69a21a
to
7124102
Compare
7124102
to
a038741
Compare
public function scanFile($file, $reuseExisting = 0, $parentId = -1, $cacheData = null, $lock = true) { | ||
$sourceScanner = $this->getSourceScanner(); | ||
if ($sourceScanner instanceof NoopScanner) { | ||
return []; | ||
list(, $internalPath) = $this->storage->resolvePath($file); | ||
return parent::scan($internalPath, $reuseExisting, $parentId, $cacheData, $lock); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DeepDiver1975 @PVince81 this is an easy fix for the issue.. but I think this class needs larger refactoring to make it support both local filesystem and objectstorage..
Looks good. Lets merge this to master and See ..... |
Looks good - see comment on PR with test scenarios #32353 (comment) |
Issue
Failing smashbox tests with sharing on objectstore.
Solution
SharingScanner did not update checksums in file cache for object store, since logic for obcjectstorage has not been implemented (SharingScanner assumes by default Local Filesystem). This PR adds logic for objectstorage.
@DeepDiver1975 @patrickjahns