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.

Sign up for GitHub

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

Jump to bottom

Add OrganizeBlob function to BufferOrganizer #333

Merged
merged 28 commits into from
Jan 27, 2022
Merged

Add OrganizeBlob function to BufferOrganizer #333

merged 28 commits into from
Jan 27, 2022

Conversation

ChristopherHogan
Copy link
Collaborator

Closes #238.

  • Adds OrganizeBlob, which triggers the BufferOrganizer to align a Blob's access score with its importance score.
  • Reduce noise by changing some adapter logging from LOG(INFO) to VLOG(1).
  • Add -d yes to ctest runs so it's easier to see which test failed in CI.

@ChristopherHogan
Stubs for BO organize function
0212afa
@ChristopherHogan
Basics of BO organize function
fc67f88
@ChristopherHogan
BO organize function mostly laid out
82954df
@ChristopherHogan
Factor GetBufferInfo out of ComputeBlobAccessScore and recompute scor…
64ad9ca 
…e on each iteration
@ChristopherHogan
Implemented BoMove
9d0ed67
@ChristopherHogan
Add metadata updates in BoMove
3b673b9
@ChristopherHogan
BoMove bug fixes and working test
3242e04
@ChristopherHogan
TestBoMove working
139ba2e
@ChristopherHogan
Enable BORG tests
6abaa71
@ChristopherHogan
Add user API and test OrganizeBlob
b466129
@ChristopherHogan
Make LocalReplaceblobIdInBucket atomic
a5d7894
@ChristopherHogan
Fix typo in hermes.conf
ca85966
@ChristopherHogan
Release Buffers only after new blob_id is set up.
17d52c8
@ChristopherHogan
OrganizeBlob working
10eebb3
@ChristopherHogan
Debugging CI failure
f234e13
@ChristopherHogan
Debugging
bf626da
@ChristopherHogan
Print test name in adapter test output
f19e81f
@ChristopherHogan
Add missing env var to installation script
23b2e2c
@ChristopherHogan
App ranks can only call the BO via RPC
e6b2ac8
@ChristopherHogan
Merge latest changes from master
4b59d49
@ChristopherHogan
Make CI output easier to examine
88e245f
@ChristopherHogan
Merge branch 'master' into chogan/bo_organize
72b1c59
@ChristopherHogan
Merge branch 'master' into chogan/bo_organize
53309cb
@ChristopherHogan
Merge branch 'master' into chogan/bo_organize
c128541
@ChristopherHogan
Release BlobInfo lock if entry doesn't exist
b4076d4
@ChristopherHogan
Merge branch 'master' into chogan/bo_organize
f3c423a
@ChristopherHogan
Remove blank TODO
2ba5b15
@ChristopherHogan
Remove LOCAL definition from install_deps.sh. It's passed in from the…
c94d1fb 
… GH action
@ChristopherHogan ChristopherHogan added enhancement New feature or request effort-large Can be completed in a month or less labels Jan 27, 2022
@ChristopherHogan ChristopherHogan self-assigned this Jan 27, 2022
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 81.512% when pulling c94d1fb on chogan/bo_organize into c834ddc on master.

@ChristopherHogan ChristopherHogan merged commit ab2b258 into master Jan 27, 2022
@ChristopherHogan ChristopherHogan deleted the chogan/bo_organize branch January 27, 2022 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ChristopherHogan ChristopherHogan

Labels
effort-large Can be completed in a month or less enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement OrganizeBlob
2 participants
@ChristopherHogan @coveralls