kevingurney commented on code in PR #37784: URL: https://github.com/apache/arrow/pull/37784#discussion_r1330480109
########## matlab/doc/matlab_interface_for_apache_arrow_design.md: ########## @@ -258,7 +258,7 @@ For large tables used in a multi-process "data processing pipeline", a user coul ## Testing To ensure code quality, we would like to include the following testing infrastructure, at a minimum: 1. C++ APIs - - GoogleTest C++ Unit Tests + - C++ Unit Tests Review Comment: That's a good question. I think it's fine to remove this for the time being (or, at least reword this section to talk about the MEX-based C++ testing approach). The thinking was that we might want to add some C++ tests in the future. However, we haven't yet had a compelling need to do that, so I think it makes sense to change the wording here. -- 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]
