-
Notifications
You must be signed in to change notification settings - Fork 2.3k
docs: Add changelog entry for JSON_EXTRACT dynamic path arguments #19045
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
docs: Add changelog entry for JSON_EXTRACT dynamic path arguments #19045
Conversation
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
|
🔧 I noticed that some CI checks failed for this PR. I'm investigating the failures and will work on fixing them. |
|
Promptless escalated the following message to you. If you respond, please remember to tag Promptless: I've investigated the CI failure on PR #19045. The issue is that the PR is missing a required "Component:" label. The fix: Please add the label "Component: Evalengine" to PR #19045. This is the appropriate label since the documentation covers changes to Once the label is added, the CI check should pass. Unfortunately, my GitHub integration has read-only access and cannot add labels automatically, so I need you to add it manually. You can add the label by:
|
|
🔧 I noticed that some CI checks failed for this PR. I'm investigating the failures and will work on fixing them. |
|
Promptless escalated the following message to you. If you respond, please remember to tag Promptless: I've investigated the CI failure on PR #19045. The issue is that the PR is missing two required labels: Required labels to add:
How to fix:
Once both labels are added, the CI check should pass. My GitHub integration has read-only access and cannot add labels automatically, so I need you to add them manually. Let me know if you need any help! |
|
Just a reminder: If you'd like me to act on any feedback you have via Github comments, just type @Promptless in your suggestion and I'll get right on it! (I won't show up in the user dropdown, but I'll process any request that has @Promptless in the comment body.) |
Documents PR #19035, which enhances
JSON_EXTRACTto support dynamic path arguments (bind variables, function results) instead of only static string literals. Also improves null handling to match MySQL behavior. Static path arguments remain optimized when mixed with dynamic ones.Trigger Events
JSON_EXTRACTwork with non-static argumentsOpen this suggestion in the Promptless dashboard
Help us improve Promptless — If this suggestion missed the mark, please share quick feedback.
If you want Promptless to make further changes on this PR, feel free to leave a comment tagging Promptless (It won't show up in the user drop down but Promptless will get it!)