Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1615#discussion_r155310298
--- Diff:
integration/spark-common/src/main/scala/org/apache/carbondata/events/LoadEvents.scala
---
@@ -31,8 +31,10 @@ case class LoadTablePreExecutionEvent(sparkSession:
SparkSession,
carbonLoadModel: CarbonLoadModel,
factPath: String,
isDataFrameDefined: Boolean,
- optionsFinal: scala.collection
- .mutable.Map[String, String]) extends Event with LoadEventInfo
+ optionsFinal: scala.collection.mutable.Map[String, String],
+ // options are needed if we need only the load options given by user
+ options: Map[String, String],
--- End diff --
Please rename the options as userProvidedOptions
---
