lidavidm commented on PR #48313: URL: https://github.com/apache/arrow/pull/48313#issuecomment-3609324929
> > I'll try to take a look but as an initial comment is the commit about moving from `boost::optional` to `std::optional` required for this PR? [a696be3](https://github.com/apache/arrow/commit/a696be38a45ccb914cd9aca874911570bec9298e) > > Yes I think so, previously when I was testing my changes on a different branch, the build was failing due to mixed usages of `boost::optional` and `std::optional`. Having this commit resolved the build failures Do you want to submit this as another PR first? -- 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]
