Skip to content

HIVE-30042: Fix LLAP encoded/vectorized read path for TIMESTAMP WITH LOCAL TIME ZONE on ORC tables - #6776

Open
vikramahuja1001 wants to merge 2 commits into
apache:masterfrom
vikramahuja1001:HIVE-30042-ORC-OrderByFix
Open

vikramahuja1001 wants to merge 2 commits into
apache:masterfrom
vikramahuja1001:HIVE-30042-ORC-OrderByFix

Conversation

@vikramahuja1001

@vikramahuja1001 vikramahuja1001 commented Sep 11, 2026 •

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Why are the changes needed?

Check https://issues.apache.org/jira/browse/HIVE-30042 and #6761 for more details

Does this PR introduce any user-facing change?

No

How was this patch tested?

Add tests in .q file

@vikramahuja1001 vikramahuja1001 changed the title HIVE-30020: Fix TIMESTAMP WITH LOCAL TIME ZONE data loss and read failure on ORC tables HIVE-30042: Fix LLAP encoded/vectorized read path for TIMESTAMP WITH LOCAL TIME ZONE on ORC tables Sep 11, 2026
@vikramahuja1001 vikramahuja1001 changed the title HIVE-30042: Fix LLAP encoded/vectorized read path for TIMESTAMP WITH LOCAL TIME ZONE on ORC tables [WIP] HIVE-30042: Fix LLAP encoded/vectorized read path for TIMESTAMP WITH LOCAL TIME ZONE on ORC tables Sep 11, 2026
result = (TimestampLocalTZWritable) previous;
}
TimestampColumnVector tcv = (TimestampColumnVector) vector;
result.set(new TimestampTZ(tcv.time[row] / 1000, tcv.nanos[row], timeZone));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Math.floorDiv() should be used here as well?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. Missed it. Have added it now. Thanks for pointing it out

@vikramahuja1001
vikramahuja1001 force-pushed the HIVE-30042-ORC-OrderByFix branch from 3625f4d to 4a623ae Compare September 25, 2026 03:50
@vikramahuja1001 vikramahuja1001 changed the title [WIP] HIVE-30042: Fix LLAP encoded/vectorized read path for TIMESTAMP WITH LOCAL TIME ZONE on ORC tables HIVE-30042: Fix LLAP encoded/vectorized read path for TIMESTAMP WITH LOCAL TIME ZONE on ORC tables Sep 25, 2026
@sonarqubecloud

Copy link
Copy Markdown

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants