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);
   }