Skip to content
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 Netezza emulator instructions for v7.2 #590

Merged
merged 1 commit into from
Jun 17, 2017

Conversation

MMatten
Copy link
Contributor

@MMatten MMatten commented Jun 17, 2017

Updates to the Netezza emulator setup instructions for v7.2.

NETEZZA.md Outdated

7. Run the tests ./gradlew :dbfit-java:netezza:test
4. Run the Netezza DbFit acceptance tests with 'gradlew :dbfit-java:netezza:integrationTest'
Copy link
Contributor

Choose a reason for hiding this comment

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

These could be gradlew :dbfit-java:netezza:integrationTest instead of 'gradlew :dbfit-java:netezza:integrationTest' - for a bit fancier formatting.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, good spot.

@MMatten MMatten force-pushed the update-netezza-emulator-instructions branch from ab85a70 to e99b48d Compare June 17, 2017 17:15
@MMatten
Copy link
Contributor Author

MMatten commented Jun 17, 2017

Updated.

NETEZZA.md Outdated

7. Run the tests ./gradlew :dbfit-java:netezza:test
4. Run the Netezza DbFit acceptance tests with `gradlew :dbfit-java:netezza:integrationTest`.
Copy link
Contributor

Choose a reason for hiding this comment

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

In one of the examples we're calling it via ./gradlew ..., in the other - gradlew .... That's maybe OK.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good spot, corrected.

NETEZZA.md Outdated
* Install the JDBC driver. This creates a seperate directory which contains the nzjdbc.jar needed.
* Optionally install the Netezza Administrator GUI (makes life a lot easier)
**NOTE:** the default passwords for the emulator are nz/nz for the linux host and admin/password for the database connection
**NOTE:** the default passwords for the emulator are `nz`/`nz` for the Linux host and `admin`/`password` for the database connection
Copy link
Contributor

Choose a reason for hiding this comment

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

I think these NOTEs are rendered on a single line (actually in the the same paragraph where the last bullet started).

Looks like Github Markdown rendering doesn't support advanced formatting. Some options I can think of are:

  • Point A

    • NOTE: Blah blah 1
    • NOTE: Blah blah 2
  • Point B

    NOTE: Blah blah 3
    NOTE: Blah blah 4

  • Point C

    NOTE: Blah blah 5

    NOTE: Blah blah 6

@MMatten MMatten force-pushed the update-netezza-emulator-instructions branch from e99b48d to bb81aea Compare June 17, 2017 22:47
@javornikolov
Copy link
Contributor

It looks good 👍 Merging...

@javornikolov javornikolov added this to the Next milestone Jun 17, 2017
@javornikolov javornikolov merged commit 65c0b80 into master Jun 17, 2017
@javornikolov javornikolov deleted the update-netezza-emulator-instructions branch June 17, 2017 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants