Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22501#discussion_r223220176
--- Diff:
core/src/test/scala/org/apache/spark/benchmark/BenchmarkBase.scala ---
@@ -48,15 +48,11 @@ abstract class BenchmarkBase {
if (!file.exists()) {
file.createNewFile()
}
- output = Some(new FileOutputStream(file))
+ output = Option(new FileOutputStream(file))
--- End diff --
Why do you replace `Some` to `Option`? Are you worrying `new
FileOutputStream(file)` becomes `null`?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
