Skip to content

Allow indexed access in standalone AQL #21739

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

Draft
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

Simran-B
Copy link
Contributor

Scope & Purpose

The [] operator to get a specific array element or object key is disabled for Computed Values and the aql Analyzer.
It might be an oversight, or it was purposefully disabled for the aql Analyzer (does it somehow interfere with the position tracking?) but it is certainly unintended for Computed Values. To be safe, we could only allow it for Computed Values, or investigate whether something breaks if it's enabled for both.

  • 🍕 New feature

Checklist

  • Tests
    • Regression tests
    • C++ Unit tests
    • integration tests
    • resilience tests
  • 📖 CHANGELOG entry made
  • 📚 documentation written (release notes, API changes, ...)
  • Backports
    • Backport for 3.11: (Please link PR)

Related Information

(Please reference tickets / specification / other PRs etc)

  • Docs PR:
  • Enterprise PR:
  • GitHub issue / Jira ticket:
  • Design document:

@Simran-B Simran-B added this to the 3.12 milestone Apr 29, 2025
@Simran-B Simran-B self-assigned this Apr 29, 2025
@cla-bot cla-bot bot added the cla-signed label Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant