Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2357#discussion_r192974805
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonSession.scala ---
@@ -171,19 +170,24 @@ class CarbonSession(@transient val sc: SparkContext,
*/
private def trySearchMode(qe: QueryExecution, sse: SQLStart): DataFrame
= {
val analyzed = qe.analyzed
+ val LOG: LogService =
LogServiceFactory.getLogService(classOf[CarbonSession].getName)
analyzed match {
case _@Project(columns, _@Filter(expr, s: SubqueryAlias))
if s.child.isInstanceOf[LogicalRelation] &&
s.child.asInstanceOf[LogicalRelation].relation
.isInstanceOf[CarbonDatasourceHadoopRelation] =>
+ LOG.info(s"Search service started and supports: ${sse.sqlText}")
runSearch(analyzed, columns, expr,
s.child.asInstanceOf[LogicalRelation])
case gl@GlobalLimit(_, ll@LocalLimit(_, p@Project(columns,
_@Filter(expr, s: SubqueryAlias))))
if s.child.isInstanceOf[LogicalRelation] &&
s.child.asInstanceOf[LogicalRelation].relation
.isInstanceOf[CarbonDatasourceHadoopRelation] =>
val logicalRelation = s.child.asInstanceOf[LogicalRelation]
+ LOG.info(s"Search service started and supports: ${sse.sqlText}")
--- End diff --
It can't get the sse.sqlText if move the log to runSearch, except change
the runSearch parameter.
I think it's better to keep the log in trySearchMode and add some
Distinguish for filter and limit.
---
