Reranko05 commented on PR #50781: URL: https://github.com/apache/arrow/pull/50781#issuecomment-5187150639
> I also see many RapidJSON references in [cpp/cmake_modules/ThirdpartyToolchain.cmake](https://github.com/Reranko05/arrow/blob/777999e80bbae38a4656f7536e28825ab9140355/cpp/cmake_modules/ThirdpartyToolchain.cmake). Are we ready to remove those yet? (I don't know, hence the question) @rok RapidJSON is no longer used by Parquet after this change, so I've removed the Parquet-specific dependency. However, there are still many RapidJSON users elsewhere in Arrow (e.g. `arrow/json`, `arrow/integration`, `arrow/extension`,` arrow/flight/sql`), so I don't think we're ready to remove the `ThirdpartyToolchain` references yet. -- 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]
