mszeszko-meta

Results 6 comments of mszeszko-meta

@adamretter it looks like RocksDB native library cannot be found or loaded (build-linux-java). I think we should fix it before merging this change.

@adamretter, separately, there are well over 100 warnings about deprecated features, syntax, enums in `build-linux-java` [here](https://github.com/facebook/rocksdb/actions/runs/15930819852/job/44939332829?pr=13609). I think we might be reaching a good critical mass to tackle this in...

> > it looks like RocksDB native library cannot be found or loaded (build-linux-java). I think we should fix it before merging this change. > > @mszeszko-meta That's strange, all...

@adamretter , is this commit a blocker for 10.4 release? In other words, does it require back porting?

TLDR; this PR consists of 3 parts: 1) Correctness for (sequentially) interleaving `PosixRandomAccessFile::MultiRead` and `PosixRandomAccessFile::ReadAsync` operations within the very same thread. 2) Reliability: enhanced error handling (e.g -EINTR, -EAGAIN, -ENOMEM)....

> LGTM. I noticed that Poll() doesn't have logic to resubmit the request if bytes_read < requested length. Maybe something to keep in mind as a todo. Good catch @anand1976...