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

[C++] Implement Map logical type #16979

Closed
asfimport opened this issue Jul 11, 2017 · 2 comments
Closed

[C++] Implement Map logical type #16979

asfimport opened this issue Jul 11, 2017 · 2 comments

Comments

@asfimport
Copy link

asfimport commented Jul 11, 2017

A map is implemented as a list of structs with fields key and value. We should separately discuss whether this merits an addition to the Arrow metadata

Reporter: Wes McKinney / @wesm
Assignee: Ben Kietzman / @bkietz

Related issues:

PRs and other links:

Note: This issue was originally created as ARROW-1207. Please see the migration documentation for further details.

@asfimport
Copy link
Author

Ben Kietzman / @bkietz:
Are there specific conventions or strong opinions about how this map type should be represented in a string? I'm leaning toward: map<utf8, int32> and map<int32, float64, keys_sorted> for unsorted/sorted keys respectively. Alternatively, I could follow dictionary type in verbosity by using map<keys=utf8, values=int32, keys_sorted=false> and map<keys=int32, values=float64, keys_sorted=true>.

@asfimport
Copy link
Author

Antoine Pitrou / @pitrou:
Issue resolved by pull request 4352
#4352

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

No branches or pull requests

2 participants