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 (#5).

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
(I've tested using Javaassist code-injection in my cloudera cluster,
as I don't have the tools to build this library and I can't upgrade
the library there anyways)
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/5
--
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: 5
Gerrit-Owner: Florentino Sainz <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to