-
Notifications
You must be signed in to change notification settings - Fork 6k
Update latest versions in documentation #21247
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
Update latest versions in documentation #21247
Conversation
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.
Nice job! Added a few comments.
You should now be able to access any Azure Databricks clusters you create and upload files to the DBFS. | ||
|
||
## Download worker dependencies | ||
## Download Microsoft Spark Worker dependencies |
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.
Leave as is or find shorter version for subheading. Formatting style guidelines ask for keeping headings to 2 lines (https://docs.microsoft.com/en-us/style-guide/scannable-content/headings#formatting-headings)
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.
I have reverted this change although I am not sure I understand how the previous change offends the guidelines asking to keep headings to 2 lines, there was just one line to the heading no? Perhaps I am missing something.
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.
@luisquintanilla Thank you so much for the review, I have addressed all your comments, please let me know if the changes look ok. Thanks! |
Thanks @Niharikadutta for making all the changes. Looks good to me. Approving. |
Updating following in .NET for Apache Spark documentation:
Fixes Spark .NET issue
Preview URLs: