Repository navigation
Additional Unit Tests for Parquet VARIANT Field Extraction #23036
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
Merged
rapids-bot
merged 56 commits into
NVIDIA:main
from
abigalekim:ak/field-extraction-tests
Jul 28, 2026
Merged
Changes from 19 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
80a89aa
test draft
693c014
Merge branch 'main' into ak/field-extraction-tests
abigalekim 7032897
Merge branch 'main' into ak/field-extraction-tests
abigalekim d4c5d2d
Merge branch 'main' of github.com:abigalekim/cudf into ak/field-extra…
e7d4bca
stuff
614b290
reviews
f3459e5
Merge branch 'main' into ak/field-extraction-tests
abigalekim 9282a81
Merge branch 'ak/field-extraction-tests' of github.com:abigalekim/cud…
097347f
addressing reviews
fc49abe
addressing reviews
556544b
Merge branch 'main' into ak/field-extraction-tests
2df4645
Merge branch 'main' into ak/field-extraction-tests
7413eb2
review
3bb935f
Merge branch 'ak/field-extraction-tests' of github.com:abigalekim/cud…
f20ea45
new file
fd1d435
Merge branch 'main' into ak/field-extraction-tests
0dd37a4
Update cpp/include/cudf/io/experimental/variant_spec.hpp
abigalekim ca74bed
Merge branch 'main' into ak/field-extraction-tests
abigalekim 07ca0e8
Merge branch 'main' into ak/field-extraction-tests
abigalekim 40d9673
stuff
1dae299
merge
08b4f31
Merge branch 'main' into ak/field-extraction-tests
42d1f10
coderabbit
88c66b8
git Merge branch 'ak/field-extraction-tests' of github.com:abigalekim…
3ff7b05
Update cpp/include/cudf/io/experimental/variant_spec.hpp
abigalekim 39bdf07
Update cpp/include/cudf/io/experimental/variant_spec.hpp
abigalekim b49dbd9
Merge branch 'main' into ak/field-extraction-tests
abigalekim 469509e
clang format
743b4c7
hopefully passes CI
26b5ff6
Merge branch 'main' into ak/field-extraction-tests
2fe74fa
Merge branch 'main' into ak/field-extraction-tests
70cf457
Merge branch 'main' into ak/field-extraction-tests
57321f1
Update cpp/tests/io/experimental/variant_extract_test.cpp
abigalekim d09b0ad
Update cpp/tests/io/experimental/variant_extract_test.cpp
abigalekim 69ed9d2
Merge branch 'main' into ak/field-extraction-tests
abigalekim b417378
Merge branch 'main' into ak/field-extraction-tests
abigalekim 7f1ad8d
Merge branch 'main' into ak/field-extraction-tests
abigalekim c93b024
clang format
0882f0a
Merge branch 'main' into ak/field-extraction-tests
10dce74
Merge branch 'main' into ak/field-extraction-tests
148a04b
Merge branch 'main' into ak/field-extraction-tests
1370221
Merge branch 'main' into ak/field-extraction-tests
15c5563
Merge branch 'main' into ak/field-extraction-tests
526bcdc
Merge branch 'main' into ak/field-extraction-tests
9c11709
Merge branch 'main' into ak/field-extraction-tests
c4e865a
Merge branch 'main' into ak/field-extraction-tests
f0c2ce1
Merge branch 'main' into ak/field-extraction-tests
01e5942
Merge branch 'main' into ak/field-extraction-tests
48536d1
Update variant enum references missed in merge from main
vuule 8ed7f3b
Merge branch 'main' into ak/field-extraction-tests
vuule c6a5180
Merge branch 'main' into ak/field-extraction-tests
d6daa7e
Merge branch 'main' into ak/field-extraction-tests
bf553c2
test
8d19aaf
Merge branch 'ak/field-extraction-tests' of github.com:abigalekim/cud…
e41f893
reflect float and bool support
03194b0
Merge branch 'main' into ak/field-extraction-tests
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,39 @@ | ||
| /* | ||
| * SPDX-FileCopyrightText: Copyright (c) 2026, NVIDIA CORPORATION & AFFILIATES. All rights reserved. | ||
| * SPDX-License-Identifier: Apache-2.0 | ||
| */ | ||
|
|
||
| #pragma once | ||
|
|
||
| #include <cstdint> | ||
|
|
||
| namespace cudf::io::parquet::experimental { | ||
|
|
||
| /** | ||
| * `@brief` Physical type ID carried in the value_header of a primitive VARIANT value. | ||
| */ | ||
| enum class variant_primitive_type : uint8_t { | ||
| null = 0, | ||
| boolean_true = 1, | ||
| boolean_false = 2, | ||
| int8 = 3, | ||
| int16 = 4, | ||
| int32 = 5, | ||
| int64 = 6, | ||
| float64 = 7, | ||
| decimal4 = 8, | ||
| decimal8 = 9, | ||
| decimal16 = 10, | ||
| date = 11, | ||
| timestamp_micros = 12, | ||
| timestamp_ntz_micros = 13, | ||
| float32 = 14, | ||
| binary = 15, | ||
| long_string = 16, | ||
| time_ntz_micros = 17, | ||
| timestamp_nanos = 18, | ||
| timestamp_ntz_nanos = 19, | ||
| uuid = 20, | ||
| }; | ||
|
|
||
| } // namespace cudf::io::parquet::experimental | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.