[GitHub] [zeppelin] Leemoonsoo commented on issue #3437: [ZEPPELIN-4319] Fix spell interpreter parse text logic

2019-09-19 Thread GitBox
Leemoonsoo commented on issue #3437: [ZEPPELIN-4319] Fix spell interpreter 
parse text logic
URL: https://github.com/apache/zeppelin/pull/3437#issuecomment-533275448
 
 
   Thanks @santosh1994 for the contribution. Somehow, this PR didn't run 
Jenkins CI test.
   Could you close, re-open this PR (or push a new commit) to trigger CI test 
again?
   


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [zeppelin] Leemoonsoo commented on issue #3437: [ZEPPELIN-4319] Fix spell interpreter parse text logic

2019-09-26 Thread GitBox
Leemoonsoo commented on issue #3437: [ZEPPELIN-4319] Fix spell interpreter 
parse text logic
URL: https://github.com/apache/zeppelin/pull/3437#issuecomment-535620652
 
 
   The branch this PR is based on looks like does not have .travis.yml.
   I suggest rebase PR to current master branch, so we can make sure everything 
is okay.
   
   But currently, master branch seems have some broken tests. I'm fixing it 
now. Let me nofify here once master branch becomes green.
   
   


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [zeppelin] Leemoonsoo commented on issue #3437: [ZEPPELIN-4319] Fix spell interpreter parse text logic

2019-10-01 Thread GitBox
Leemoonsoo commented on issue #3437: [ZEPPELIN-4319] Fix spell interpreter 
parse text logic
URL: https://github.com/apache/zeppelin/pull/3437#issuecomment-537202929
 
 
   @santosh1994 Just fixed master branch CI issue. Could you try rebase this to 
latest master branch and see if it passes all tests?


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [zeppelin] Leemoonsoo commented on issue #3437: [ZEPPELIN-4319] Fix spell interpreter parse text logic

2019-10-15 Thread GitBox
Leemoonsoo commented on issue #3437: [ZEPPELIN-4319] Fix spell interpreter 
parse text logic
URL: https://github.com/apache/zeppelin/pull/3437#issuecomment-542325635
 
 
   Looks like Jenkins have a problem not triggering some PR 
([INFRA-19249](https://issues.apache.org/jira/browse/INFRA-19249)).
   
   Anyway, I checked your travis link and everything looks good to me.
   
   Thanks @santosh1994 for the contribution. I'm merging this to master.


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services