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
[CARBONDATA-4267][Doc][summer-2021]Update and modify some content in quick-start-guide.md #4197
Conversation
Modify minor errors and correct some misunderstandings in the document
Can one of the admins verify this patch? |
add to whitelist |
docs/quick-start-guide.md
Outdated
**NOTE** | ||
- In this flow, we can use the built-in SparkSession `spark` instead of `carbon`. | ||
We also can create a new SparkSession instead of the built-in SparkSession `spark` if need. | ||
It need to add "org.apache.spark.sql.CarbonExtensions" into spark configuration "spark.sql.extensions". | ||
``` | ||
SparkSession newSpark = SparkSession | ||
val newSpark : SparkSession = SparkSession |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why need to change like this ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_2.4.5/4077/ |
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12602/job/ApacheCarbonPRBuilder2.3/5822/ |
Build Success with Spark 3.1, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_3.1/225/ |
Build Failed with Spark 2.3.4, Please check CI http://121.244.95.60:12602/job/ApacheCarbonPRBuilder2.3/5827/ |
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_2.4.5/4082/ |
Build Success with Spark 3.1, Please check CI http://121.244.95.60:12602/job/ApacheCarbon_PR_Builder_3.1/230/ |
LGTM |
Modify minor errors and correct some misunderstandings in the document
Why is this PR needed?
Modify small errors in the document and add some instructions to improve the readability of the document.For example,
1、The local file path of '/local-path/sample.csv' cannot be recognized
2、There some minor errors in the code of 'Option 2: Using SparkSession with CarbonExtensions(since 2.0)'
3、Repeated comments between different parts of the document are added to improve the readability of the document
What changes were proposed in this PR?
Modified some errors in the code and added some necessary instructions
Does this PR introduce any user interface change?
Is any new testcase added?
JIRA Issue: https://issues.apache.org/jira/browse/CARBONDATA-4267