[GitHub] zeppelin pull request #1486: [ZEPPELIN-1518] Fix for: Lambda expressions are...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis...

[astroshim]

GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1509] Bug of testcase path in .travis.yml

### What is this PR for?
This PR fixes path of spark testcase.


### What type of PR is it?
Bug Fix

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


### Questions:
* Does the licenses files need update? no
* 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/astroshim/zeppelin ZEPPELIN-1509

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

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


commit bc74738750366833366f6aa9cc8b3f814a996c6f
Author: astroshim 
Date:   2016-09-30T04:12:45Z

fix testcase path

commit dead6812c9934beba83b9dda5eacaab07461ec94
Author: astroshim 
Date:   2016-10-07T02:14:24Z

Merge branch 'master' of https://github.com/apache/zeppelin into 
ZEPPELIN-1509




---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis...

[astroshim]

Github user astroshim closed the pull request at:

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


---
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 #1378: [ZEPPELIN-1358] Add support to display Pandas DataFram...

[bzz]

Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1378
  
@bustios let me know if you still think it's possible to make it optional 
and I'll be happy to merge 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #1495: ZEPPELIN-1325: WIP - Python, advanced interpret...

[bzz]

GitHub user bzz opened a pull request:

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

ZEPPELIN-1325: WIP - Python, advanced interpreter impl

### What is this PR for?
This is different implementation of Python interpreter that communicates to 
external Python 2/3 process (not just REPL) though RPC. 

It aims for rich feature set, missing in current implementation:
 - better error handling
 - PEP-8 syntax compatibility
 - code-completion
 - job progress/cancellation.
 - windows support

