LIVY-182. Add SPARK_HOME/conf to classpath when SPARK_CONF_DIR is not…

[jonalter]

… defined

LIVY-182

[codecov-io]

Codecov Report

Merging #296 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #296      +/-   ##
============================================
+ Coverage     70.57%   70.59%   +0.02%     
- Complexity      670      673       +3     
============================================
  Files            92       92              
  Lines          4812     4812              
  Branches        720      720              
============================================
+ Hits           3396     3397       +1     
+ Misses          927      923       -4     
- Partials        489      492       +3

Impacted Files	Coverage Δ	Complexity Δ
...in/java/com/cloudera/livy/rsc/ContextLauncher.java	80.93% <ø> (-2.12%)	19% <ø> (ø)
.../java/com/cloudera/livy/rsc/driver/JobWrapper.java	80.64% <ø> (ø)	8% <ø> (+1%)	✅
...n/java/com/cloudera/livy/rsc/driver/RSCDriver.java	77.58% <ø> (+1.29%)	40% <ø> (+1%)	✅
.../java/com/cloudera/livy/rsc/rpc/RpcDispatcher.java	69.56% <ø> (+3.26%)	20% <ø> (+1%)	✅

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7c434c...f3fcbe8. Read the comment docs.

[alex-the-man]

Mind if I ask what's this change for?
We are planning to add multiple SPARK_HOME supports. We might need to do this in another way so it works with #232.

[jonalter]

You are correct. Also, #232 fixes this issue (see line 207 here: https://github.com/cloudera/livy/pull/232/files#diff-bc70dcf9c8f35cb4efd2f347bb004a9fR193)
Closing this PR and Ticket.