search

resource-disaggregation / jiffy

Virtual Memory Abstraction for Serverless Architectures
Apache License 2.0
45 stars 14 forks source link

fix bug in ds_file_node::load() and in fifo_queue_partition::dequeue() #113

Closed hesuyouren closed 4 years ago

hesuyouren commented 4 years ago

What changes were proposed in this pull request?

  1. Fix a bug in ds_file_node::load(), which chain.name and chain.metadata were not defined. The bug leads to no response when calling client.load() in pyjiffy.
  2. Fix a bug in fifo_queue_partition::dequeue(), which lacks returning element in RETURN_OK(). The bug leads to failure of pyjiffy test.

How was this patch tested?

  1. By running client.load() in pyclient. It no longer gets stuck.
  2. Now all pyclient tests pass.

Please review https://ucbrise.github.io/jiffy/contributing/ before opening a pull request.

AmplabJenkins commented 4 years ago

Can one of the admins verify this patch?

anuragkh commented 4 years ago

Add to whitelist