github-actions[bot] commented on code in PR #60323: URL: https://github.com/apache/doris/pull/60323#discussion_r3536513693
########## contrib/apache-orc: ########## @@ -1 +0,0 @@ -Subproject commit be0f1b73a7aeb78824a03e0dcb692c50a176d513 Review Comment: This PR deletes the gitlink for `contrib/apache-orc` and also deletes `contrib/datasketches-cpp`, but `.gitmodules` still declares both submodules. That leaves checkouts inconsistent: `git submodule update --init --recursive` will not populate these dependency directories from this commit even though the repository still advertises them, and BE builds/tests that expect the ORC or DataSketches sources can fail. These removals are unrelated to the brokers TVF feature, so please restore the gitlinks or move the submodule cleanup into a dedicated change that also updates all build references and `.gitmodules`. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
