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

RC4 rebase master #2034

Merged
merged 37 commits into from
Apr 1, 2020
Merged

RC4 rebase master #2034

merged 37 commits into from
Apr 1, 2020

Conversation

jude-zhu
Copy link
Contributor

@jude-zhu jude-zhu commented Apr 1, 2020

No description provided.

dangleptr and others added 30 commits March 20, 2020 17:24
Co-authored-by: heng <heng.chen@vesoft.com>
Co-authored-by: yaphet <darion.wang@vesoft.com>
* Detect OS type and exit if not built in Linux

Signed-off-by: JaySon-Huang <jayson.hjs@gmail.com>

* Disable for FreeBSD

Signed-off-by: JaySon-Huang <jayson.hjs@gmail.com>
* alter user & change psw

* built in roles

* cbs comments

* fix comments

* update table

* addenable_authorize doc
* ttl doc modification

* fix comments

* yt comments

* silver days comments

* timestamp update
* Let action not fail fast.

* Disable the ci will product artifact.
Co-authored-by: heng <heng.chen@vesoft.com>
Co-authored-by: yaphet <darion.wang@vesoft.com>
…#1962)

* Fixed a bug caused by not filtering the versions of edges when executing "go from vid over *".

* address CPWstatic's comment.

* Modify the listEdges interface and add parameters to directly return all latest versions of edges.

* address laura's comment.

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
* modify job manger doc

* add flush

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
vesoft-inc#1973)

* Fix the read performace issue after delete range with rocksdb-5.15.10

* Fix the move issue

* fix the move issue

* Rebase and address bright-starry-sky's comments

Co-authored-by: heng <heng.chen@vesoft.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
…improve update rocksdb options (vesoft-inc#1959)

* open bloom filter in default conf, add event listener to collect stats of compaction

* imporve update rocksdb configs

* fix ut error

Co-authored-by: yaphet <darion.wang@vesoft.com>
Co-authored-by: heng <heng.chen@vesoft.com>
Co-authored-by: Doodle <13706157+critical27@users.noreply.github.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
…nder a space, (vesoft-inc#1983)

an unreasonable error message is returned (SpaceId `xxx 'is nonexistent)

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
* pr-commit-msg

* add examples

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
…inc#1943)

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
* Supports the detection keyword, name_label supports string

* add tests

* address Shylock's comment

* modify  to add reserved keyword

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Co-authored-by: heng <heng.chen@vesoft.com>
Co-authored-by: bright-starry-sky <56461666+bright-starry-sky@users.noreply.github.com>
* fixed loop depend problem

* fixed error for clang9 and gcc9

* Addressed laura-ding's comment
dangleptr and others added 6 commits March 31, 2020 14:59
* fix go without yield return errror result

* fix format

* address dangleptr's comment

* address issue change TTL_DURATION not negative

* add onlyStructure is true test case

Co-authored-by: bright-starry-sky <56461666+bright-starry-sky@users.noreply.github.com>
* When deleting a vertex, if the edge does not exist,
then we should continue to delete the vertex instead of reporting an error.

* added the testcase for delete vertex without edge.
* 1. meta client sub mit job only to leader 2. sort jobs in desc id order

* now show jobs return in desc order, adjust UT

Co-authored-by: dangleptr <37216992+dangleptr@users.noreply.github.com>
* upload oss

* update

* Remove redundant

* address @yixinlu's comment

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Copy link
Contributor

@laura-ding laura-ding left a comment

Choose a reason for hiding this comment

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

Well done

@jude-zhu jude-zhu added the ready-for-testing PR: ready for the CI test label Apr 1, 2020
@codecov-io
Copy link

Codecov Report

Merging #2034 into v1.0.0-rc4 will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           v1.0.0-rc4    #2034   +/-   ##
===========================================
  Coverage       86.72%   86.72%           
===========================================
  Files             641      641           
  Lines           61059    61059           
===========================================
  Hits            52951    52951           
  Misses           8108     8108           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7fa476e...7fa476e. Read the comment docs.

@dangleptr dangleptr merged commit 2ea95f3 into vesoft-inc:v1.0.0-rc4 Apr 1, 2020
@jude-zhu jude-zhu deleted the mergemaster branch March 23, 2021 09:12
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Jan 31, 2023
…ist exprs (vesoft-inc#2034)

* add some tests for predicate expr

* add more cases

* fix the bug of all/any/single/none expr

Co-authored-by: codesigner <codesigner.huang@vesoft.com>
Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>

Co-authored-by: jie.wang <38901892+jievince@users.noreply.github.com>
Co-authored-by: codesigner <codesigner.huang@vesoft.com>
Co-authored-by: Sophie <84560950+Sophie-Xie@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.