Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
apache
/
datafusion-comet
Public
Notifications
You must be signed in to change notification settings
Fork
384
Star
1.3k
Code
Issues
548
Pull requests
213
Discussions
Actions
Projects
Security and quality
0
Insights
Additional navigation options
Code
Issues
Pull requests
Discussions
Actions
Projects
Security and quality
Insights
fix: enforce null-key rejection and mapKeyDedupPolicy in native map construction
- #5854
#5854
Open
peterxcli
wants to merge 25 commits into
apache:main
apache/datafusion-comet:main
from
peterxcli:fix/map-null-key-and-dedup-policy
peterxcli/datafusion-comet:fix/map-null-key-and-dedup-policy
Copy head branch name to clipboard
Conversation
Commits
25
(25)
Checks
Files changed
Open
fix: enforce null-key rejection and mapKeyDedupPolicy in native map construction
#5854
peterxcli
wants to merge 25 commits into
apache:main
apache/datafusion-comet:main
from
peterxcli:fix/map-null-key-and-dedup-policy
peterxcli/datafusion-comet:fix/map-null-key-and-dedup-policy
Copy head branch name to clipboard
Commits
Commits on Sep 10, 2026
fix: enforce null-key rejection and mapKeyDedupPolicy in native map construction
Show description for 32bf30b
peterxcli
committed
32bf30b
View commit details
Copy full SHA for 32bf30b
Browse repository at this point
Commits on Sep 12, 2026
Merge remote-tracking branch 'upstream/main' into fix/map-null-key-and-dedup-policy
Show description for 3eabcdc
peterxcli
committed
3eabcdc
View commit details
Copy full SHA for 3eabcdc
Browse repository at this point
Commits on Sep 15, 2026
fix: read the right row when a map builder gets a sliced list argument
Show description for e4cd899
peterxcli
committed
e4cd899
View commit details
Copy full SHA for e4cd899
Browse repository at this point
fix: report whichever of a null or duplicate map key comes first
Show description for 0a37af9
peterxcli
committed
0a37af9
View commit details
Copy full SHA for 0a37af9
Browse repository at this point
feat: decline a collated key type in the native map constructors
Show description for 846819a
peterxcli
committed
846819a
View commit details
Copy full SHA for 846819a
Browse repository at this point
docs: scope the floating-point map key note to what Spark actually does
Show description for f7f87f3
peterxcli
committed
f7f87f3
View commit details
Copy full SHA for f7f87f3
Browse repository at this point
test: pin the map length mismatch error class instead of comparing engines
Show description for 0021e46
peterxcli
committed
0021e46
View commit details
Copy full SHA for 0021e46
Browse repository at this point
Merge remote-tracking branch 'upstream/main' into fix/map-null-key-and-dedup-policy
peterxcli
committed
86bf5d1
View commit details
Copy full SHA for 86bf5d1
Browse repository at this point
test: map_from_entries stays native under LAST_WIN in the routing fixtures
Show description for 3085702
peterxcli
committed
3085702
View commit details
Copy full SHA for 3085702
Browse repository at this point
Commits on Sep 16, 2026
Merge remote-tracking branch 'upstream/main' into fix/map-null-key-and-dedup-policy
Show description for d162644
peterxcli
committed
d162644
View commit details
Copy full SHA for d162644
Browse repository at this point
fix: keep the map_from_arrays null-array guard so ANSI casts short-circuit
Show description for 8f54bdc
peterxcli
committed
8f54bdc
View commit details
Copy full SHA for 8f54bdc
Browse repository at this point
Commits on Sep 18, 2026
fix: nest the map_from_arrays null guards so a NULL keys array skips the values expression
Show description for 6d1d327
peterxcli
committed
6d1d327
View commit details
Copy full SHA for 6d1d327
Browse repository at this point
test: pin the entry order of a duplicate key under LAST_WIN
Show description for 855b039
peterxcli
committed
855b039
View commit details
Copy full SHA for 855b039
Browse repository at this point
test: cover a values array longer than the keys array in map_from_arrays
peterxcli
committed
101b2da
View commit details
Copy full SHA for 101b2da
Browse repository at this point
fix: decline a nondeterministic child of map_from_arrays before it reaches the null guards
Show description for 8c9ebdf
peterxcli
committed
8c9ebdf
View commit details
Copy full SHA for 8c9ebdf
Browse repository at this point
Commits on Sep 19, 2026
fix: capture the map dedup policy with the expression rather than per native iterator
Show description for 5ba4923
peterxcli
committed
5ba4923
View commit details
Copy full SHA for 5ba4923
Browse repository at this point
fix: read the map dedup policy when a plan is first executed, and keep it
Show description for 6962053
peterxcli
committed
6962053
View commit details
Copy full SHA for 6962053
Browse repository at this point
Commits on Sep 20, 2026
Merge remote-tracking branch 'upstream/main' into fix/map-null-key-and-dedup-policy
Show description for 0f5bfb0
peterxcli
committed
0f5bfb0
View commit details
Copy full SHA for 0f5bfb0
Browse repository at this point
fix: read the map dedup policy per native plan, matching Spark outside whole-stage codegen
Show description for 362f288
peterxcli
committed
362f288
View commit details
Copy full SHA for 362f288
Browse repository at this point
docs: use prettier emphasis style in the map_funcs audit
Show description for 88c877b
peterxcli
committed
88c877b
View commit details
Copy full SHA for 88c877b
Browse repository at this point
Commits on Sep 28, 2026
fix: report map builder errors in Spark's row order and trim the per-batch validation
Show description for 881b871
peterxcli
committed
881b871
View commit details
Copy full SHA for 881b871
Browse repository at this point
fix: guard only the keys argument of native map_from_arrays
Show description for acaaa6c
peterxcli
committed
acaaa6c
View commit details
Copy full SHA for acaaa6c
Browse repository at this point
test: re-run the same Dataset in the dedup policy test and trim duplicated map coverage
Show description for fa4b66b
peterxcli
committed
fa4b66b
View commit details
Copy full SHA for fa4b66b
Browse repository at this point
docs: keep the map constructor notes in one place in the map_funcs audit
Show description for 3044570
peterxcli
committed
3044570
View commit details
Copy full SHA for 3044570
Browse repository at this point
Commits on Oct 2, 2026
Merge remote-tracking branch 'upstream/main' into HEAD
Show description for 94770a1
peterxcli
committed
94770a1
View commit details
Copy full SHA for 94770a1
Browse repository at this point
You can’t perform that action at this time.