Skip to content
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

Remove unused block_fs cache functionality #2499

Merged
merged 1 commit into from
Dec 7, 2021

Conversation

pinkwah
Copy link
Contributor

@pinkwah pinkwah commented Dec 3, 2021

No description provided.

@pinkwah pinkwah self-assigned this Dec 3, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #2499 (611f39c) into main (a1fdf7f) will decrease coverage by 0.00%.
The diff coverage is 88.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2499      +/-   ##
==========================================
- Coverage   63.59%   63.59%   -0.01%     
==========================================
  Files         660      658       -2     
  Lines       54986    54885     -101     
  Branches     4344     4341       -3     
==========================================
- Hits        34970    34904      -66     
+ Misses      18702    18666      -36     
- Partials     1314     1315       +1     
Impacted Files Coverage Δ
libres/lib/job_queue/job_queue.cpp 9.58% <0.00%> (ø)
libres/lib/res_util/block_fs.cpp 53.51% <88.88%> (-0.90%) ⬇️
libres/lib/enkf/block_fs_driver.cpp 81.35% <100.00%> (-0.97%) ⬇️
libres/lib/enkf/enkf_fs.cpp 71.25% <100.00%> (-0.15%) ⬇️
ert_gui/simulation/run_dialog.py 75.70% <0.00%> (-0.71%) ⬇️
res/enkf/enkf_main.py 83.33% <0.00%> (-0.08%) ⬇️
res/util/__init__.py 100.00% <0.00%> (ø)
libres/lib/res_util/matrix_stat.cpp 0.00% <0.00%> (ø)
res/util/stat.py
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1fdf7f...611f39c. Read the comment docs.

@dafeda
Copy link
Contributor

dafeda commented Dec 7, 2021

Do you know if ENABLE_CACHE has ever been used to activate cache?

block_fs_fsync(block_fs);
}
block_fs->write_count++;
if (block_fs->fsync_interval &&
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe just a curiosity, but this condition is never met in the c testsuite (in other words, block_fs_fsync is never called in the tests). If that is easy to remedy, it would be very nice :)

@pinkwah
Copy link
Contributor Author

pinkwah commented Dec 7, 2021

Jenkins test this please

Copy link
Contributor

@eivindjahren eivindjahren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pinkwah pinkwah merged commit 7fefd87 into equinor:main Dec 7, 2021
@pinkwah pinkwah deleted the remove-cache branch December 7, 2021 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants