apache / distributedlog

Apache DistributedLog
Apache License 2.0
185 stars 74 forks source link

Make distributedlog compiled with latest bookkeeper version #240

Closed sijie closed 6 years ago

sijie commented 6 years ago

Descriptions of the changes in this PR:

There are code changes on PendingReadOp for new bookkeeper api in current master. DistributedLog uses PendingReadOp for some administration tools. So the current master doesn't compile with the latest bookkeeper version. This code change is to fix that.

The change here includes:

sijie commented 6 years ago

@jiazhai I think CI failed due to one flaky test. one of the travis ci passed - https://travis-ci.org/apache/distributedlog/jobs/308759899