[GitHub] [incubator-livy] jerryshao commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
jerryshao commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243#issuecomment-539780785 Yes, it is super weird, I've never met this issue before, can you reproduce it locally?

[GitHub] [incubator-livy] huianyi commented on issue #238: [LIVY-689] Deliver stage process message to the end user using thriftserver

2019-10-08 Thread GitBox
huianyi commented on issue #238: [LIVY-689] Deliver stage process message to the end user using thriftserver URL: https://github.com/apache/incubator-livy/pull/238#issuecomment-539778627 @yiheng Could you please review the new commit again? All checks have passed now.

[GitHub] [incubator-livy] wypoon commented on issue #241: [LIVY-692][THRIFT] Use configured principal for Kerberos authentication

2019-10-08 Thread GitBox
wypoon commented on issue #241: [LIVY-692][THRIFT] Use configured principal for Kerberos authentication URL: https://github.com/apache/incubator-livy/pull/241#issuecomment-539767924 By impersonation, do you mean that the Spark application Livy spawns (to run queries) is run as the user

[GitHub] [incubator-livy] peter-toth commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
peter-toth commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243#issuecomment-539537422 @jerryshao it run into this weird ``` [ERROR] error: error reading

[GitHub] [incubator-livy] peter-toth edited a comment on issue #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
peter-toth edited a comment on issue #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243#issuecomment-539537422 @jerryshao it run into this weird ``` [ERROR] error: error reading

[GitHub] [incubator-livy] jerryshao closed pull request #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
jerryshao closed pull request #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-livy] jerryshao commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
jerryshao commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243#issuecomment-539476957 Please reopen the PR or push an empty commit to trigger travis. This is an

[GitHub] [incubator-livy] peter-toth opened a new pull request #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
peter-toth opened a new pull request #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243 ## What changes were proposed in this pull request? Upgrade the jackson dependency to 2.9.10 which fixes many CVEs:

[GitHub] [incubator-livy] peter-toth removed a comment on issue #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
peter-toth removed a comment on issue #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243#issuecomment-539468103 recheck this please This is an automated message from the Apache

[GitHub] [incubator-livy] peter-toth commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
peter-toth commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243#issuecomment-539468733 retest this please This is an automated message from the Apache Git

[GitHub] [incubator-livy] peter-toth commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10

2019-10-08 Thread GitBox
peter-toth commented on issue #243: [LIVY-693] Upgrade jackson to 2.9.10 URL: https://github.com/apache/incubator-livy/pull/243#issuecomment-539468103 recheck this please This is an automated message from the Apache Git

[GitHub] [incubator-livy] codecov-io edited a comment on issue #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429

2019-10-08 Thread GitBox
codecov-io edited a comment on issue #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/incubator-livy/pull/244#issuecomment-539467203 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/244?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io commented on issue #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429

2019-10-08 Thread GitBox
codecov-io commented on issue #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/incubator-livy/pull/244#issuecomment-539467203 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/244?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io edited a comment on issue #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429

2019-10-08 Thread GitBox
codecov-io edited a comment on issue #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/incubator-livy/pull/244#issuecomment-539467203 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/244?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-08 Thread GitBox
codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-534452181 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/236?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-08 Thread GitBox
codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-534452181 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/236?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-08 Thread GitBox
codecov-io edited a comment on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-534452181 # [Codecov](https://codecov.io/gh/apache/incubator-livy/pull/236?src=pr=h1) Report > Merging

[GitHub] [incubator-livy] peter-toth opened a new pull request #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429

2019-10-08 Thread GitBox
peter-toth opened a new pull request #244: [LIVY-694] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/incubator-livy/pull/244 ## What changes were proposed in this pull request? Upgrade the jetty dependency to 9.4.18.v20190429 to fix CVEs. ## How was this

[GitHub] [incubator-livy] captainzmc commented on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-08 Thread GitBox
captainzmc commented on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-539443012 > there are still some suggestions not applied.. Sorry for missing some suggestions.

[GitHub] [incubator-livy] mgaido91 commented on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-08 Thread GitBox
mgaido91 commented on issue #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#issuecomment-539370492 there are still some suggestions not applied..

[GitHub] [incubator-livy] mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain

2019-10-08 Thread GitBox
mgaido91 commented on a change in pull request #236: [LIVY-678] Thrift ldap authentication, based on ldapurl, basedn, domain URL: https://github.com/apache/incubator-livy/pull/236#discussion_r332359031 ## File path: server/src/main/scala/org/apache/livy/LivyConf.scala ##

[GitHub] [incubator-livy] mgaido91 commented on issue #241: [LIVY-692][THRIFT] Use configured principal for Kerberos authentication

2019-10-08 Thread GitBox
mgaido91 commented on issue #241: [LIVY-692][THRIFT] Use configured principal for Kerberos authentication URL: https://github.com/apache/incubator-livy/pull/241#issuecomment-539369856 @wypoon have you checked also impersonation? Moreover, I think the current behavior is correct and I

[GitHub] [incubator-livy] jerryshao commented on issue #196: [LIVY-606][LIVY-152] fix the invalid conf of pyspark.archives and sparkr.package

2019-10-08 Thread GitBox
jerryshao commented on issue #196: [LIVY-606][LIVY-152] fix the invalid conf of pyspark.archives and sparkr.package URL: https://github.com/apache/incubator-livy/pull/196#issuecomment-539350608 No any special concern, just need another careful revisit to avoid regression.