Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix issue #1033 - property access from entity in array #1034

Closed
wants to merge 1 commit into from
Closed

Fix issue #1033 - property access from entity in array #1034

wants to merge 1 commit into from

Conversation

MuhammadTahaNaveed
Copy link
Member

@MuhammadTahaNaveed MuhammadTahaNaveed commented Jul 13, 2023 •
edited
Loading

  • Issue was with agtype_access_operator. It was overseeing the possiilty of scalar types as element in array/object.
  • Added regression tests.

resolves #1033

@MuhammadTahaNaveed
Fix issue apache#1033 - property access from entity in array
9e9385f 
- Issue was with agtype_access_operator. It was overseeing the
  possiilty of scalar types as element in array/object.
- Added regression tests.
@MuhammadTahaNaveed MuhammadTahaNaveed mentioned this pull request Jul 13, 2023
Closed
@jrgemignani
Copy link
Contributor

Ooops. We both put out a PR for the same thing, lol

@jrgemignani jrgemignani mentioned this pull request Jul 14, 2023
Merged
@MuhammadTahaNaveed MuhammadTahaNaveed deleted the issue_1033 branch July 15, 2023 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrgemignani jrgemignani Awaiting requested review from jrgemignani

@dehowef dehowef Awaiting requested review from dehowef

@rafsun42 rafsun42 Awaiting requested review from rafsun42

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Property access from edge/vertex in array gives unexpected error
2 participants
@MuhammadTahaNaveed @jrgemignani