Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document types of $file_size and $file_modified_time #22627

Closed
elharo opened this issue Apr 29, 2024 · 4 comments · Fixed by #22642
Closed

Document types of $file_size and $file_modified_time #22627

elharo opened this issue Apr 29, 2024 · 4 comments · Fixed by #22642

Comments

@elharo
Copy link
Contributor

elharo commented Apr 29, 2024

On https://prestodb.io/docs/current/connector/hive.html I see

The Hive connector exposes extra hidden metadata columns in Hive tables. You can query these columns as a part of SQL query like any other columns of the table.

$path : Filepath for the given row data
$file_size : Filesize for the given row
$file_modified_time : Last file modified time for the given row

However this fails to say whether these are int32s or int64s. Add that detail. I'm pretty sure the correct answer is int64.

@steveburnett
Copy link
Contributor

@agrawalreetika, I believe this was originally your contribution.

Can you confirm if $file_size and $file_modified_time are int32s or int64s?
I can update the doc once I know. Thanks!

@aditi-pandit
Copy link
Contributor

@steveburnett : $file_size and $file_modified_time are int64_t for Presto Native Engine. I recently added these in Velox.

ref
facebookincubator/velox#8800
#21965

This would be compatible with Presto java. Though please confirm with Reetika as well.

@agrawalreetika
Copy link
Member

@steveburnett : $file_size and $file_modified_time are int64_t for Presto Java as well.

@steveburnett
Copy link
Contributor

Thanks @aditi-pandit and @agrawalreetika! I will open a doc PR to update the page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants