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

    https://github.com/apache/spark/pull/22133#discussion_r211537581
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
 ---
    @@ -626,6 +626,7 @@ object DataSource extends Logging {
           
serviceLoader.asScala.filter(_.shortName().equalsIgnoreCase(provider1)).toList 
match {
             // the provider format did not match any given registered aliases
             case Nil =>
    +          val latestDocsURL = "https://spark.apache.org/docs/latest";
    --- End diff --
    
    But what if we happen to have more information specific to newer versions 
..? For example, we could happen to have different group name rule, etc. in the 
future. Avoiding pointing out the latest shouldn't be too difficult .. You 
could just say "Please refer deployment section in Apache Avro data source 
guide.". if we happen to have other new changes specific to newer versions 
there, we should go down and fix the links in all the branches strictly in 
theory.


---

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

Reply via email to