Github user jaceklaskowski commented on a diff in the pull request:

    https://github.com/apache/spark/pull/17917#discussion_r115711771
  
    --- Diff: 
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaRelation.scala
 ---
    @@ -143,4 +143,6 @@ private[kafka010] class KafkaRelation(
             validateTopicPartitions(partitions, partitionOffsets)
         }
       }
    +
    +  override def toString: String = "kafka"
     }
    --- End diff --
    
    Hehe...been thinking about it too, but thought it'd be too much for such a 
simple change.
    
    It merits a JIRA issue though where one had to find out what and how to 
display. After all, you could read from multiple topics with different Kafka 
brokers and offsets. I was worried it'd be too much to handle as part of this 
issue.
    
    I'm up for working on the other issue, but don't think we should hold this 
one up. WDYT?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to