Doroszlai, Attila
Doroszlai, Attila
Thanks @Montura for the patch, @errose28 for the review.
/pending Can you please describe how it was tested?
Thanks @Montura for the contribution.
Thanks @chungen0126 for the fix. Why is this targeted at feature branch `HDDS-7593`? `isFileClosed` is already on `master`, it was added for HDDS-8436. It should be fixed on `master` first....
> Should I close this pr and create a new one for master? Not necessary. If you agree with the move to `master`, the PR's base branch can be changed....
@chungen0126 sorry, looks like GitHub does not realize only 2 commits are targeted at `master`: ``` * d76c7ef733 add test * 6180d45b79 [hsync] BasicOzoneClientAdapterImpl#isFileClosed using OFSPath which is an Utility...
@chungen0126 Start interactive rebase: ``` git rebase --interactive origin/master ``` Delete all commits from the editor except: ``` d76c7ef733 add test 6180d45b79 [hsync] BasicOzoneClientAdapterImpl#isFileClosed using OFSPath which is an Utility...
@chungen0126 force push is better in this case than a new PR
Thanks a lot @chungen0126 for updating the patch to cover snapshot-specific methods and rebasing for `master`.
Thanks @chungen0126 for the fix, @jojochuang for the review.