zjffdu opened a new pull request #4195:
URL: https://github.com/apache/zeppelin/pull/4195
### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide -
https://zeppelin.apache.org/contribution/contri
Can you add a logic to check if the max result size is reached? If so, print
out a warning message. Essentially, the same behavior as found in the python
interpreter.
if exceed_limit: print("\n%html Results are
limited by {}.".format(self.max_result))
ThanksDenny
On S
asfgit closed pull request #4184:
URL: https://github.com/apache/zeppelin/pull/4184
--
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 the specific comment.
To unsubscribe, e-mail: dev-unsubscr..
asfgit closed pull request #4188:
URL: https://github.com/apache/zeppelin/pull/4188
--
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 the specific comment.
To unsubscribe, e-mail: dev-unsubscr..
asfgit closed pull request #4190:
URL: https://github.com/apache/zeppelin/pull/4190
--
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 the specific comment.
To unsubscribe, e-mail: dev-unsubscr..
Reamer commented on pull request #4192:
URL: https://github.com/apache/zeppelin/pull/4192#issuecomment-890540495
"Cluster.local" as you mentioned is not always correct. I would prefer
either a domain value that can be configured or add documentation that you need
to configure your cluster
zjffdu commented on pull request #4186:
URL: https://github.com/apache/zeppelin/pull/4186#issuecomment-890526987
@aib628 Recently CI seems unable recently, we may need to wait for a while
until CI become stable.
--
This is an automated message from the Apache Git Service.
To respond to
aib628 commented on pull request #4186:
URL: https://github.com/apache/zeppelin/pull/4186#issuecomment-89051
> @aib628 Just force push will trigger the CI, and BTW besides the NPE in
log, is there any user experience effect of this issue ??
@zjffdu no, log only
--
This is an a
zjffdu commented on pull request #4192:
URL: https://github.com/apache/zeppelin/pull/4192#issuecomment-890476547
@ebuildy Thanks for the clarification, this sounds like convention, right ?
Anyway, this PR LGTM
--
This is an automated message from the Apache Git Service.
To respond to th
