[GitHub] zeppelin issue #2442: [ZEPPELIN-2679] JDBC. precode for session

2017-08-09 Thread tinkoff-dwh 
Github user tinkoff-dwh commented on the issue:

https://github.com/apache/zeppelin/pull/2442
  
Ready to review


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2508: [ZEPPELIN-2817] Support default interpreter setting in...

2017-08-09 Thread reminia 
Github user reminia commented on the issue:

https://github.com/apache/zeppelin/pull/2508
  
Test added @felixcheung 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Created] (ZEPPELIN-2844) Zeppelin removes first underscore from the notebook

2017-08-09 Thread Amit Ghosh (JIRA) 
Amit Ghosh created ZEPPELIN-2844:


 Summary: Zeppelin removes first underscore from the notebook
 Key: ZEPPELIN-2844
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2844
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.7.2
Reporter: Amit Ghosh


Zeppelin removes first underscore from the notebook. For example: I ran the 
given below query using platform where I used ${bdl_table_schema}  = “asda_cdt” 
, ${category} = “healthcare_medicines” and ${store_dim_country_code}  = “GB”.
 
create table ${bdl_table_schema}.store_${category}
as
select store_nbr,banner_desc,latitude_dgr,longitude_dgr,open_date
from ww_core_dim_tables.store_dim
where country_code ='${store_dim_country_code}' and base_div_nbr = 1 and 
current_ind = 'Y';
 
But table created is  “storehealthcare_medicines”, which should be 
“store_healthcare_medicines”



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Created] (ZEPPELIN-2843) Zeppelin removes first underscore from the notebook

2017-08-09 Thread Amit Ghosh (JIRA) 
Amit Ghosh created ZEPPELIN-2843:


 Summary: Zeppelin removes first underscore from the notebook
 Key: ZEPPELIN-2843
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2843
 Project: Zeppelin
  Issue Type: Bug
  Components: Interpreters
Affects Versions: 0.7.2
Reporter: Amit Ghosh


Zeppelin removes first underscore from the notebook. For example: I ran the 
given below query using platform where I used ${bdl_table_schema}  = “asda_cdt” 
, ${category} = “healthcare_medicines” and ${store_dim_country_code}  = “GB”.
 
create table ${bdl_table_schema}.store_${category}
as
select store_nbr,banner_desc,latitude_dgr,longitude_dgr,open_date
from ww_core_dim_tables.store_dim
where country_code ='${store_dim_country_code}' and base_div_nbr = 1 and 
current_ind = 'Y';
 
But table created is  “storehealthcare_medicines”, which should be 
“store_healthcare_medicines”



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[GitHub] zeppelin pull request #2506: [ZEPPELIN-2813] revisions comparator

2017-08-09 Thread tinkoff-dwh 
GitHub user tinkoff-dwh reopened a pull request:

https://github.com/apache/zeppelin/pull/2506

[ZEPPELIN-2813] revisions comparator

### What is this PR for?
Sometimes need to see the difference between versions and to switch to 
another version and look for changes are not convenient (the page reloaded). 
This feature allows you to compare any two versions of the notebook.


### What type of PR is it?
Feature

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2813

### How should this be tested?
1 make some commits. Сchange the contents of paragraphs (delete, add, edit)
2 open Revisions comparator
3 compare revisions and check diff

### Screenshots (if appropriate)

