[GitHub] [incubator-livy] codecov-io edited a comment on issue #155: [LIVY-565] Ensure column buffer for string and binary contains all data.

2019-03-14 Thread GitBox
codecov-io edited a comment on issue #155: [LIVY-565] Ensure column buffer for 
string and binary contains all data.
URL: https://github.com/apache/incubator-livy/pull/155#issuecomment-473040479
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=h1) 
Report
   > Merging 
[#155](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/57677897b486bab5bb39e78eb0d7f734262411ae?src=pr=desc)
 will **decrease** coverage by `0.12%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/155/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=tree)
   
   ```diff
   @@ Coverage Diff @@
   ## master#155  +/-   ##
   ===
   - Coverage 68.82%   68.7%   -0.13% 
   - Complexity  906 908   +2 
   ===
 Files   100 100  
 Lines  56625662  
 Branches848 848  
   ===
   - Hits   38973890   -7 
   - Misses 12141221   +7 
 Partials551 551
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `73.23% <0%> (-6.34%)` | `33% <0%> (ø)` | |
   | 
[...in/java/org/apache/livy/rsc/driver/JobWrapper.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvSm9iV3JhcHBlci5qYXZh)
 | `82.85% <0%> (-5.72%)` | `8% <0%> (-1%)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.33% <0%> (-3.34%)` | `7% <0%> (ø)` | |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `80.08% <0%> (+1.27%)` | `43% <0%> (+2%)` | :arrow_up: |
   | 
[...c/main/scala/org/apache/livy/repl/ReplDriver.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cmVwbC9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpdnkvcmVwbC9SZXBsRHJpdmVyLnNjYWxh)
 | `33.33% <0%> (+2.56%)` | `7% <0%> (ø)` | :arrow_down: |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `55.88% <0%> (+2.94%)` | `7% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=footer).
 Last update 
[5767789...b9e3c18](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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] [incubator-livy] vanzin opened a new pull request #157: [BUILD] Run ITs with the correct version of Spark.

2019-03-14 Thread GitBox
vanzin opened a new pull request #157: [BUILD] Run ITs with the correct version 
of Spark.
URL: https://github.com/apache/incubator-livy/pull/157
 
 
   All IT runs were running with Spark 2.2...
   


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] [incubator-livy] vanzin commented on issue #119: [LIVY-527] added autocompletion api and implementation for ipython

2019-03-14 Thread GitBox
vanzin commented on issue #119: [LIVY-527] added autocompletion api and 
implementation for ipython
URL: https://github.com/apache/incubator-livy/pull/119#issuecomment-473091585
 
 
   Sorry, totally missed your question. I'm also not sure I understand the 
question.
   
   It would be really good to have some test for this. It doesn't need to be an 
integration tests, but it needs to be tested somehow. Otherwise the changes 
look good.


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] [incubator-livy] ajbozarth commented on issue #135: [LIVY-544] Allow interpreterExecutor run in ThreadPool

2019-03-14 Thread GitBox
ajbozarth commented on issue #135: [LIVY-544] Allow interpreterExecutor run in 
ThreadPool
URL: https://github.com/apache/incubator-livy/pull/135#issuecomment-473070449
 
 
   I see no issues with the code, but since I don't have any experience with 
the SQL side of Livy I will leave final review and merging to a more 
knowledgable committer 


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] [incubator-livy] codecov-io edited a comment on issue #155: [LIVY-565] Ensure column buffer for string and binary contains all data.

2019-03-14 Thread GitBox
codecov-io edited a comment on issue #155: [LIVY-565] Ensure column buffer for 
string and binary contains all data.
URL: https://github.com/apache/incubator-livy/pull/155#issuecomment-473040479
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=h1) 
Report
   > Merging 
[#155](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/57677897b486bab5bb39e78eb0d7f734262411ae?src=pr=desc)
 will **decrease** coverage by `0.28%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/155/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #155  +/-   ##
   
   - Coverage 68.82%   68.54%   -0.29% 
   + Complexity  906  904   -2 
   
 Files   100  100  
 Lines  5662 5662  
 Branches848  848  
   
   - Hits   3897 3881  -16 
   - Misses 1214 1227  +13 
   - Partials551  554   +3
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `73.23% <0%> (-6.34%)` | `33% <0%> (ø)` | |
   | 
[...in/java/org/apache/livy/rsc/driver/JobWrapper.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvSm9iV3JhcHBlci5qYXZh)
 | `82.85% <0%> (-5.72%)` | `8% <0%> (-1%)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.33% <0%> (-3.34%)` | `7% <0%> (ø)` | |
   | 
[.../scala/org/apache/livy/sessions/SessionState.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-Y29yZS9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpdnkvc2Vzc2lvbnMvU2Vzc2lvblN0YXRlLnNjYWxh)
 | `58.33% <0%> (-2.78%)` | `2% <0%> (ø)` | |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `73.91% <0%> (-1.25%)` | `20% <0%> (ø)` | |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `77.96% <0%> (-0.85%)` | `41% <0%> (ø)` | |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `55.88% <0%> (+2.94%)` | `7% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=footer).
 Last update 
[5767789...aabb6c9](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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] [incubator-livy] codecov-io commented on issue #155: [LIVY-565] Ensure column buffer for string and binary contains all data.

2019-03-14 Thread GitBox
codecov-io commented on issue #155: [LIVY-565] Ensure column buffer for string 
and binary contains all data.
URL: https://github.com/apache/incubator-livy/pull/155#issuecomment-473040479
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=h1) 
Report
   > Merging 
[#155](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/57677897b486bab5bb39e78eb0d7f734262411ae?src=pr=desc)
 will **decrease** coverage by `0.28%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/155/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #155  +/-   ##
   
   - Coverage 68.82%   68.54%   -0.29% 
   + Complexity  906  904   -2 
   
 Files   100  100  
 Lines  5662 5662  
 Branches848  848  
   
   - Hits   3897 3881  -16 
   - Misses 1214 1227  +13 
   - Partials551  554   +3
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `73.23% <0%> (-6.34%)` | `33% <0%> (ø)` | |
   | 
[...in/java/org/apache/livy/rsc/driver/JobWrapper.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvSm9iV3JhcHBlci5qYXZh)
 | `82.85% <0%> (-5.72%)` | `8% <0%> (-1%)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.33% <0%> (-3.34%)` | `7% <0%> (ø)` | |
   | 
[.../scala/org/apache/livy/sessions/SessionState.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-Y29yZS9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpdnkvc2Vzc2lvbnMvU2Vzc2lvblN0YXRlLnNjYWxh)
 | `58.33% <0%> (-2.78%)` | `2% <0%> (ø)` | |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `73.91% <0%> (-1.25%)` | `20% <0%> (ø)` | |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `77.96% <0%> (-0.85%)` | `41% <0%> (ø)` | |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `55.88% <0%> (+2.94%)` | `7% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=footer).
 Last update 
[5767789...aabb6c9](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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] [incubator-livy] codecov-io edited a comment on issue #155: [LIVY-565] Ensure column buffer for string and binary contains all data.

2019-03-14 Thread GitBox
codecov-io edited a comment on issue #155: [LIVY-565] Ensure column buffer for 
string and binary contains all data.
URL: https://github.com/apache/incubator-livy/pull/155#issuecomment-473040479
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=h1) 
Report
   > Merging 
[#155](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=desc) 
into 
[master](https://codecov.io/gh/apache/incubator-livy/commit/57677897b486bab5bb39e78eb0d7f734262411ae?src=pr=desc)
 will **decrease** coverage by `0.28%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-livy/pull/155/graphs/tree.svg?width=650=0MkVbiUFwE=150=pr)](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master #155  +/-   ##
   
   - Coverage 68.82%   68.54%   -0.29% 
   + Complexity  906  904   -2 
   
 Files   100  100  
 Lines  5662 5662  
 Branches848  848  
   
   - Hits   3897 3881  -16 
   - Misses 1214 1227  +13 
   - Partials551  554   +3
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=tree) | 
Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...ain/scala/org/apache/livy/utils/SparkYarnApp.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2VydmVyL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS91dGlscy9TcGFya1lhcm5BcHAuc2NhbGE=)
 | `73.23% <0%> (-6.34%)` | `33% <0%> (ø)` | |
   | 
