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

[doc] cypher v.s. ngql add query result #1862

Merged
merged 30 commits into from
Mar 6, 2020
Merged

Conversation

jude-zhu
Copy link
Contributor

@jude-zhu jude-zhu commented Mar 4, 2020

as title

critical27 and others added 29 commits January 20, 2020 15:59
* fix meta would crash in multiple replica when build plan failed

* add error log
Implement the following features:

 * space level
create space my_space (partition_num = 10, replica_factor = 1, charset = utf8, collate = utf8_bin);
create space my_space (partition_num = 10, replica_factor = 1, charset = utf8);
create space my_space (partition_num = 10, replica_factor = 1, collate = utf8_bin);
create space my_space (partition_num = 10, replica_factor = 1);
show create space my_space;

desc space my_space;

 * show charset;

 * show collation;
* Lookup benchmark

* To optimize the lookup benckmark

* all compact before index can

* improve benchmark

* Updated comments

Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
* group-by statement is updated

* format updated

* minor update

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
* Support bidirectional traverse.

* Add basic bidirectional traverse test.

* Fix ut.

* Add ut for multi edge traverse.

* Fix clang compile error.

* Address comment.

* Adddress @dangleptr's comment.

* Address comment.

Co-authored-by: laura-ding <48548375+laura-ding@users.noreply.github.com>
Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
* modify block cache

* fix comment

* wum

Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
Fixed an error that the space could not be found when executing multiple statements (use space statements)
when executing update vertex/edge.

for example:

USE issue1827_update;UPDATE VERTEX 100 SET player.age = 31;
* add bidirect

* add example bidirect

* add semicolon

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
* delete vertices

* address laura-ding's comment

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
* Delete DCHECK in SchemaWriter::appendCol and refactor the part of col_types genaration..

* Return error when get rows failed from interim result.

* Refactored the feedResult part.

* Check if inputs has duplicate column.

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
 * Add a return_partly in get, when it is set to true, we can get the keys which exist, and ignore those keys which are not existed.
 * To implement this, add a tryGet method in KVStore, which return each status of key.
 * Remove async thread in GetProcessor
@jude-zhu jude-zhu requested a review from wilsonyou March 4, 2020 06:10
@jude-zhu jude-zhu requested review from amber-moe and whitewum March 4, 2020 06:10
@jude-zhu jude-zhu added the ready-for-testing PR: ready for the CI test label Mar 4, 2020
@whitewum whitewum changed the title [doc] cyphe v.s. ngql add query result [doc] cypher v.s. ngql add query result Mar 4, 2020
@dangleptr dangleptr merged commit 7a068f9 into vesoft-inc:master Mar 6, 2020
@jude-zhu jude-zhu deleted the world branch June 10, 2020 06:47
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
yixinglu added a commit to yixinglu/nebula that referenced this pull request Jan 31, 2023
* refine AliasType deducing when planning

* resolve conflicts

Co-authored-by: codesigner <codesigner.huang@vesoft.com>
Co-authored-by: Yee <2520865+yixinglu@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants