-
Notifications
You must be signed in to change notification settings - Fork 15
Adds a resource page for the jupyter video #152
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
base: main
Are you sure you want to change the base?
Conversation
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
317brian
left a comment
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.
🐨 Something not touched in this PR: https://pr-152.d57o9tjhbibhd.amplifyapp.com/courses/druid-basics/#get-started-with-druid
But should we update that section to reference the notebooks (at least as an option if not the preferred way)
| Metrics play an important role in day-to-day operations, whether it’s monitoring the average "segment scan time" for a table or the 98th percentile query response time. Setting up and researching Druid's metrics is critical for all administrators. | ||
|
|
||
| Run through the [metrics notebook](https://github.com/implydata/learn-druid/blob/main/notebooks/05-operations/03-metrics-configuration.ipynb) from the [learn-druid](https://github.com/implydata/learn-druid) repository. | ||
| Run through the [metrics notebook](/resources/blob/main/notebooks/05-operations/03-metrics-configuration.ipynb) from the [Learn Druid developer environment](/courses/resources). |
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 are these set to /resources/? I think the notebook links need to remain the Github URL
Applies to all the instances of this
| Metrics play an important role in day-to-day operations, whether it’s monitoring the average "segment scan time" for a table or the 98th percentile query response time. Setting up and researching Druid's metrics is critical for all administrators. | ||
|
|
||
| Run through the [metrics notebook](https://github.com/implydata/learn-druid/blob/main/notebooks/05-operations/03-metrics-configuration.ipynb) from the [learn-druid](https://github.com/implydata/learn-druid) repository. | ||
| Run through the [metrics notebook](/resources/blob/main/notebooks/05-operations/03-metrics-configuration.ipynb) from the [Learn Druid developer environment](/courses/resources). |
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.
Some link checkers will fail without the .md, so it's better ot just always include it. Applies to all instances of this
| Run through the [metrics notebook](/resources/blob/main/notebooks/05-operations/03-metrics-configuration.ipynb) from the [Learn Druid developer environment](/courses/resources). | |
| Run through the [metrics notebook](/resources/blob/main/notebooks/05-operations/03-metrics-configuration.ipynb) from the [Learn Druid developer environment](/courses/resources.md). |
| <YouTubePlayer videoId="q6r0d_wQ6Do" /> | ||
| <br/> | ||
|
|
||
| For more information see the [Learn Druid repository](https://github.com/implydata/learn-druid)). |
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.
Extra )
| For more information see the [Learn Druid repository](https://github.com/implydata/learn-druid)). | |
| For more information see the [Learn Druid repository](https://github.com/implydata/learn-druid). |
| <!--TBD UPDATE FINAL VIDEO--> | ||
|
|
||
|
|
||
| ## Set up your learning environment |
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.
Should we add this to the index page of every course? Or at least a line that says "If you haven't set up your learning environment yet, go here."
I think it only exists in the data modeling course atm
Create a resource page "Learn Druid developer environment" with the video and context around using the notebooks instead of always linking to the repo directly.