Skip to content

Conversation

@eevanwong
Copy link

Description

Extremely minor. Just wanted to update the running on dockers link as there was some confusion about the dockers link on slack.

I'm kind of confused with some of these changes because I did not touch them but apparently I deleted and re-added them.

* More example queries and sample data
* Find out ways to be involved or discuss Drill
* Remote Execution Installation Instructions
* [Running Drill on Docker instructions](docs/dev/Docker.md)
Copy link
Member

Choose a reason for hiding this comment

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

Initially, the link was correct, but for some reason, it became broken... Could you please instead fix the page for the Drill website? Here is the corresponding doc: https://github.com/apache/drill/blob/gh-pages/_docs/install/installing-drill-in-embedded-mode/011-running-drill-on-docker.md

We shouldn't use the relative link here since this README.md is also displayed in https://hub.docker.com/r/apache/drill and this link will be broken there.

Copy link
Author

@eevanwong eevanwong Feb 22, 2021

Choose a reason for hiding this comment

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

Ok got it. As this seems to be an issue related to another branch, so I'll have to close this ticket and then make another pr for the gh-pages.

@eevanwong eevanwong closed this Feb 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants