You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@orc.apache.org by do...@apache.org on 2021/11/02 03:38:07 UTC
[orc] branch branch-1.6 updated: ORC-1043: Fix C++ compilation
error in CentOS 7 (#957)
This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch branch-1.6
in repository https://gitbox.apache.org/repos/asf/orc.git
The following commit(s) were added to refs/heads/branch-1.6 by this push:
new 56f3bd2 ORC-1043: Fix C++ compilation error in CentOS 7 (#957)
56f3bd2 is described below
commit 56f3bd24692e5ae8a93c2bfbef8ac95ee07f70af
Author: William Hyun <wi...@apache.org>
AuthorDate: Mon Nov 1 20:37:03 2021 -0700
ORC-1043: Fix C++ compilation error in CentOS 7 (#957)
### What changes were proposed in this pull request?
This PR aims to fix a C++ compilation error in CentOS 7.
### Why are the changes needed?
To recover compilation.
### How was this patch tested?
Manually test on CentOS 7.
(cherry picked from commit 1ccec56e7dcc153b55047dc0c1b93383c531f731)
Signed-off-by: Dongjoon Hyun <do...@apache.org>
(cherry picked from commit 596f4632f8742a82a59b94fb9f1d4f080d7db195)
Signed-off-by: Dongjoon Hyun <do...@apache.org>
---
c++/test/TestMurmur3.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/c++/test/TestMurmur3.cc b/c++/test/TestMurmur3.cc
index 7bf9e05..2c05cd3 100644
--- a/c++/test/TestMurmur3.cc
+++ b/c++/test/TestMurmur3.cc
@@ -33,7 +33,7 @@ namespace orc {
" we had everything before us, we had nothing before us,"
" we were all going direct to Heaven,"
" we were all going direct the other way.";
- uint32_t len = sizeof(origin) / sizeof(uint8_t) - 1;
+ uint32_t len = static_cast<uint32_t>(sizeof(origin) / sizeof(uint8_t) - 1);
uint64_t hash = Murmur3::hash64(origin, len);
EXPECT_EQ(305830725663368540L, hash);
}