Marco Gaido

> Also, I'm not sure about the glassfish dependency you mentioned in the previous comment... If you check https://github.com/apache/incubator-livy/commit/545a5c3017e6daca022a61e8c51dbaefc98f8433, you'll see that I had to introduce a glassfish dependency, which...

not sure how convenient this change is. There may be different solutions, such as retrieving the logs of spark apps directly from spark, where they are separated. I'll leave it...

@jahstreet I am not the best guy to take a look at this honestly. I am reviewing this PR in a few hours, but would be great to have feedbacks...

actually the CI is failing...there are test failures...

@Limmen the CI was failing as stated in my comment. First we need the CI to pass to review a patch. If you want to reopen it and make the...

@meisam this PR looks old and inactive. May you please close it? Thanks.

@tmnd1991 you can't, JIRAs are assigned when related patches are merged and they are resolved

> @mgaido91 I think your mixing up how Spark handles JIRA assignments with Livy's policy. We assign JIRAs once a PR is open for the issue. For some reason though,...

I am not sure about this change, may you please elaborate more on the previous behavior and why it is problematic? Maybe an example or some screenshots would help clarifying....

thanks for the explanation @wypoon. From the PR description I was confused by the sentence " Livy removes trailing zeros ... before storing its string representation ...", so I was...