apache / apache/hugegraph

[Feature] New support for gte(text) syntax

Open
#2,159 2 comments 0 reactions 0 assignees View on GitHub
feature
Dominant language
Java
Stars
3.2k
Forks
636
Avg merge
3d 11h
Merged PRs (30d)
14

Description

### Feature Description (功能描述)

Here is the related issue #2152 to add this new feature.

Let me introduce the problem here.
When I run the following query: `g.V().has('vp0', not(not(gte('iUn2s').or(gte('730618742'))))).out().path()`. An exception is thrown without further message to identify the root cause. I think this query is syntactically correct, but I keep triggering this kind of problem.

- Expected behavior:
No exception should be expected to throw. Or further messages or prompts should be thrown.

- Actual behavior:
A `java.lang.NumberFormatException` is thrown.
```
HugeGraph exception:
java.lang.NumberFormatException: Error encountered evaluating script: g.V().has('vp0', not(not(gte('iUn2s').or(gte('730618742'))))).out().path()
```

- Feature to add:
HugeGraph does not support `gte('iUn2s')`, only `gte(number)` is allowed. But in other well-known GraphDBs, `gt()`, `gte()`, `lt()`, `lte()` do support other data types such as string values.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.