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

feat(interactive): Adapt to new unified schema #3725

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

zhanglei1949
Copy link
Collaborator

@zhanglei1949 zhanglei1949 commented Apr 18, 2024

Migrating the graph schema of Interactive to the Flex Unified graph schema.

Those changes are included in this PR.

  • Graph, Vertex and Edge can have descriptions.
  • Introduce version in schema, which denotes the version of the schema definition. If not given, v0.0 is used. The schema of version v0.0 is exactly the previous schema definition. Therefore, the schema of the graph used previously can continue to be used without any modifications.
  • default_value and nullable are not supported in interactive currently.
  • type_id and property_id shall not be provided by user, mutable_csr will generate the internal id instead.

@zhanglei1949 zhanglei1949 force-pushed the refine-schema branch 2 times, most recently from f10c2aa to 0b069d3 Compare April 24, 2024 02:12
@zhanglei1949 zhanglei1949 force-pushed the refine-schema branch 5 times, most recently from 88f0b5b to a972196 Compare April 25, 2024 06:37
@zhanglei1949 zhanglei1949 requested a review from liulx20 April 25, 2024 06:56
@zhanglei1949
Copy link
Collaborator Author

zhanglei1949 commented Apr 25, 2024

Currently we can not return vertex/edge types in stored procedure, so codegen of simple_match_10.cypher fails. We will support it in the future @shirly121 .

#3755

@zhanglei1949 zhanglei1949 changed the title feat(interactive): Migrate schema to new unified definition feat(interactive): Adapt to new unified schema Apr 25, 2024
Support parsing the data type from the new unified type definition.

```yaml
GS_DATA_TYPE:  # choose one
  # optional value: DT_SIGNED_INT32, DT_UNSIGNED_INT32, DT_SIGNED_INT64
  #                 DT_UNSIGNED_INT64, DT_BOOL, DT_FLOAT, DT_DOUBLE
  primitive_type:
  string:  # choose one
    long_text: # string with unlimited length
    var_char:  # string with variable length, bounded by max_length
      max_length: <uint32>
  temporal: # choose one
    date32:
    timestamp:
```

Also compatible with previous schema files.

migrate schema to new unified schema def

fix format and add default_value and nullable check

minor

fix

todo: fix-ci

[GIE Compiler] support new format of flex schema in compiler

minor fix

minor fix

minor

revert unneccessary changes

fix

remove unused ci

fix ci

fix

fix

add description

minor fix

minor fix

fix

fix codegen

minor fix

fixing ci
@zhanglei1949 zhanglei1949 merged commit b28ece8 into alibaba:main Apr 26, 2024
58 checks passed
@zhanglei1949 zhanglei1949 deleted the refine-schema branch July 5, 2024 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants