-
Notifications
You must be signed in to change notification settings - Fork 240
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
Correct 21.10 docs such as PCBS related FAQ [skip ci] #3815
Merged
Merged
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
17fc362
Signed-off-by: Hao Zhu <hazhu@nvidia.com>
viadea e7ca540
Signed-off-by: Hao Zhu <hazhu@nvidia.com>
viadea 8ca2570
Signed-off-by: Hao Zhu <hazhu@nvidia.com>
viadea ae6952e
Update docs/download.md
viadea 5a7f962
Update docs/download.md
viadea File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -10,7 +10,7 @@ nav_order: 11 | |||||||||||||||
|
||||||||||||||||
### What versions of Apache Spark does the RAPIDS Accelerator for Apache Spark support? | ||||||||||||||||
|
||||||||||||||||
The RAPIDS Accelerator for Apache Spark requires version 3.0.1, 3.0.2, 3.0.3, 3.1.1, or 3.1.2 of | ||||||||||||||||
The RAPIDS Accelerator for Apache Spark requires version 3.0.1, 3.0.2, 3.0.3, 3.1.1, 3.1.2 or 3.2.0 of | ||||||||||||||||
Apache Spark. Because the plugin replaces parts of the physical plan that Apache Spark considers to | ||||||||||||||||
be internal the code for those plans can change even between bug fix releases. As a part of our | ||||||||||||||||
process, we try to stay on top of these changes and release updates as quickly as possible. | ||||||||||||||||
|
@@ -287,13 +287,15 @@ AdaptiveSparkPlan isFinalPlan=false | |||||||||||||||
|
||||||||||||||||
### Are cache and persist supported? | ||||||||||||||||
|
||||||||||||||||
Yes cache and persist are supported, but they are not GPU accelerated yet. We are working with | ||||||||||||||||
the Spark community on changes that would allow us to accelerate compression when caching data. | ||||||||||||||||
Yes cache and persist are supported, the cache is GPU accelerated | ||||||||||||||||
but still stored on the host memory. | ||||||||||||||||
Please refer to [RAPIDS Cache Serializer](./additional-functionality/cache-serializer.md) | ||||||||||||||||
for more details. | ||||||||||||||||
Comment on lines
+290
to
+293
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @viadea I think abellina means the indenting for the text in the editor. Try the following:
Suggested change
|
||||||||||||||||
|
||||||||||||||||
### Can I cache data into GPU memory? | ||||||||||||||||
|
||||||||||||||||
No, that is not currently supported. It would require much larger changes to Apache Spark to be able | ||||||||||||||||
to support this. | ||||||||||||||||
No, that is not currently supported. | ||||||||||||||||
It would require much larger changes to Apache Spark to be able to support this. | ||||||||||||||||
|
||||||||||||||||
### Is PySpark supported? | ||||||||||||||||
|
||||||||||||||||
|
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
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
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.
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.
indentation is a little off here.
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.
@abellina i think the indentation is correct, right? I mean the indentation before "Yes" looks good to me.
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.
+1 to what @sameerz said. It's a minor nit, but it is an odd break in this line.