orc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From xndai <...@git.apache.org>
Subject [GitHub] orc pull request #142: [ORC-218] Cache timezone information in the library.
Date Mon, 31 Jul 2017 21:59:36 GMT
Github user xndai commented on a diff in the pull request:

    https://github.com/apache/orc/pull/142#discussion_r130473949
  
    --- Diff: c++/src/CMakeLists.txt ---
    @@ -125,6 +125,83 @@ include_directories (
       ${LZ4_INCLUDE_DIRS}
       )
     
    +# To avoid reading the Timezone database from disk, we load the file during the
    +# build and inject them as binary into the library. This can increase the size
    +# of library but avoids doing system calls at runtime. The behavior can be
    +# disabled by using the CMake flag NO_EMBEDDED_TZ_DB
    +if (NOT DEFINED EMBEDDED_TZ_DB)
    +  set(EMBEDDED_TZ_DB ON)
    --- End diff --
    
    A side effect of this is when the time zone info files are updated, the Orc lib has to
be recompiled and redistributed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message