incubator-hugegraph icon indicating copy to clipboard operation
incubator-hugegraph copied to clipboard

[Feature] New support for gte(text) syntax

Open llooFlashooll opened this issue 2 years ago • 2 comments

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.

llooFlashooll avatar Mar 10 '23 14:03 llooFlashooll

Dear developers, during my continuous testing. The features should also extend to gt(), gte(), lt(), lte(). Those syntaxes also don't support string values

llooFlashooll avatar Mar 16 '23 02:03 llooFlashooll

Dear developers, during my continuous testing. The features should also extend to gt(), gte(), lt(), lte(). Those syntaxes also don't support string values

Thanks, u could also edit it in original comment

image

imbajin avatar Mar 16 '23 05:03 imbajin