SAY-5 commented on PR #2400: URL: https://github.com/apache/age/pull/2400#issuecomment-4284550857
Thanks @MuhammadTahaNaveed, apologies for the noisy diff. The two Cypher-level cases (`[1,2,3][1..null]` / `[null..2]`) actually exercise a different code path from `agtype_access_slice` and that path still treats AGTV_NULL as an omitted bound — so my C change doesn't cover them and the expected output didn't match. I've dropped those regression cases in the latest push, keeping only the two direct `agtype_access_slice` calls that actually hit the fixed code. Happy to follow up with a separate PR for the list-indexing path once I trace the execution plan side. I also used an AI coding assistant (Claude Code) while iterating on this — noted in the new commit message. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
