Re: [PR] MINOR: [C++][Docs] Fix doxygen issue in ree_util.h [arrow]
zanmato1984 commented on PR #46012: URL: https://github.com/apache/arrow/pull/46012#issuecomment-2807222820 No problem at all. Just learning how we are organizing the doc. Thank you @amoeba ! -- 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]
Re: [PR] MINOR: [C++][Docs] Fix doxygen issue in ree_util.h [arrow]
amoeba commented on PR #46012: URL: https://github.com/apache/arrow/pull/46012#issuecomment-2807103886 Hi @zanmato1984, it's not documented, no. It didn't look like something we need/want to document, what do you think? -- 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]
Re: [PR] MINOR: [C++][Docs] Fix doxygen issue in ree_util.h [arrow]
zanmato1984 commented on PR #46012: URL: https://github.com/apache/arrow/pull/46012#issuecomment-2807026451 Aha, hit the same one after upgrading to Apple clang 17.0.0. Thank you @amoeba for fixing this! I have one side questions though: is the piece of doc in question accessible in our official website? I can't find it under places like [C++ API Reference](https://arrow.apache.org/docs/cpp/api.html). -- 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]
Re: [PR] MINOR: [C++][Docs] Fix doxygen issue in ree_util.h [arrow]
conbench-apache-arrow[bot] commented on PR #46012: URL: https://github.com/apache/arrow/pull/46012#issuecomment-2776369833 After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 2625ff3bf55f778b7457847babec8df289849e66. There were no benchmark performance regressions. 🎉 The [full Conbench report](https://github.com/apache/arrow/runs/39934330165) has more details. It also includes information about 12 possible false positives for unstable benchmarks that are known to sometimes produce them. -- 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]
Re: [PR] MINOR: [C++][Docs] Fix doxygen issue in ree_util.h [arrow]
kou merged PR #46012: URL: https://github.com/apache/arrow/pull/46012 -- 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]
