Adds a resource page for the jupyter video#152
Conversation
|
This pull request is automatically being deployed by Amplify Hosting (learn more). |
317brian
left a comment
There was a problem hiding this comment.
🐨 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.
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.
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.
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). |
| @@ -40,18 +40,15 @@ We're in the process of migrating this content. Check back soon. | |||
|
|
|||
| ## Set up your learning environment | |||
There was a problem hiding this comment.
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.