[GitHub] [incubator-iceberg] rdblue commented on a change in pull request #587: Improve docs on converting the Avro schema

[GitBox <gi...@apache.org>]

rdblue commented on a change in pull request #587: Improve docs on converting the Avro schema
URL: https://github.com/apache/incubator-iceberg/pull/587#discussion_r343262768
 
 

 ##########
 File path: site/docs/spark.md
 ##########
 @@ -15,9 +15,9 @@
  - limitations under the License.
  -->
 
-# Spark
+# Apache Spark
 
-Iceberg uses Spark's DataSourceV2 API for data source and catalog implementations. Spark DSv2 is an evolving API with different levels of support in Spark versions.
+Apache Iceberg uses Spark's DataSourceV2 API for data source and catalog implementations. Spark DSv2 is an evolving API with different levels of support in Spark versions.
 
 Review comment:
   I think this is another example of where we don't want to use "Apache" in headings. I think instead the heading should be "Spark" and the text should be "Iceberg uses Apache Spark's DataSourceV2 API . . .". Iceberg also doesn't need to be "Apache Iceberg" since the first reference is already "Apache Iceberg" in the site header.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org