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

Enhance Spark Writer #1512

Merged
merged 9 commits into from
Jan 14, 2020
Merged

Enhance Spark Writer #1512

merged 9 commits into from
Jan 14, 2020

Conversation

darionyaphet
Copy link
Contributor

@darionyaphet darionyaphet commented Dec 25, 2019

  1. Currently we are using schema name for the config's key. But actually some user will loading data into the same schema from different data source. So I have modified the config's structure to support it.

  2. Support error log save into separated HDFS file and reload.

  3. Support rate limit in spark writer

dangleptr
dangleptr previously approved these changes Dec 25, 2019
Copy link
Contributor

@dangleptr dangleptr 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

Conflicts:
	src/tools/spark-sstfile-generator/pom.xml
	src/tools/spark-sstfile-generator/src/main/scala/com/vesoft/nebula/tools/generator/v2/SparkClientGenerator.scala
@darionyaphet darionyaphet added ready-for-testing PR: ready for the CI test and removed in progress labels Jan 2, 2020
@nebula-community-bot
Copy link
Member

Unit testing passed.

@darionyaphet darionyaphet changed the title Support multi tag or edge with the same name Enhance Spark Writer Jan 2, 2020
@nebula-community-bot
Copy link
Member

Unit testing passed.

@jude-zhu jude-zhu requested a review from dangleptr January 3, 2020 06:49
@nebula-community-bot
Copy link
Member

Unit testing passed.

…into spark-config

Conflicts:
	src/tools/spark-sstfile-generator/src/main/resources/application.conf
	src/tools/spark-sstfile-generator/src/main/scala/com/vesoft/nebula/tools/generator/v2/SparkClientGenerator.scala
@darionyaphet
Copy link
Contributor Author

TODO : I will update the document later :D

@@ -12,7 +12,7 @@

<groupId>com.vesoft</groupId>
<artifactId>nebula-utils</artifactId>
<version>1.0.0-rc2</version>
<version>1.0.0-rc3</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Have we uploaded rc3 version jni? rc2 is ok I guess

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I understand that RC3 will release later so I update this version number.
Actually, when RC2 release we should update this version.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, remember to upload it to mvn

@jude-zhu jude-zhu requested a review from critical27 January 10, 2020 05:52
@dangleptr dangleptr merged commit a9324ae into vesoft-inc:master Jan 14, 2020
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
* change config to list

* change config to list

Co-authored-by: dangleptr <37216992+dangleptr@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.

5 participants