![comparator](https://user-images.githubusercontent.com/25951039/28702781-cf1cedce-7378-11e7-9034-7036f4440bf3.gif)


### Questions:
* Does the licenses files need update? yes (updated)
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/tinkoff-dwh/zeppelin ZEPPELIN-2813

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2506.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 #2506


commit a192b95dbb1f8b1cb07123194c77a12c28978eec
Author: Tinkoff DWH 
Date:   2017-07-27T12:31:07Z

[ZEPPELIN-2813] revisions comparator for note

commit b9498142b40d80ba07581818011996a66056a9f3
Author: Tinkoff DWH 
Date:   2017-07-27T14:35:26Z

[ZEPPELIN-2813] license

commit 4ce5286c80a5ed022aa7d171f8e1614158d73f1b
Author: tinkoff-dwh 
Date:   2017-08-01T04:28:27Z

Merge remote-tracking branch 'origin/master' into ZEPPELIN-2813

commit 514b3f57bce2acfe94965142c9a8d8f357ce2f3f
Author: tinkoff-dwh 
Date:   2017-08-01T05:42:52Z

small fixes, added documentation

commit dc67f8fad302b93fbc3de297226695be4e14f294
Author: tinkoff-dwh 
Date:   2017-08-09T09:43:50Z

[ZEPPELIN-2813] refactoring




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2506: [ZEPPELIN-2813] revisions comparator

2017-08-09 Thread tinkoff-dwh 
Github user tinkoff-dwh closed the pull request at:

https://github.com/apache/zeppelin/pull/2506


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-09 Thread SachinJanani 
Github user SachinJanani commented on the issue:

https://github.com/apache/zeppelin/pull/2518
  
This build is failing due to some environment issue in jenkins can someone 
please help me out.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2521: [ZEPPELIN-2841] fix a problem in shell interpreter . W...

2017-08-09 Thread vistep 
Github user vistep commented on the issue:

https://github.com/apache/zeppelin/pull/2521
  
I tried again and it passed.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2521: [ZEPPELIN-2841] fix a problem in shell interpreter . W...

2017-08-09 Thread Leemoonsoo 
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2521
  
build sometimes fail because of network connection problem of flaky tests.
Could you try restart failed one?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2511: [ZEPPELIN-2821] Fix Missing import

2017-08-09 Thread asfgit 
Github user asfgit closed the pull request at:

https://github.com/apache/zeppelin/pull/2511


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2521: [ZEPPELIN-2841] fix a problem in shell interpreter . W...

2017-08-09 Thread vistep 
Github user vistep commented on the issue:

https://github.com/apache/zeppelin/pull/2521
  
I replaced System.getenv("HOME") with  System.getProperty("user.home") .
This time, Jenkins said "[ERROR] Failed to execute goal 
com.github.eirslett:frontend-maven-plugin:1.3:yarn (yarn test) on project 
zeppelin-web: Failed to run task: 'yarn run test' failed. (error code 1) -> 
[Help 1]"
I don`t understand why my code affects zeppelin-web ?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2514: replace pegdown to markdown zeppelin interpreter

2017-08-09 Thread Leemoonsoo 
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2514
  
CI went green https://travis-ci.org/cloverhearts/zeppelin/builds/262539649.
LGTM and merge to master if no further comment.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2462: ZEPPELIN-2527 Changed editor cursor to thin

2017-08-09 Thread malayhm 
Github user malayhm commented on the issue:

https://github.com/apache/zeppelin/pull/2462
  
Should we merge this PR?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2506: [ZEPPELIN-2813] revisions comparator

2017-08-09 Thread tinkoff-dwh 
Github user tinkoff-dwh commented on the issue:

https://github.com/apache/zeppelin/pull/2506
  
@1ambda 
fixed

![comparator](https://user-images.githubusercontent.com/25951039/29118118-296285a0-7d1a-11e7-9a73-540af586ade4.gif)



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2514: replace pegdown to markdown zeppelin interpreter

2017-08-09 Thread cloverhearts 
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/2514
  
https://travis-ci.org/cloverhearts/zeppelin

ci pass :)


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2484: [ZEPPELIN-2711] basic metrics for paragraphs & noteboo...

2017-08-09 Thread jongyoul 
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/2484
  
Agreed. we can improve this feature after adopting one.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #2523: [ZEPPELIN-2764] update packages and documentati...

2017-08-09 Thread khalidhuseynov 
GitHub user khalidhuseynov opened a pull request:

https://github.com/apache/zeppelin/pull/2523

[ZEPPELIN-2764] update packages and documentation for ZeppelinHubRepo 
storage

### What is this PR for?
update the packages/documentation/links due to migration of domain and 
renaming


### What type of PR is it?
Documentation | Refactoring

### Todos
* [x] - update package/class names
* [x] - update config
* [x] - update docs

### What is the Jira issue?
[ZEPPELIN-2764](https://issues.apache.org/jira/browse/ZEPPELIN-2764)

### How should this be tested?
CI pass

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? n
* Is there breaking changes for older versions? n
* Does this needs documentation? n


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/khalidhuseynov/zeppelin fix/ZEPPELIN-2764

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2523.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 #2523


commit 934b45e4fbe722b59b17bb133952067469a01d21
Author: Khalid Huseynov 
Date:   2017-08-09T06:11:42Z

rename repo packages

commit 25c7620458380410b66293432ecf5479d0f6b54d
Author: Khalid Huseynov 
Date:   2017-08-09T07:41:06Z

update links and logs

commit df0f2a01d78389c9b2b48cbf76ee676acb8dbdde
Author: Khalid Huseynov 
Date:   2017-08-09T08:11:40Z

rename repo class

commit 6d9f600523c5561b997a04e44702de77282db32c
Author: Khalid Huseynov 
Date:   2017-08-09T08:30:25Z

update token naming

commit 0531523bb1e42f59171a50feaf6ce2e591a71353
Author: Khalid Huseynov 
Date:   2017-08-09T08:40:35Z

update docs




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2518: Fixing zeppelin-2281 by clearing output when data is a...

2017-08-09 Thread SachinJanani 
Github user SachinJanani commented on the issue:

https://github.com/apache/zeppelin/pull/2518
  
@Leemoonsoo  Thanks for the review.Have started the travis build. 
@felixcheung With this change it will not clear the output if the execution 
fails and keep the previous output.But if there any exception then it will 
print them


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2511: [ZEPPELIN-2821] Fix Missing import

2017-08-09 Thread felixcheung 
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/2511
  
awesome!


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #2521: [ZEPPELIN-2841] fix a problem in shell interpreter . W...

2017-08-09 Thread vistep 
Github user vistep commented on the issue:

https://github.com/apache/zeppelin/pull/2521
  
I have fix the build problem by setting travis according to what Jenkins 
said at last part.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---