![screen shot 2016-10-07 at 13 17 
13](https://cloud.githubusercontent.com/assets/5582506/19178701/6f190798-8c90-11e6-9826-525803b35dfd.png)


### What type of PR is it?
Improvement

### TODOs
* [x] Initial PoC
* [ ] add `%python2.dep` interpreter impl
* [ ] install default python dependencies automatically
* [ ] update grpc version to 1.0 to make it python3 compatible
* [ ] make sure it works on windows
* [ ] change  ambiguous name/package `python2` to something better, like 
`python-remote` or `python-rich` or something else
* [ ] update `bin_licenses/LICENSE`

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

### How should this be tested?
Bind `Python2` (name will be changed) interpreter and run any python code 
i.e [ZEPPELIN-1133](https://issues.apache.org/jira/browse/ZEPPELIN-1133)

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


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

$ git pull https://github.com/bzz/incubator-zeppelin 
python/advanced-interpreter

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

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


commit de1caa5c49d336d3afaad2304eb52081fbeea9e2
Author: Alexander Bezzubov 
Date:   2016-08-10T04:26:50Z

Add DRAFT of .proto, generated files and an automation script

commit 2a47c824865c2e5a645d1807f268440a71d9b2cf
Author: Alexander Bezzubov 
Date:   2016-08-10T04:27:22Z

Add first DRAFT of Python Interpreter servier impl

commit 08345debe06e59d6f73b29425eea24c2297ba32f
Author: Alexander Bezzubov 
Date:   2016-08-12T10:05:16Z

Add first DRAFT of Python Interpreter client impl

commit d2a1e506e575dfaa15e8b41d048709457b211fbd
Author: Alexander Bezzubov 
Date:   2016-08-12T13:28:29Z

Update .proto and code generation scripts

commit 1345532aff3fd9b7a10f7aa8845ed7bb618e4af1
Author: Alexander Bezzubov 
Date:   2016-08-19T09:20:40Z

Make interpreter interprete and report errors \w line numbers

commit f3d4794cf03a16a00e7f4fd37f34ef223c620346
Author: Alexander Bezzubov 
Date:   2016-08-26T07:09:26Z

add error detection to JVM and Python Shutdown() stub

commit c0c97c1496c4287d273353358cb7baded52db962
Author: Alexander Bezzubov 
Date:   2016-08-27T04:39:26Z

Notes do not share context: vars, imports, etc




---
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 #1493: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

[khalidhuseynov]

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1493
  
i believe there's also `org.apache.zeppelin.realm` package under `zeppelin` 
and in the long run maybe need to move these realms there rather than keeping 
under `server`


---
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 #1494: [ZEPPELIN-451] Save codes and messages as multi...

[sixmen]

GitHub user sixmen opened a pull request:

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

[ZEPPELIN-451] Save codes and messages as multi-line

### What is this PR for?
Save Paragraph texts and result msgs as array of strings instead of one 
long string.
One long strings are not VCS-friendly.

### What type of PR is it?
Improvement

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

### How should this be tested?
Modify notebooks and check saved note files

### Screenshots (if appropriate)
https://cloud.githubusercontent.com/assets/1409279/19177502/bdd40b28-8c84-11e6-8b43-3d6e1318e1c0.png;>
https://cloud.githubusercontent.com/assets/1409279/19177508/c218d768-8c84-11e6-9d4f-ae5227ccb96b.png;>

### Questions:
* Does the licenses files need update? NO
* Is there breaking changes for older versions? Notebooks from new zeppelin 
can not be imported in old zeppelin.
* Does this needs documentation? NO


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

$ git pull https://github.com/sixmen/zeppelin note_multiline_string

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

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


commit 2a69495d3ef0a97712989dfcf909aecfb2762801
Author: Sangmin Yoon 
Date:   2016-10-05T01:58:40Z

rename NotebookImportDeserializer to DateDeserializer

commit 2bd5fa9c998c5ad426163ace6e6227f789967342
Author: Sangmin Yoon 
Date:   2016-10-05T08:03:17Z

save Paragraph.text as list of strings

commit 155a405a5bf93ae1ee8cf34583590e1b24ebaa4c
Author: Sangmin Yoon 
Date:   2016-10-05T08:39:59Z

save InterpreterResult.msg as list of strings




---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

[astroshim]

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
@minahlee I just rebased. Thank you for your great efforts.


---
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 #1475: [ZEPPELIN-1509] Bug of testcase path in .travis.yml

[minahlee]

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1475
  
@astroshim #1483 is just merged, can you rebase master and see if CI 
failure on spark 1.1 and 1.2 is gone?


---
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 #1483: [ZEPPELIN-1509] Run certain SparkInterpreter te...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1486: [ZEPPELIN-1518] Fix for: Lambda expressions are not wo...

[minahlee]

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1486
  
Merging to master and branch-0.6 if there is no more discussion


---
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 #1493: ZEPPELIN-1472 - Create new LdapRealm based on Apache K...

[gss2002]

Github user gss2002 commented on the issue:

https://github.com/apache/zeppelin/pull/1493
  
@vinayshukla here is the pull request for the LdapRealm enhancement we 
discussed last week at HadoopWorld/Strata


---
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 #1493: ZEPPELIN-1472 - Create new LdapRealm based on A...

[gss2002]

GitHub user gss2002 opened a pull request:

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

ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm

### What is this PR for?
Provides LdapRealm Functionality similar to what Apache Knox provides. This 
is critical as in large enterprise environments Active Directory Global 
Catalogs are used for lookup with samAccountName and using a DN Template is not 
an option as their are multiple OUs. Also searching on "userPrincipalName" is 
risky in an AD environment since the explicit UPN vs Implicit UPN can be 
different this is definitely the case with environments using Office 365. And 
the LDAP userPrincipalName attribute is the explicit UPN which can be defined 
by the directory administrator to any value and it can be duplicated. 
SamAccountName is unique per domain and Microsoft states best practice is to 
not allow duplicate samAccountName's across the forest. 

Information about samAccountName and userPrincipalName with ActiveDirectory

http://windowsitpro.com/active-directory/q-does-samaccountname-object-have-be-unique-active-directory-domain-or-entire-fores

https://jorgequestforknowledge.wordpress.com/2010/10/12/user-principal-names-in-ad-part-1/
 
### What type of PR is it?
Improvement

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

### How should this be tested?
shiro.ini
[main]
ldapRealm = org.apache.zeppelin.server.LdapRealm
ldapRealm.contextFactory.systemUsername = 
CN=hdpbind,OU=Svc,DC=exadc,DC=w2k,DC=example,DC=com
ldapRealm.contextFactory.systemPassword = ldapPassword
ldapRealm.searchBase = dc=w2k,dc=example,dc=com
ldapRealm.userSearchBase = dc=w2k,dc=example,dc=com
ldapRealm.groupSearchBase = dc=w2k,dc=example,dc=com
ldapRealm.contextFactory.url = ldap://exampledc1.exadc.w2k.example.com:3268
ldapRealm.userSearchAttributeName = sAMAccountName
ldapRealm.contextFactory.authenticationMechanism = simple
ldapRealm.userObjectClass = user
ldapRealm.groupObjectClass = group
ldapRealm.memberAttribute = member
securityManager.realms = $ldapRealm


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


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

$ git pull https://github.com/gss2002/zeppelin master

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

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


commit 4b5963a2019f1fded13e6ce9942033101ef2acf1
Author: Initial Commit 
Date:   2016-10-07T00:55:42Z

ZEPPELIN-1472 - Create new LdapRealm based on Apache Knox LdapRealm
Class

In our environment we attempted to use the ActiveDirectoryGroupRealm and
the LdapGroupRealm but unfortunately those implementations against Shiro
do not support ADLDAP Global Catalog. Also searching on
"userPrincipalName" is risky in an AD environment since the explicit UPN
vs Implicit UPN can be different. And the LDAP userPrincipalName
attribute is the explicit UPN which can be defined by the directory
administrator to any value and it can be duplicated.. SamAccountName is
unique per domain and Microsoft states best practice is to not allow
duplicate samAccountName's per the forest. I have attached a
semi-working modified KnoxLdapRealm which works against samAccountName
and global catalog for auth.

http://windowsitpro.com/active-directory/q-does-samaccountname-object-have-be-unique-active-directory-domain-or-entire-fores

https://jorgequestforknowledge.wordpress.com/2010/10/12/user-principal-names-in-ad-part-1/




---
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 #1316: [ZEPPELIN-1118] Migrate Zeppelin Live Demo feature fro...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1316
  
@corneadoug I don't think live demo is related to contents of community 
page.

Any help on writing terms of services would be appreciated.


---
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 #1488: [ZEPPELIN-1519] Fix notebook importer AM/PM omi...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1490: [ZEPPELIN-1480] Blocking message pending 10000 for BLO...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1490
  
Merge to master and branch-0.6 if there're no more discussions.


---
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 #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

[corneadoug]

Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1363
  
@Peilin-Yang Actually it's a bit hard for me to help since I can't rebase 
for you (Don't have the rights on your branch), but I can see that it is a 
pretty messy conflict you got.

For the Rebase process, you need to do a:
```
git fetch apache-github
git rebase apache-github/master
```
(apache-github being a remote for this official repo)

At the first round of conflicts on `paragraph.controller.js`, I would 
advise to copy paste the file from master, and replace your local file with 
conflict with it.
Then to copy paste the changes of your first commit into it (you can see 
the first commit diff 
[Here](https://github.com/apache/zeppelin/pull/1492/commits/b92e92ff8821a58c47d7510818ec518414499c83))

After that first round, the other conflicts should be easier


---
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 #1492: Refactor/handsontable service

[corneadoug]

GitHub user corneadoug opened a pull request:

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

Refactor/handsontable service

Test PR to help rebase

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

$ git pull https://github.com/corneadoug/incubator-zeppelin 
Refactor/HandsontableService

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

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


commit b92e92ff8821a58c47d7510818ec518414499c83
Author: Peilin Yang 
Date:   2016-08-25T17:06:23Z

add text/numeric conversion support to table display

commit 972cbe2304306e0f844107373ce38b5f90d33c30
Author: Peilin Yang 
Date:   2016-09-10T20:58:55Z

rebase merge

commit 7976585849fb09d7a5de66c9b2074f8fc3aa7b4b
Author: Peilin Yang 
Date:   2016-09-12T16:41:20Z

add validator for the columns

commit 33f34168ef76fff8952fb0e1ba81ae02e82c67cd
Author: Peilin Yang 
Date:   2016-09-12T16:43:58Z

commit

commit f6fde09f471f5c03d3b4918f46f1f4f43fe0b4fc
Author: Peilin Yang 
Date:   2016-09-12T16:44:04Z

Merge branch 'master' into ypeilin/table_num_cell_format

commit d088fc2e22c217d06fd93ef85c90d60ff482ff80
Author: Peilin Yang 
Date:   2016-09-12T17:23:50Z

change the format of numeric column

commit ba7112b33588320d5c5645bb909b7bc2c274f96d
Author: Peilin Yang 
Date:   2016-09-22T11:26:56Z

Merge branch 'master' into ypeilin/table_num_cell_format

commit 40f0a276fb346f81b9cc7d75a4e5ee974e938c03
Author: Peilin Yang 
Date:   2016-09-22T15:44:21Z

fix the bug: data type lost when switching between tables and charts.

commit caeffa9268af6a30d754336b372690444e698850
Author: Damien CORNEAU 
Date:   2016-10-04T09:01:21Z

Move HandsonTable functions to a Service

commit 4e9b1f306a7ac71d12cb70a78c4ace7bf29ad101
Author: Damien CORNEAU 
Date:   2016-10-05T07:02:34Z

Resort column after type change

commit 82770b20dc3783564f1bfbc02e4e904032ae0ca7
Author: Damien CORNEAU 
Date:   2016-10-05T07:50:57Z

Small 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 issue #1486: [ZEPPELIN-1518] Fix for: Lambda expressions are not wo...

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1486
  
I think master also needs this patch.


---
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 #1470: ZEPPELIN-1423 - Allow users to specify pre/post-execut...

[agoodm]

Github user agoodm commented on the issue:

https://github.com/apache/zeppelin/pull/1470
  
@Leemoonsoo I have addressed your second comment, let me know if this looks 
good to merge into 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

[Peilin-Yang]

Github user Peilin-Yang commented on the issue:

https://github.com/apache/zeppelin/pull/1363
  
@corneadoug I have trouble with the rebase:( ... can you help a little bit?


---
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 #1461: [MINOR] Rearrange the contents in "About Zeppel...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1461: [MINOR] Rearrange the contents in "About Zeppelin"

[corneadoug]

Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1461
  
Go 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

[cloverhearts]

Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1441
  
@AhyoungRyu @corneadoug 
Thank you very much for your feedback.

 Filtering system
I think the filter I added here actually will improve filtering performance 
since as soon as you press return key, it will show filtering result. And if 
there is no input from user for more than 2 sec, it stops filtering.

The reason I implemented filtering myself is to fix the latency issue with 
large number of notebooks. If you filter on every keydown event, it will freeze 
browser when you have a lot of notebooks.(tested with 250 notebooks). 

Will it be better if I add UI letting user know that they should press 
return button to complete filtering?


@corneadoug 
actually, I've used the interpreter filtering in the first implementation.
However, the behavior is a bit different from job filtering. So I had to 
define separate filters.

 Filtering input and No results
Your opinion is right.
Okay, I will fix it.

 Interpreter dropdown filtering
I think this feature is very important.
because currently there is no way to manage your interpreter which are not 
bound to notebook.
also, if you use a lot of notebooks and interpreter, I think users would 
want to see and sort it.
Personally I also wanted to have this kind of view when i was using cron 
scheduler to operating a notebook then can not find problem for interpreter 
bind.
I think this feature is a good way to be feedback to the user.

 Possible future work that could use a jira issue
Yes, I agree with your opinion.
i will make a jira issue and implement those in future.

Thank you again.


---
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 #1339: [ZEPPELIN-1332] Remove spark-dependencies & suggest ne...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1339
  
@jongyoul Thanks for your feedback! Yeah I didn't try to cover that case. 
So you mean we need to support ppl who are using [this upstart 
option](http://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/install/install.html#optional-start-apache-zeppelin-with-a-service-manager),
 am I right? :)


---
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 #1461: [MINOR] Rearrange the contents in "About Zeppelin"

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1461
  
Merging this if there are no more discussions 


---
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-1523) Date strings are shown incorrectly in graph.

[Sangmin Yoon (JIRA)]

Sangmin Yoon created ZEPPELIN-1523:
--

 Summary: Date strings are shown incorrectly in graph.
 Key: ZEPPELIN-1523
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1523
 Project: Zeppelin
  Issue Type: Bug
  Components: front-end
Affects Versions: 0.7.0
Reporter: Sangmin Yoon


{code}
%jdbc
select '2016-09-01' as date, 10 as count
{code}

If you input above, and see the graph, the date is shown as 'Thu Sep 01 2016  
00:00:00  GTM+09:00'.
(And if you export table as csv, dates become getTime() values)

This problem was created by https://issues.apache.org/jira/browse/ZEPPELIN-1372.
Every date string is converted to a moment instance, so date is displayed as 
above.

I think automatic date detection is unnecessary unlike number detection.

I think mixed date format (like 11/30/2015 and 2014-05-06) is not common,
and if the date format is same, dates can be sorted without converting to 
moment instances.

For verbose date strings (e.g. 19/Sep/16), it may be better to add option like 
https://issues.apache.org/jira/browse/ZEPPELIN-1371 than automatic detection.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] zeppelin issue #1461: [MINOR] Rearrange the contents in "About Zeppelin"

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1461
  
@corneadoug rebased but it still failed in Selenium test. The error log is 
like below 
```
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 76.542 sec 
- in org.apache.zeppelin.integration.SparkParagraphIT

Results :

Failed tests: 
  ParagraphActionsIT.testWidth:314 New Width is : 4
Expected: 
 but: was 
  ParagraphActionsIT.testWidth:314 New Width is : 8
Expected: 
 but: was 

Tests in error: 
  
ZeppelinIT.testSparkInterpreterDependencyLoading:234->AbstractZeppelinIT.waitForParagraph:70->AbstractZeppelinIT.pollingWait:96
 » Timeout

Tests run: 16, Failures: 2, Errors: 1, Skipped: 0

[INFO] 
[INFO] --- scalatest-maven-plugin:1.0:test (test) @ zeppelin-server ---
Discovery starting.
Oct 06, 2016 8:09:31 AM org.openqa.selenium.safari.SafariDriverServer start
INFO: Server started on port 13724
Oct 06, 2016 8:09:31 AM 
org.openqa.selenium.safari.SafariDriverCommandExecutor stop
INFO: Shutting down
Oct 06, 2016 8:09:31 AM 
org.openqa.selenium.safari.SafariDriverCommandExecutor stop
INFO: Stopping server
Oct 06, 2016 8:09:31 AM org.openqa.selenium.safari.SafariDriverServer stop
INFO: Stopping server
Oct 06, 2016 8:09:31 AM 
org.openqa.selenium.safari.SafariDriverCommandExecutor stop
INFO: Shutdown complete
Discovery completed in 4 seconds, 967 milliseconds.
Run starting. Expected test count is: 1
Zeppelin is already running
WelcomePageSuite:
- Welcome sign is correct
Zeppelin stop [  OK  ]
Run completed in 13 seconds, 782 milliseconds.
Total number of tests run: 1
Suites: completed 3, aborted 0
Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
All tests passed.
[INFO] 
[INFO] --- maven-antrun-plugin:1.7:run (stop-zeppelin) @ zeppelin-server ---
[INFO] Executing tasks

main:
 [exec] Zeppelin is not running
[INFO] Executed tasks
[INFO] 
[INFO] --- maven-failsafe-plugin:2.16:verify (default) @ zeppelin-server ---
[INFO] Failsafe report directory: 
/home/travis/build/apache/zeppelin/zeppelin-server/target/failsafe-reports
[INFO] 

[INFO] Reactor Summary:
[INFO] 
[INFO] Zeppelin: Interpreter .. SUCCESS [ 
12.285 s]
[INFO] Zeppelin: Zengine .. SUCCESS [  
6.291 s]
[INFO] Zeppelin: Display system apis .. SUCCESS [  
3.479 s]
[INFO] Zeppelin: Spark dependencies ... SUCCESS [ 
37.316 s]
[INFO] Zeppelin: Spark  SUCCESS [  
6.898 s]
[INFO] Zeppelin: Server ... FAILURE [09:22 
min]
[INFO] 

[INFO] BUILD FAILURE
[INFO] 

[INFO] Total time: 10:29 min
[INFO] Finished at: 2016-10-06T08:09:45+00:00
[INFO] Final Memory: 46M/376M
[INFO] 

[ERROR] Failed to execute goal 
org.apache.maven.plugins:maven-failsafe-plugin:2.16:verify (default) on project 
zeppelin-server: There are test failures.
[ERROR] 
[ERROR] Please refer to 
/home/travis/build/apache/zeppelin/zeppelin-server/target/failsafe-reports for 
the individual test results.
[ERROR] -> [Help 1]
[ERROR] 
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e 
switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR] 
[ERROR] For more information about the errors and possible solutions, 
please read the following articles:
[ERROR] [Help 1] 
http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR] 
[ERROR] After correcting the problems, you can resume the build with the 
command
[ERROR]   mvn  -rf :zeppelin-server
```

Seems not irrelevant with this change :) 


---
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 #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port shou...

[asfgit]

Github user asfgit closed the pull request at:

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


---
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 #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

[corneadoug]

Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1441
  
@cloverhearts Just tested and checked the code.
Overall same feedbacks as @AhyoungRyu 

 Filtering system
The one you used seems a bit too complicated (also the filter should have 
been in a new file).
Since it is also slow, using the same method used in interpreter search 
would be better.

 Filtering input
Try to reuse the same as the interpreter search, it should have the proper 
font and keep the same style between pages.

 Interpreter dropdown filtering
I would remove this feature mainly because it only works with the default 
interpreter.
Maybe also remove the default interpreter written on each job?

 No results
Current message is: `Data does not exist`, `No Job found` would be better

 Possible future work that could use a Jira issue
* Removing the legend (ready, finished, abort etc...), that info is already 
on each dot.
* Make Interpreter and Job page top part fixed, so that we can search even 
while scrolling



---
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 #1483: [ZEPPELIN-1509] Run certain SparkInterpreter tests onl...

[minahlee]

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1483
  
Merging if there is no more discussion


---
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 #1461: [MINOR] Rearrange the contents in "About Zeppel...

[AhyoungRyu]

GitHub user AhyoungRyu reopened a pull request:

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

[MINOR] Rearrange the contents in "About Zeppelin"

### What is this PR for?
Just rearranged the placement of Zeppelin logo, title and text in "About 
Zeppelin". In the mobile screen as well. 

### What type of PR is it?
Improvement

### What is the Jira issue?
No Jira issue for this 

### How should this be tested?
To check this change in your local, build only `zeppelin-web` with `./grunt 
build` and then browse with `./grunt serve`.

### Screenshots (if appropriate)
- **Before** 
  1. desktop & tablet screen
https://cloud.githubusercontent.com/assets/10060731/18860571/8aa85aec-84b9-11e6-9112-b6869291f8f0.png;
 width="500px">

  2. mobile screen
https://cloud.githubusercontent.com/assets/10060731/18860576/91f623a6-84b9-11e6-96ba-e34c43221143.png;
 width="300px">

- **After** 
  1. desktop & tablet screen
https://cloud.githubusercontent.com/assets/10060731/19140869/b5ab79e0-8bcb-11e6-946f-df63045e6f1e.png;
 width="500px">

  2. mobile screen
https://cloud.githubusercontent.com/assets/10060731/19140879/c034b476-8bcb-11e6-977b-29deb6776f13.png;
 width="300px">



### Questions:
* Does the licenses files need update? no
* 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/AhyoungRyu/zeppelin improve/aboutZeppelin

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

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


commit 00d8a5737197c44b7ff149e585cd7ad9c11eb3b6
Author: AhyoungRyu 
Date:   2016-09-27T04:49:43Z

Rearrange 'About Zeppelin'

commit 75926881dd3211d2c53fc38d7b764fd9e6af2777
Author: AhyoungRyu 
Date:   2016-10-06T04:48:49Z

Apply responsive design

commit 804d8bbcda0b1f0a33c6e475597656c88e993276
Author: AhyoungRyu 
Date:   2016-10-06T04:51:10Z

Add a newline




---
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 #1461: [MINOR] Rearrange the contents in "About Zeppel...

[AhyoungRyu]

Github user AhyoungRyu closed the pull request at:

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


---
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 #1347: ZEPPELIN-1284. Unable to run paragraph with default in...

[H4ml3t]

Github user H4ml3t commented on the issue:

https://github.com/apache/zeppelin/pull/1347
  
Sorry I was running of time too, and I decided to downgrade both Spark and 
Zeppelin versions, for which I was sure everything would have worked. I suggest 
to do the same.

Cheers,
Luca


---
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 #1461: [MINOR] Rearrange the contents in "About Zeppelin"

[corneadoug]

Github user corneadoug commented on the issue:

https://github.com/apache/zeppelin/pull/1461
  
@AhyoungRyu Rebasing from master should fix the CI issue


---
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 #1489: ZEPPELIN-1321 Zeppelin HTTP and HTTPS port should be m...

[r-kamath]

Github user r-kamath commented on the issue:

https://github.com/apache/zeppelin/pull/1489
  
cool. will merge this if there are no more discussions


---
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 #1441: [ ZEPPELIN-967 ] Implement search bar in job manager

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

https://github.com/apache/zeppelin/pull/1441
  
Hi @cloverhearts. Thanks for the improvement!
I just tested and got some small feedbacks :)

1.  Is there any particular reason to have dropdown menu next to the filter 
bar? If so, seems need to be sorted to alphabetic order or something 
![screen shot 2016-10-06 at 2 43 29 
pm](https://cloud.githubusercontent.com/assets/10060731/19141916/5d959250-8bd4-11e6-9e0c-d97e79372a23.png)

2. As you can see in the below gif image, there is latency after some 
typing. 

![latency](https://cloud.githubusercontent.com/assets/10060731/19141960/b941aeea-8bd4-11e6-8da4-b15b84cffd37.gif)
We have a good example in Interpreter menu :) The search bar in interpreter 
menu, it shows the searched result right after typing. 

3. The applied font in dropdown menu and the filter bar is different. 
https://cloud.githubusercontent.com/assets/10060731/19142016/2eb386a8-8bd5-11e6-80df-fedc606df0a0.png;>




---
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.
---