hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hive QA (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-21290) Restore historical way of handling timestamps in Parquet while keeping the new semantics at the same time
Date Thu, 21 Mar 2019 17:11:00 GMT

    [ https://issues.apache.org/jira/browse/HIVE-21290?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16798277#comment-16798277
] 

Hive QA commented on HIVE-21290:
--------------------------------

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  0s{color} |
{color:green} The patch does not contain any @author tags. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  1m 45s{color} | {color:blue}
Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  7m 18s{color}
| {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  8m  1s{color} |
{color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  3m  1s{color}
| {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m 38s{color} | {color:blue}
common in master has 63 extant Findbugs warnings. {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  4m 16s{color} | {color:blue}
ql in master has 2255 extant Findbugs warnings. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  8m 45s{color} |
{color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 26s{color} | {color:blue}
Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  9m 15s{color}
| {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  8m  6s{color} |
{color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  8m  6s{color} | {color:green}
the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 44s{color} | {color:red}
ql: The patch generated 25 new + 193 unchanged - 22 fixed = 218 total (was 215) {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  2m  2s{color} | {color:red}
root: The patch generated 25 new + 194 unchanged - 22 fixed = 219 total (was 216) {color}
|
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m  0s{color}
| {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  5m 11s{color} |
{color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  8m 24s{color} |
{color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 13s{color}
| {color:green} The patch does not generate ASF License warnings. {color} |
| {color:black}{color} | {color:black} {color} | {color:black} 69m 11s{color} | {color:black}
{color} |
\\
\\
|| Subsystem || Report/Notes ||
| Optional Tests |  asflicense  javac  javadoc  findbugs  checkstyle  compile  |
| uname | Linux hiveptest-server-upstream 3.16.0-4-amd64 #1 SMP Debian 3.16.36-1+deb8u1 (2016-09-03)
x86_64 GNU/Linux |
| Build tool | maven |
| Personality | /data/hiveptest/working/yetus_PreCommit-HIVE-Build-16613/dev-support/hive-personality.sh
|
| git revision | master / 38682a4 |
| Default Java | 1.8.0_111 |
| findbugs | v3.0.0 |
| checkstyle | http://104.198.109.242/logs//PreCommit-HIVE-Build-16613/yetus/diff-checkstyle-ql.txt
|
| checkstyle | http://104.198.109.242/logs//PreCommit-HIVE-Build-16613/yetus/diff-checkstyle-root.txt
|
| modules | C: common ql . U: . |
| Console output | http://104.198.109.242/logs//PreCommit-HIVE-Build-16613/yetus.txt |
| Powered by | Apache Yetus    http://yetus.apache.org |


This message was automatically generated.



> Restore historical way of handling timestamps in Parquet while keeping the new semantics
at the same time
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: HIVE-21290
>                 URL: https://issues.apache.org/jira/browse/HIVE-21290
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Zoltan Ivanfi
>            Assignee: Karen Coppage
>            Priority: Major
>         Attachments: HIVE-21290.1.patch, HIVE-21290.2.patch, HIVE-21290.2.patch
>
>
> This sub-task is for implementing the Parquet-specific parts of the following plan:
> h1. Problem
> Historically, the semantics of the TIMESTAMP type in Hive depended on the file format.
Timestamps in Avro, Parquet and RCFiles with a binary SerDe had _Instant_ semantics, while
timestamps in ORC, textfiles and RCFiles with a text SerDe had _LocalDateTime_ semantics.
> The Hive community wanted to get rid of this inconsistency and have _LocalDateTime_ semantics
in Avro, Parquet and RCFiles with a binary SerDe as well. *Hive 3.1 turned off normalization
to UTC* to achieve this. While this leads to the desired new semantics, it also leads to incorrect
results when new Hive versions read timestamps written by old Hive versions or when old Hive
versions or any other component not aware of this change (including legacy Impala and Spark
versions) read timestamps written by new Hive versions.
> h1. Solution
> To work around this issue, Hive *should restore the practice of normalizing to UTC* when
writing timestamps to Avro, Parquet and RCFiles with a binary SerDe. In itself, this would
restore the historical _Instant_ semantics, which is undesirable. In order to achieve the
desired _LocalDateTime_ semantics in spite of normalizing to UTC, newer Hive versions should
record the session-local local time zone in the file metadata fields serving arbitrary key-value
storage purposes.
> When reading back files with this time zone metadata, newer Hive versions (or any other
new component aware of this extra metadata) can achieve _LocalDateTime_ semantics by *converting
from UTC to the saved time zone (instead of to the local time zone)*. Legacy components that
are unaware of the new metadata can read the files without any problem and the timestamps
will show the historical Instant behaviour to them.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message