liangyepianzhou opened a new pull request, #4675: URL: https://github.com/apache/bookkeeper/pull/4675
### Motivation Many PRs cannot be merged because of this test failure. Even after rerunning multiple times, they still fail due to the same error. ` Error: OUTPUT>native-io-jni/cpp/native_io_jni.c:168:21: error: passing argument 1 of 'fflush' makes pointer from integer without a cast [-Wint-conversion] ` https://github.com/apache/bookkeeper/actions/runs/18648851159/job/53162582666?pr=4674 https://github.com/apache/bookkeeper/actions/runs/18552216345/job/53161739919?pr=4673 https://github.com/apache/bookkeeper/actions/runs/18526985175/job/52800716265?pr=4672 https://github.com/apache/bookkeeper/actions/runs/18526392532/job/52798756494?pr=4653 ### Changes fflush requires a FILE*, but we're passing an int file descriptor. On Windows, the correct equivalent is _commit(fd), which takes an int fd. So, change the mapping of fsync from fflush to _commit. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
