Use async client in Spark writer #1405
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
darionyaphet
added
in progress
ready-for-testing
|
Unit testing passed. |
darionyaphet
requested review from
bright-starry-sky,
CPWstatic,
critical27,
dangleptr,
dutor,
jude-zhu,
liuyu85cn,
wfystx and
whitewum
|
Unit testing passed. |
1 similar comment
|
Unit testing passed. |
|
Unit testing passed. |
critical27
approved these changes
Dec 31, 2019
|
Unit testing passed. |
|
close #1415 |
tong-hao
pushed a commit
to tong-hao/nebula
that referenced
this pull request
Jun 1, 2021
yixinglu
pushed a commit
to yixinglu/nebula
that referenced
this pull request
Jan 31, 2023
<!-- Thanks for your contribution! In order to review PR more efficiently, please add information according to the template. --> ## What type of PR is this? - [ ] bug - [ ] feature - [X] enhancement ## What problem(s) does this PR solve? #### Issue(s) number: #### Description: PRD of vertex key changed again... Here is the new version of logic: 1. No matter use_vertex_key is on or off, the query result should be consistent with 3.1. 2. If use_vertex_key is on, we will use the vertex key to detect whether the vertex exists, which is the logic in 3.X version. 3. If use_vertex_key is off, we will prefix scan by vertex to detect whether the vertex exists, **which will bring performance regression** 4. We will insert the vertex only when use_vertex_key is on > By default, use_vertex_key is off ## How do you solve it? This PR reverts all related changes before (vesoft-inc#4685 vesoft-inc#4680 vesoft-inc#4675 vesoft-inc#4629), and apply the logic above. Related changes: 1. So the upgrade tool is necessary again, so bring it back. 2. The TCK cases keeps same as before vesoft-inc#4629 ## Special notes for your reviewer, ex. impact of this fix, design document, etc: > Maybe review the last commit is enough ## Checklist: Tests: - [X] Unit test(positive and negative cases) - [ ] Function test - [ ] Performance test - [ ] N/A Affects: - [X] Documentation affected (Please add the label if documentation needs to be modified.) - [ ] Incompatibility (If it breaks the compatibility, please describe it and add the label.） - [ ] If it's needed to cherry-pick (If cherry-pick to some branches is required, please label the destination version(s).) - [ ] Performance impacted: Consumes more CPU/Memory ## Release notes: Please confirm whether to be reflected in release notes and how to describe: > ex. Fixed the bug ..... Migrated from vesoft-inc#4716 Co-authored-by: Doodle <13706157+critical27@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.