[...in/java/org/apache/livy/rsc/driver/JobWrapper.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvSm9iV3JhcHBlci5qYXZh)
 | `82.85% <0%> (-5.72%)` | `8% <0%> (-1%)` | |
   | 
[...ala/org/apache/livy/scalaapi/LivyScalaClient.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9MaXZ5U2NhbGFDbGllbnQuc2NhbGE=)
 | `83.33% <0%> (-3.34%)` | `7% <0%> (ø)` | |
   | 
[.../scala/org/apache/livy/sessions/SessionState.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-Y29yZS9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpdnkvc2Vzc2lvbnMvU2Vzc2lvblN0YXRlLnNjYWxh)
 | `58.33% <0%> (-2.78%)` | `2% <0%> (ø)` | |
   | 
[...c/src/main/java/org/apache/livy/rsc/RSCClient.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9SU0NDbGllbnQuamF2YQ==)
 | `73.91% <0%> (-1.25%)` | `20% <0%> (ø)` | |
   | 
[...ain/java/org/apache/livy/rsc/driver/RSCDriver.java](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-cnNjL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9saXZ5L3JzYy9kcml2ZXIvUlNDRHJpdmVyLmphdmE=)
 | `77.96% <0%> (-0.85%)` | `41% <0%> (ø)` | |
   | 
