[kudu-CR] This toString is shown in Spark UI SQL view, right now what users see in the Spark UI is just the object default toString ("Scan org.apache.kudu.spark.kudu.KuduRelation@8ad78a") which doesn't provide good information.

["Florentino Sainz (Code Review)" <ge...@cloudera.org>]

Hello Kudu Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/12773

to look at the new patch set (#4).

Change subject: This toString is shown in Spark UI SQL view,  right now what users see in the Spark UI is just the object  default toString ("Scan org.apache.kudu.spark.kudu.KuduRelation@8ad78a")  which doesn't provide good information.
......................................................................

This toString is shown in Spark UI SQL view, 
right now what users see in the Spark UI is just the object 
default toString ("Scan org.apache.kudu.spark.kudu.KuduRelation@8ad78a") 
which doesn't provide good information.

With this change, Spark UI will show the right name 
(right now I'm injecting it in JavaAssist) 
and also the table which is being read
(so users know what the Scanner is actually Scanning).

Spark UI SQL will now show "Scan Kudu my_table" (similar to 
other sources, like Parquet/Hive, which shows "Scan Parquet my_table" ) instead allowing
much easier profiling on complex queries/DFs. So users can see that its
a KUDU source and the name of the table.


Change-Id: I58e11c36a0c7467baf43f99b9db41e2b4643bec8
---
M java/kudu-spark/src/main/scala/org/apache/kudu/spark/kudu/DefaultSource.scala
1 file changed, 8 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/12773/4
-- 
To view, visit http://gerrit.cloudera.org:8080/12773
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I58e11c36a0c7467baf43f99b9db41e2b4643bec8
Gerrit-Change-Number: 12773
Gerrit-PatchSet: 4
Gerrit-Owner: Florentino Sainz <ti...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)