Jeff Zhang created ZEPPELIN-5218:
Summary: Don't exit shell interpreter if there's more logging
produced
Key: ZEPPELIN-5218
URL: https://issues.apache.org/jira/browse/ZEPPELIN-5218
Project: Zeppelin
Reamer commented on pull request #4031:
URL: https://github.com/apache/zeppelin/pull/4031#issuecomment-765244322
I think we should use custom exceptions.
> User code operates at a higher layer and doesn’t generate
TApplicationExceptions, rather making use of user-defined exceptions,
Reamer commented on pull request #4028:
URL: https://github.com/apache/zeppelin/pull/4028#issuecomment-765435792
I will include this PR in master and branch-0.9 on Monday (25.01.2021) if no
further comments are received.
Thi
zjffdu commented on pull request #4031:
URL: https://github.com/apache/zeppelin/pull/4031#issuecomment-765485575
@Reamer That's make sense, I have updated the PR with user defined exception
`InterpreterRPCException`
This is
zjffdu commented on pull request #4022:
URL: https://github.com/apache/zeppelin/pull/4022#issuecomment-765533770
@Reamer I tried several times, but always fails in the following step
```
+ conda env update -f /env_python_3_with_R.yml --prune
Collecting package metadata (repodata
Reamer commented on pull request #4031:
URL: https://github.com/apache/zeppelin/pull/4031#issuecomment-765549606
The user defined exception looks much better. Do you have any idea why each
function also throws the parent `TException`?
--
zjffdu commented on pull request #4031:
URL: https://github.com/apache/zeppelin/pull/4031#issuecomment-765866889
> The user defined exception looks much better. Do you have any idea why
each function also throws the parent `TException`?
That is auto generated by thrift.
--
asfgit closed pull request #4029:
URL: https://github.com/apache/zeppelin/pull/4029
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
zjffdu commented on pull request #4023:
URL: https://github.com/apache/zeppelin/pull/4023#issuecomment-765867115
Will merge if no more comment, will fix the flink 1.10 test in another PR
This is an automated message from the
zjffdu commented on pull request #4025:
URL: https://github.com/apache/zeppelin/pull/4025#issuecomment-765867159
Will merge if no more comment
This is an automated message from the Apache Git Service.
To respond to the messag
zjffdu commented on pull request #4016:
URL: https://github.com/apache/zeppelin/pull/4016#issuecomment-765867323
ping @fforever14
This is an automated message from the Apache Git Service.
To respond to the message, please lo
zjffdu commented on pull request #3975:
URL: https://github.com/apache/zeppelin/pull/3975#issuecomment-765867377
ping @milooy
This is an automated message from the Apache Git Service.
To respond to the message, please log on
zjffdu commented on pull request #3886:
URL: https://github.com/apache/zeppelin/pull/3886#issuecomment-765867516
We have already upgraded thrift to 0.13, could you close this PR ?
@emreegriboz
This is an automated message f
asfgit closed pull request #4023:
URL: https://github.com/apache/zeppelin/pull/4023
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
fforever14 commented on pull request #4016:
URL: https://github.com/apache/zeppelin/pull/4016#issuecomment-765884157
Sorry. I didn't check github these days due to recent work.
I will add test code this weekend.
This is an
zjffdu commented on pull request #4016:
URL: https://github.com/apache/zeppelin/pull/4016#issuecomment-765884422
Thanks @fforever14
This is an automated message from the Apache Git Service.
To respond to the message, please
16 matches
Mail list logo
