[
https://issues.apache.org/jira/browse/IMPALA-5078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17154041#comment-17154041
]
Tim Armstrong commented on IMPALA-5078:
---------------------------------------
[~sy117] it will be hard to get too far without resolving that compile issue.
hash.pb.h is generated as part of the build process. That should be built
automatically, so not sure what went wrong there. It's possible we're missing a
dependency in our build scripts. Can you try making this change to
be/src/kudu/util/CMakeLists.txt?
{noformat}
diff --git a/be/src/kudu/util/CMakeLists.txt b/be/src/kudu/util/CMakeLists.txt
index 82286acab..caf0b3faf 100644
--- a/be/src/kudu/util/CMakeLists.txt
+++ b/be/src/kudu/util/CMakeLists.txt
@@ -318,6 +318,7 @@ ADD_EXPORTABLE_LIBRARY(kudu_util
DEPS ${UTIL_LIBS}
NONLINK_DEPS gen_version_info
EXPORTED_DEPS ${EXPORTED_UTIL_LIBS})
+add_dependencies(kudu_util block_bloom_filter_proto)
#######################################
# kudu_util_compression
{noformat}
You'll need to add any new .cc files to one of the CMakeFiles.txt. In the case
of expr-test, it is already added as a test in be/src/exprs/CMakeLists.txt.
If you want to build just a single test, then "make expr-test" should work. Or
"make -j $(nproc) expr-test" for a parallel build.
Side note - you can find the full commands that would be used to build each
file in compile_commands.json in your impala build directory.
compile_commands.json is generated as part of the build process.
> Break up expr-test.cc
> ---------------------
>
> Key: IMPALA-5078
> URL: https://issues.apache.org/jira/browse/IMPALA-5078
> Project: IMPALA
> Issue Type: Improvement
> Components: Backend
> Reporter: Henry Robinson
> Assignee: Sean Yeh
> Priority: Minor
> Labels: newbie, ramp-up
> Attachments: Screen Shot 2020-06-30 at 12.19.16 PM.png
>
>
> {{expr-test.cc}} clocks in at 7129 lines, which is about enough for my emacs
> to start slowing down a bit. Let's see if we can refactor it enough to have a
> couple of test files. Maybe moving all the string instructions into a
> separate {{expr-string-test.cc}}, and having a common header will be enough
> to make it a bit more manageable.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
