Add support for array operators for postgres dialect

pietermartin / sqlg

TinkerPop graph over sql

MIT License

246 stars 51 forks source link

Add support for array operators for postgres dialect #360

Closed vermaharsh closed 5 years ago

vermaharsh commented 5 years ago

add array operator support
Optimize Gremlin step with ArrayContains and ArrayOverlaps predicates
Add tests

vermaharsh commented 5 years ago

@pietermartin could you please review this change and accept in mainline?

This adds support for using array operators @> and && which are performant when working with a large list of integers.

pietermartin commented 5 years ago

Thanks for the PR, looks good. I'll review it properly on the weekend. Can you explain the need for the tableau repository changes you made to the root pom?

vermaharsh commented 5 years ago

Can you explain the need for the tableau repository changes you made to the root pom?

It is for easily publishing jars to a repository that my team uses internally. I can take it out and only keep in the fork that we maintain. Let me know.

pietermartin commented 5 years ago

Yeah I think rather keep it in your local branch. Thanks

vermaharsh commented 5 years ago

Done. Removed the tableau artifactory specific changes.