Skip to content

CDK-886: Add instructions for replacing kite-dataset in /usr/bin #76

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.

Sign up for GitHub

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

Open
DennisDawson wants to merge 5 commits into
base: master
Choose a base branch
from
Open

CDK-886: Add instructions for replacing kite-dataset in /usr/bin #76

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
2c4f68e
Per CDK-886, add instructions for replacing kite-dataset in /usr/bin
Feb 23, 2015
9784cd0
Updates based on comments 2/23.
Feb 24, 2015
d84b15c
Updates based on feedback 3/5/2015.
Mar 5, 2015
264c770
Changed path to /usr/lib/kite/bin/ for packages.
Mar 5, 2015
3009b08
Update Install-Kite.md
Mar 28, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions Install-Kite.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,19 @@ chmod +x {{site.dataset-command}}
```

The first command downloads the Kite tools JAR to a local file named _{{site.dataset-command}}_. The second changes the access rights on the JAR so that the classes in the JAR are executable.

Some versions of the Cloudera QuickStart VM comes with Kite installed. However, it might not be the latest version. To check the version, from a terminal window, enter the following command.

```
kite-dataset --version
```

If the version number doesn't match the version you downloaded, you have two options.

@rdblue rdblue Mar 5, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These instructions download a specific version, so you can just state the version here. There's a chance that the user is reading an old version of this guide, so it is also a good time to point them at the latest, which is always at /docs/current/Install-Kite.html

@DennisDawson DennisDawson Mar 28, 2015

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This doesn't make sense to me in this context. These instructions are for verifying that the version installed is the version in use.


1. Include the path to `kite-dataset` when you invoke commands. For example, if the JAR file is in the current directory, use `./kite-dataset`.

1. Replace the installed version of `kite-dataset.jar`.

@rdblue rdblue Feb 23, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No need to use the jar file extension, this is usually called kite-dataset. Also, this is {{site.dataset-command}} above, should that be used here as well?

@DennisDawson DennisDawson Feb 24, 2015

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Replaced all occurrences with {{site.dataset-command}}

@DennisDawson DennisDawson Feb 24, 2015

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Line 15 - Some versions of the Cloudera QuickStart VM [come] with Kite installed. Changing this to "Some versions of CDH come with Kite installed."

a. Download `kite-dataset.jar` to the home directory.

@rdblue rdblue Feb 23, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No need to specify a target directory.

@DennisDawson DennisDawson Feb 24, 2015

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay, added more explanation. I don't want to use a variable name like / if I can avoid it. I just said "in this example, it is downloaded to the ~/ directory."

b. In a terminal window on the VM, navigate to `/usr/bin`.
c. Enter the command `sudo cp ~/kite-dataset ./kite-dataset`

@rdblue rdblue Feb 23, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Steps b and c should be combined into a single command.

This also replaces a wrapper script rather than the command, so we need to figure out how to replace the underlying file by having two sets of instructions: for packages and for parcels. The wrapper script sets up the java classpath so we don't want to overwrite it.

@rdblue rdblue Feb 23, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For packages, replace /usr/lib/kite/bin/kite-dataset. For parcels, it is /opt/cloudera/parcels//lib/kite/bin/kite-dataset.

@DennisDawson DennisDawson Feb 24, 2015

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

So, wait, is this a general CDH issue, and not something specific to the QuickStart VM?
I'm assuming this is the case, and updating the docs accordingly.

d. To verify that the correct version is in use, enter the command `kite-dataset --version`.