pitrou commented on PR #50568: URL: https://github.com/apache/arrow/pull/50568#issuecomment-5105923424
> Since my PR doesn't modify `reader_test.cc`, I'm not sure whether this indicates a latent issue in the test or a Unity/MSVC-specific lookup interaction. Would you prefer that I include those qualifications in this PR, or is there another approach you'd recommend? Yes, it's quite typical of unity builds which amalgamate several source files (`.cc` files) into a single translation unit, which might surface multiple sub-namespaces named `internal`. That's why you may have to use the fully-qualified namespace to fix the issue. -- 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]