[...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala](https://codecov.io/gh/apache/incubator-livy/pull/155/diff?src=pr=tree#diff-c2NhbGEtYXBpL3NyYy9tYWluL3NjYWxhL29yZy9hcGFjaGUvbGl2eS9zY2FsYWFwaS9TY2FsYUpvYkhhbmRsZS5zY2FsYQ==)
 | `55.88% <0%> (+2.94%)` | `7% <0%> (ø)` | :arrow_down: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=footer).
 Last update 
[5767789...aabb6c9](https://codecov.io/gh/apache/incubator-livy/pull/155?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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] [incubator-livy] vanzin opened a new pull request #156: [LIVY-566] Cancel Spark jobs related to statement on cancellation.

2019-03-14 Thread GitBox
vanzin opened a new pull request #156: [LIVY-566] Cancel Spark jobs related to 
statement on cancellation.
URL: https://github.com/apache/incubator-livy/pull/156
 
 
   


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] [incubator-livy] vanzin closed pull request #151: [LIVY-566] When cancel a sql job through livy, actually the sparkjob still running

2019-03-14 Thread GitBox
vanzin closed pull request #151: [LIVY-566] When cancel a sql job through livy, 
actually the sparkjob still running 
URL: https://github.com/apache/incubator-livy/pull/151
 
 
   


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] [incubator-livy] vanzin commented on issue #151: [LIVY-566] When cancel a sql job through livy, actually the sparkjob still running

2019-03-14 Thread GitBox
vanzin commented on issue #151: [LIVY-566] When cancel a sql job through livy, 
actually the sparkjob still running 
URL: https://github.com/apache/incubator-livy/pull/151#issuecomment-473022594
 
 
   Closing due to no reply. I'll post my own fix.


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] [incubator-livy] Tagar commented on issue #135: [LIVY-544] Allow interpreterExecutor run in ThreadPool

2019-03-14 Thread GitBox
Tagar commented on issue #135: [LIVY-544] Allow interpreterExecutor run in 
ThreadPool
URL: https://github.com/apache/incubator-livy/pull/135#issuecomment-472971878
 
 
   @WeiWenda I am not a committer. 
   
   Thanks for this great improvement. It would be awesome to have concurrentSQL
   option in Livy, like we  have in bare Zeppelin. 
   
   cc @mgaido91 @jerryshao @vanzin @ajbozarth @zjffdu - please help to review. 
   
   Thank you. 


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] [incubator-livy] vanzin commented on issue #149: [LIVY-565] Fix the bug that sql query row count is less than the actual row count of the table at some condition

2019-03-14 Thread GitBox
vanzin commented on issue #149: [LIVY-565] Fix the bug that sql query row count 
is less than the actual row count of the table at some condition
URL: https://github.com/apache/incubator-livy/pull/149#issuecomment-472954995
 
 
   I'm closing this for lack of response. I'll post my own fix.


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] [incubator-livy] vanzin closed pull request #149: [LIVY-565] Fix the bug that sql query row count is less than the actual row count of the table at some condition

2019-03-14 Thread GitBox
vanzin closed pull request #149: [LIVY-565] Fix the bug that sql query row 
count is less than the actual row count of the table at some condition
URL: https://github.com/apache/incubator-livy/pull/149
 
 
   


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