cloud-fan commented on code in PR #43334: URL: https://github.com/apache/spark/pull/43334#discussion_r1377168785
########## sql/api/src/main/scala/org/apache/spark/sql/catalyst/trees/QueryContexts.scala: ########## @@ -128,6 +129,45 @@ case class SQLQueryContext( sqlText.isDefined && originStartIndex.isDefined && originStopIndex.isDefined && originStartIndex.get >= 0 && originStopIndex.get < sqlText.get.length && originStartIndex.get <= originStopIndex.get + } + + override def callSite: String = throw new UnsupportedOperationException +} + +case class DatasetQueryContext( + override val fragment: String, + override val callSite: String) extends QueryContext { + override val contextType = QueryContextType.Dataset + + override def objectType: String = throw new UnsupportedOperationException + override def objectName: String = throw new UnsupportedOperationException + override def startIndex: Int = throw new UnsupportedOperationException + override def stopIndex: Int = throw new UnsupportedOperationException + + override lazy val summary: String = { + val builder = new StringBuilder + builder ++= "== Dataset ==\n" + builder ++= "\"" + + builder ++= fragment + builder ++= "\"" + builder ++= " was called from " Review Comment: shall we add a `\n` before the call site? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org