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

    https://github.com/apache/spark/pull/22458#discussion_r219370221
  
    --- Diff: 
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala 
---
    @@ -467,9 +467,9 @@ private[hive] class HiveClientImpl(
             properties = filteredProperties,
             stats = readHiveStats(properties),
             comment = comment,
    -        // In older versions of Spark(before 2.2.0), we expand the view 
original text and store
    -        // that into `viewExpandedText`, and that should be used in view 
resolution. So we get
    -        // `viewExpandedText` instead of `viewOriginalText` for viewText 
here.
    --- End diff --
    
    This comment is for `viewText`, please rephrase and keep it, thanks!


---

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

Reply via email to