Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21837#discussion_r205647732
--- Diff:
external/avro/src/main/scala/org/apache/spark/sql/avro/AvroOptions.scala ---
@@ -21,16 +21,18 @@ import org.apache.hadoop.conf.Configuration
import org.apache.spark.internal.Logging
import org.apache.spark.sql.catalyst.util.CaseInsensitiveMap
+import org.apache.spark.sql.internal.SQLConf
/**
* Options for Avro Reader and Writer stored in case insensitive manner.
*/
class AvroOptions(
@transient val parameters: CaseInsensitiveMap[String],
- @transient val conf: Configuration) extends Logging with Serializable {
+ @transient val conf: Configuration,
+ @transient val sqlConf: SQLConf) extends Logging with Serializable {
--- End diff --
We may just get SQLConf by calling `SQLConf.get` without passing it in.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
