kbendick commented on a change in pull request #3976:
URL: https://github.com/apache/iceberg/pull/3976#discussion_r791367478



##########
File path: site/docs/aws.md
##########
@@ -444,6 +444,9 @@ spark-sql --packages 
org.apache.iceberg:iceberg-spark3-runtime:{{ versions.icebe
 [Hive](https://docs.aws.amazon.com/emr/latest/ReleaseGuide/emr-hive.html), 
[Flink](https://docs.aws.amazon.com/emr/latest/ReleaseGuide/emr-flink.html),
 [Trino](https://docs.aws.amazon.com/emr/latest/ReleaseGuide/emr-presto.html) 
that can run Iceberg.
 
+Recently, Amazon EMR 6.5.0 
[announced](https://aws.amazon.com/about-aws/whats-new/2022/01/amazon-emr-supports-apache-iceberg/)
 support of Apache Iceberg's Spark 3 Runtime.
+With the current release, you can use Apache Spark 3.1.2 on EMR clusters with 
the Iceberg table format. Please refer the [official 
documentation](https://docs.aws.amazon.com/emr/latest/ReleaseGuide/emr-iceberg-create-cluster.html)
 to create a cluster with Iceberg installed.

Review comment:
       So I understand this better myself, “the current release” is referring 
to EMR 6.5.0 (or EMR in general)?
   
   I would try to leave the version compatability information in the EMR docs 
personally. EMR has a large number of releases that each support a unique set 
of software. IIRC, There are really good docs for seeing what is supported 
where.
   
   So I would be in favor of deferring to the EMR docs for the exact 
combination of dependencies supported etc.
   
   Maybe `Starting with EMR version 6.5.0, EMR clusters officially support the 
Apache Iceberg table format. EMR can be configured to have the necessary Apache 
Iceberg dependencies installed without requiring the user to use additional 
bootstrap actions to configure the cluster. Please refer to the official 
documentation on how to create a cluster with Iceberg installed.`
   
   Additionally, links to some sort of version support / dependency matrix 
would go a long way. Given that this information is already well documented for 
EMR for many things, it seems best to keep that there if possible as the 
support matrix is quite large and changes over time.




-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to