Elgato_dark/zeppelin
1
0
Fork 0
mirror of https://github.com/apache/zeppelin synced 2026-05-24 09:38:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

[ZEPPELIN-3092] Replace printStackTrace() with error logging

Browse source
This commit is contained in:
Mohamed Magdy 2017-12-22 21:51:37 +01:00
parent db94d55d94
commit 9101e58f28
1 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/repo/GitHubNotebookRepo.java
View file

@ -59,9 +59,9 @@ public class GitHubNotebookRepo extends GitNotebookRepo {
remoteAddCommand.setUri(new URIish(zeppelinConfiguration.getZeppelinNotebookGitURL()));
remoteAddCommand.call();
} catch (GitAPIException e) {
e.printStackTrace();
LOG.error("Error configuring GitHub", e);
} catch (URISyntaxException e) {
e.printStackTrace();
LOG.error("Error in GitHub URL provided", e);
}
}
@ -86,8 +86,7 @@ public class GitHubNotebookRepo extends GitNotebookRepo {
pullCommand.call();
} catch (GitAPIException e) {
LOG.error("Error when pulling latest changes from remote repository");
e.printStackTrace();
LOG.error("Error when pulling latest changes from remote repository", e);
}
}
@ -104,8 +103,7 @@ public class GitHubNotebookRepo extends GitNotebookRepo {
pushCommand.call();
} catch (GitAPIException e) {
LOG.error("Error when pushing latest changes from remote repository");
e.printStackTrace();
LOG.error("Error when pushing latest changes from remote repository", e);
}
}
}