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.

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

NullPointerException on simple queries #117

Open
bblonski opened this issue Jan 18, 2017 · 4 comments
Open

NullPointerException on simple queries #117

bblonski opened this issue Jan 18, 2017 · 4 comments
Assignees
Labels
Milestone

Comments

@bblonski
Copy link

I have several times run into a NullPointerException on OrientEdge:48. The rawDocument comes back null and throws the NPE on rawDocument.getClassName(). I'm not sure what's causing the issue. Inspecting the database shows no issues and the queries will work for a time until suddenly they do not and will consistently throw NPEs until the database is reset.

Version is v3.2.3.0 against OrientDB server v2.2.10.

@lvca lvca added this to the 2.2.x milestone Jan 18, 2017
@lvca lvca added the bug label Jan 18, 2017
@bblonski
Copy link
Author

Further investigation seems to indicate the issue occurs when records are removed from the database. The traversal still tries to iterate through missing vertices which causes the null pointers. I've tracked it back farther to OrientVertex:65. The call to .rawIterator() appears to return deleted vertices. Perhaps the call should be to .iterator() instead?

@ken-at-sri
Copy link

ken-at-sri commented Jan 19, 2017

In addition to calling .iterator() as bblonski suggested, I added a filter to ignore any nulls while streaming over the collection. This appears to have fixed the issue we were having.

@luigidellaquila
Copy link
Member

Hi @bblonski

Is this issue still valid? I checked the code but too many things have changed, so the patch from @ken-at-sri cannot be applied anymore

Thanks

Luigi

@luigidellaquila
Copy link
Member

Ops, sorry, checked the wrong implementation (TinkerPop2), assigning to @maggiolo00 for a check

Thanks

Luigi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants