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] TTL cn #1934

Merged
merged 39 commits into from
Mar 17, 2020
Merged

[doc] TTL cn #1934

merged 39 commits into from
Mar 17, 2020

Conversation

jude-zhu
Copy link
Contributor

update ttl cn doc

critical27 and others added 30 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 changed the title [doc] Ttl cn [doc] TTL cn Mar 17, 2020
whitewum
whitewum previously approved these changes Mar 17, 2020
Copy link
Contributor

@whitewum whitewum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great. Please update the EN version as well

panda-sheep
panda-sheep previously approved these changes Mar 17, 2020
Copy link
Contributor

@panda-sheep panda-sheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent!

@jude-zhu jude-zhu dismissed stale reviews from panda-sheep and whitewum via 0dc0172 March 17, 2020 10:07
panda-sheep
panda-sheep previously approved these changes Mar 17, 2020
Copy link
Contributor

@panda-sheep panda-sheep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@laura-ding laura-ding added the ready-for-testing PR: ready for the CI test label Mar 17, 2020
@dangleptr dangleptr merged commit 7da052f into vesoft-inc:master Mar 17, 2020
@jude-zhu jude-zhu deleted the ttl branch June 10, 2020 06:46
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
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.