You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by mwkang <gi...@git.apache.org> on 2016/03/21 08:05:10 UTC
[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...
GitHub user mwkang opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/787
[ZEPPELIN-751]Log an error and continue reading the remaining notebooks
### What is this PR for?
Log an Amazon Service Exception.
Continue reading the remaining notebooks into S3ObjectSummary.
### What type of PR is it?
Bug Fix
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-751
### Questions:
* Do you have a plan using Guava?
(Sometimes Optional is good stuff)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mwkang/incubator-zeppelin master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/787.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #787
----
commit 23bb71c4dec44ce0d832e296a28b863fe2ffb66c
Author: Minwoo Kang <mi...@outlook.com>
Date: 2016-03-21T06:55:47Z
[ZEPPELIN-751]Log an error and continue reading the remaining notebooks
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/787#issuecomment-199362026
Changes looks good to me!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/787
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...
Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/787#issuecomment-200181420
looks good!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/787#issuecomment-199939486
@mwkang I think ZEPPELIN-754 is not much related to ZEPPELIN-751. Then how about remove last commit about ZEPPELIN-754 and create a new pullrequest for it?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...
Posted by mwkang <gi...@git.apache.org>.
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/787#issuecomment-199574692
Oh.. It is my fault.
This request includes also [ZEPPELIN-754]Support seoul region.
Because I push '0d98c98' commit before Close this reqeust.
Is it okay?
Or What I have to do?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/787#issuecomment-200365246
Merging it into the master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...
Posted by mwkang <gi...@git.apache.org>.
Github user mwkang commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/787#issuecomment-200116775
Okay. I remove last commit.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---