Support map_keys & map_values for MAP type#12194
Conversation
map_values for MAP typemap_keys for MAP type
|
Encounter a bug. Fixed it by changing field name to |
I think it is caused by the pre-defined field name in datafusion/datafusion/functions-nested/src/map.rs Lines 33 to 39 in 7e9ea3a |
…-support-map-values # Conflicts: # datafusion/functions-nested/src/lib.rs
|
@dharanad could you add some doc for this function? |
map_keys for MAP typemap_keys & map_values for MAP type
|
Sorry for the mess, i thought since the change was similar for both |
Co-authored-by: Alex Huang <[email protected]>
|
After fixing CI, it is ready to go |
@Weijun-H Any idea, how do i fix the docs formatting issue |
You could check here https://datafusion.apache.org/contributor-guide/howtos.html#how-to-format-md-document |
Co-authored-by: Alex Huang <[email protected]>
|
ship it! |
|
Thanks @dharanad contribution, and @Blizzara and @jayzhan211 for reviewing. |
|
Thank You @Weijun-H @jayzhan211 @Blizzara |
Which issue does this PR close?
Closes #12147 & #12148
Rationale for this change
What changes are included in this PR?
map_keys&map_valuesscalar function. Ref : https://duckdb.org/docs/sql/functions/map.html#map_valuesmapAre these changes tested?
Are there any user-facing changes?