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

Databricks doc updates #395

Merged
merged 6 commits into from
Jul 22, 2020
Merged

Conversation

sameerz
Copy link
Collaborator

@sameerz sameerz commented Jul 21, 2020

  1. Please write a description in this text box of the changes that are being
    made.

Updated Getting Started on Databricks documentation. Will need to push to gh-pages after this.

  1. Please ensure that you have written units tests for the changes made/features
    added.

Tested locally with docker run --rm -it --volume="$PWD:/srv/jekyll" -p 4000:4000 sameerz/jekyll:0.1 jekyll serve

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz sameerz added the documentation Improvements or additions to documentation label Jul 21, 2020
@sameerz sameerz added this to the Jul 20 - Jul 31 milestone Jul 21, 2020
@sameerz sameerz self-assigned this Jul 21, 2020
@sameerz
Copy link
Collaborator Author

sameerz commented Jul 21, 2020

@mengdong can you take a look as well?

@sameerz
Copy link
Collaborator Author

sameerz commented Jul 21, 2020

build

Copy link
Contributor

@mengdong mengdong left a comment

Choose a reason for hiding this comment

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

Hi Sameer, I mainly want to suggest 2 things as commented.

krajendrannv
krajendrannv previously approved these changes Jul 21, 2020
@revans2
Copy link
Collaborator

revans2 commented Jul 21, 2020

build

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
Copy link
Collaborator Author

sameerz commented Jul 21, 2020

build

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
Copy link
Collaborator Author

sameerz commented Jul 22, 2020

build

@sameerz sameerz requested a review from tgravescs July 22, 2020 18:22
@sameerz sameerz merged commit 0ff857d into NVIDIA:branch-0.2 Jul 22, 2020
@sameerz sameerz deleted the databricks-doc-updates branch August 24, 2020 22:32
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Update Databricks documentation

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* replace tabs with spaces

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* (revert) make title text consistent with other getting started guides

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* Pointing to Spark and Plugin configuration pages

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* Add a Hints section on how to save logs on Databricks

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* fix typo

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Update Databricks documentation

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* replace tabs with spaces

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* (revert) make title text consistent with other getting started guides

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* Pointing to Spark and Plugin configuration pages

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* Add a Hints section on how to save logs on Databricks

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>

* fix typo

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants