Skip to content

Conversation

@1ambda
Copy link
Member

@1ambda 1ambda commented Nov 11, 2016

What is this PR for?

Add 2 JDBC configs and useful links to JDBC doc so that users can connect those backends easily :)

What type of PR is it?

Documentation

Todos

  • - Task

What is the Jira issue?

ZEPPELIN-1650

How should this be tested?

See screenshots.

Screenshots (if appropriate)

example_redshift

example_mariadb

Questions:

  • Does the licenses files need update? - NO
  • Is there breaking changes for older versions? - NO
  • Does this needs documentation? - NO

@1ambda
Copy link
Member Author

1ambda commented Nov 11, 2016

I wanted to add drill settings in this PR but failed to test it. Anyone recently tried drill (1.8) using zeppelin 0.7-SNAPSHOT?

@AhyoungRyu
Copy link
Contributor

@1ambda Thanks for updating JDBC docs. I added more JDBC Driver docs & Maven repo urls for other examples as you did. Could you check 1ambda#1? :)

@1ambda
Copy link
Member Author

1ambda commented Nov 12, 2016

@AhyoungRyu Thanks :)

I merged it. Additionally, added a postgres section so that we can remove seperated postgres doc easily next time :)

Don't make users confused when seperated postgre doc is removed
@1ambda 1ambda closed this Nov 14, 2016
@1ambda 1ambda reopened this Nov 14, 2016
@1ambda 1ambda closed this Nov 14, 2016
@1ambda 1ambda reopened this Nov 14, 2016
@1ambda 1ambda closed this Nov 14, 2016
@1ambda 1ambda reopened this Nov 14, 2016
@1ambda
Copy link
Member Author

1ambda commented Nov 14, 2016

@AhyoungRyu Updated postgres image :) Thanks again.

CI is failing but it doens't related this PR i think.

@AhyoungRyu
Copy link
Contributor

@1ambda Yeah the changes are looks good to me. Thanks again for taking care of the docs. Definitely will help the JDBC users.
Merge if there are no more discussions.

@asfgit asfgit closed this in 9bf56d9 Nov 14, 2016
tae-jun pushed a commit to tae-jun/zeppelin that referenced this pull request Nov 23, 2016
…C doc

### What is this PR for?

Add 2 JDBC configs and useful links to JDBC doc so that users can connect those backends easily :)

### What type of PR is it?

Documentation

### Todos
* [ ] - Task

### What is the Jira issue?

[ZEPPELIN-1650](https://issues.apache.org/jira/browse/ZEPPELIN-1650)

### How should this be tested?

See screenshots.

### Screenshots (if appropriate)

<img width="706" alt="example_redshift" src="https://cloud.githubusercontent.com/assets/4968473/20208653/01b1e468-a833-11e6-9166-b05d15e9a5fe.png">
<img width="577" alt="example_mariadb" src="https://cloud.githubusercontent.com/assets/4968473/20208654/02b49a22-a833-11e6-9b29-5a61f2b6a4bb.png">

### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO

Author: 1ambda <[email protected]>
Author: AhyoungRyu <[email protected]>

Closes apache#1621 from 1ambda/jdbc-doc-improve and squashes the following commits:

0aab878 [1ambda] docs: Update postgres img ;)
6009eb9 [1ambda] docs: Add postgres section, hive2 doc link
ce24af1 [AhyoungRyu] Make mariadb and redshift screenshots same with others
2fb5db1 [1ambda] docs: Add mariadb, redshift configs
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