[DB] Support SQL Field Query in Alcor

futurewei-cloud / alcor

Alcor: Cloud native SDN platform powered by Kubernetes and Istio

MIT License

32 stars 33 forks source link

[DB] Support SQL Field Query in Alcor #703

Closed pkommoju closed 2 years ago

pkommoju commented 2 years ago

Ignite SqlFieldQuery changes.

Ignite allows fast searching of the KV store using a field (secondary key/index) in the value field through SQL interface.
This change is almost entirely in the commonlib (Ignite interface layer).
To enable this feature, add the annotation @QuerySqlField(index = true) to the field to index on.
When a class member is annotated with @QuerySqlField(index = true), a get() using filterParams will use SQLFieldsQuery if all the filter params are also in the set of QuerySqlFields.
If fields m1, m2, and m3 are in the QuerySqlField set, in that order, and thet get supplies only m1, or m1 and m2, or all three, then Ignite interface will use SqlFieldsQuery, otherwise it will fallback to Scan Query. This is called index prefix search.
One optimization not in this commit is to cache the queries and reuse them. If needed, this optimization can be added later.
For an example use, look at web...NodeInfo class. and services/sqlquery and services/scanquery.

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging 78ad65068e02d844ad4589f8d858e0be947cb4d8 into 554092a76b15dd72db356f961a8fbdb49af5bf81 - view on LGTM.com

new alerts:

2 for Dereferenced variable may be null

xieus commented 2 years ago

@pkommoju There is a conflict with existing master after PR #704 was merged.

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging 5a1ea86074cbadc8a87c329a4b10f1e0b25f88c5 into a8436227f3d3d459efbfbff0b9a5195c7ea07e5d - view on LGTM.com

new alerts:

2 for Dereferenced variable may be null

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging af58b497e971604ead94d2e4d4064dd5da60036e into a8436227f3d3d459efbfbff0b9a5195c7ea07e5d - view on LGTM.com

new alerts:

2 for Dereferenced variable may be null

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging 02f04a50df336f764bf1821f559b540185d75151 into a8436227f3d3d459efbfbff0b9a5195c7ea07e5d - view on LGTM.com

new alerts:

2 for Dereferenced variable may be null

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging eb19113bdef0cf9313741f6372dcdb998f8562b8 into a8436227f3d3d459efbfbff0b9a5195c7ea07e5d - view on LGTM.com

new alerts:

2 for Dereferenced variable may be null

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging 3cf7110cbeb759076f02b8d77f4ad5d13b815c09 into a8436227f3d3d459efbfbff0b9a5195c7ea07e5d - view on LGTM.com

new alerts:

2 for Dereferenced variable may be null