### What is this PR for?
To increase logging in the application, so as it
### What type of PR is it?
Refactoring
### Todos
* [x] - replace all printStackTrace() to LOGGER.error
* [x] - all catch block that doesn't throw exception, should have logging
### Is there a relevant Jira issue?
Yes, ZEPPELIN-541
### Screenshots (if appropriate)
N/A
Author: Prabhjyot Singh <prabhjyotsingh@gmail.com>
Closes#579 from prabhjyotsingh/increaseLogging and squashes the following commits:
ba053f5 [Prabhjyot Singh] revert note.json
648c65c [Prabhjyot Singh] fix CI
6d927fc [Prabhjyot Singh] Every catch should log error/info message or should throw an exception
7f8b70c [Prabhjyot Singh] replace printStackTrace() to LOGGER.error
When the query response contains null column values the following NPE is thrown:
java.lang.NullPointerException
at org.apache.zeppelin.postgresql.PostgreSqlInterpreter.replaceReservedChars(PostgreSqlInterpreter.java:278)
at org.apache.zeppelin.postgresql.PostgreSqlInterpreter.executeSql(PostgreSqlInterpreter.java:235)
Author: tzolov <christian.tzolov@gmail.com>
Closes#348 from tzolov/ZEPPELIN-349 and squashes the following commits:
de70c88 [tzolov] ZEPPELIN-349: Format test code
fff3448 [tzolov] ZEPPELIN-349: Resolve NPE on null cell values
Create postgresql README.md documentation
Author: tzolov <christian.tzolov@gmail.com>
Author: Christian Tzolov <christian.tzolov@gmail.com>
Closes#237 from tzolov/ZEPPELIN-251 and squashes the following commits:
82dc1e4 [tzolov] ZEPPELIN-252: Replace the postgresql/README.md content with a link to the Zeppelin Interpreters doc
30dc5b7 [Christian Tzolov] ZEPPELIN-251: Add Psql/Hawq video tutorial
bb271af [tzolov] ZEPPELIN-251: Postgresql Interpreter README.md documentation
Author: tzolov <christian.tzolov@gmail.com>
Closes#198 from tzolov/ZEPPELIN-215 and squashes the following commits:
35c40d4 [tzolov] ZEPPELIN-215: Handle failures during jdbc connectio initialization
271ef14 [tzolov] ZEPPELIN-215: Handle addition and removal of completions on Data Model changes
d87942e [tzolov] ZEPPELIN-215: Update the auto-complete metadata (schema, tables, columns) on update SQL queries
91ce7f4 [tzolov] ZEPPELIN-215: Fix a case where the cursor is close after the end of the buffer
d2393d6 [tzolov] ZEPPELIN-215: clean logging messages
4dd2a36 [tzolov] ZEPPELIN-215: resolve delimiter bug
0a3910d [tzolov] ZEPPELIN-215: Add DB schema names to the completion list
cef2b95 [tzolov] ZEPPELIN-215: Use JDBC.setMastRows(maxResults) to restrict the size of the displied rows
c6acd0e [tzolov] ZEPPELIN-215: PostgreSQL completer - initial implementation
modified: conf/zeppelin-site.xml.template
modified: pom.xml
new file: postgresql/pom.xml
new file: postgresql/src/main/java/org/apache/zeppelin/postgresql/PostgreSqlInterpreter.java
new file: postgresql/src/test/java/org/apache/zeppelin/postgresql/PostgreSqlInterpreterTest.java
Author: tzolov <christian.tzolov@gmail.com>
Closes#172 from tzolov/ZEPPELIN-70 and squashes the following commits:
5de507d [tzolov] ZEPPELIN-70: Remove the obsolete dependencies from the POM
a3db5f7 [tzolov] ZEPPELING-70: Rebase to upstream master
745ad3d [tzolov] ZEPPELIN-70: Add PostgreSQL JDBC license statement
a7d26dd [tzolov] ZEPPELIN-70: Closed previous connection before open new one. Register psql in ZeppelinConfiguration
886eab5 [tzolov] ZEPPELIN-70: Add property to control the number of SQL results shown in the display.
880bde0 [tzolov] ZEPPELIN-70: Improve javadoc. Clean pom formating
ba1b5b8 [tzolov] ZEPPELIN-70: Fix spelling
60544a1 [tzolov] ZEPPELIN-70: Add jdbc driver name as configuration property
841d67c [tzolov] ZEPPELIN-70: Add PostgreSQL Interpreter
