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



##########
File path: docs/versioned/spark/spark-queries.md
##########
@@ -168,7 +168,7 @@ To inspect a table's history, snapshots, and other 
metadata, Iceberg supports me
 Metadata tables are identified by adding the metadata table name after the 
original table name. For example, history for `db.table` is read using 
`db.table.history`.
 
 {{< hint info >}}
-As of Spark 3.0, the format of the table name for inspection 
(`catalog.database.table.metadata`) doesn't work with Spark's default catalog 
(`spark_catalog`). If you've replaced the default catalog, you may want to use 
`DataFrameReader` API to inspect the table. 
+In Spark 3.0 and 3.1, the Spark session catalog (`spark_catalog`) does not 
support table names with multipart identifiers such as 
`catalog.database.table.metadata`. To work around this, for querying metadata 
tables, configure a different catalog that uses the Iceberg `SparkCatalog` 
class, or use the Spark `DataFrameReader` API. From Spark 3.2 onwards, the 
session catalog supports table names with multipart identifiers.

Review comment:
       Nit: Consider `Prior to Spark 3.2` instead of `Spark 3.0 and Spark 3.1`.
   
   I don't think this applies at all to Spark 2.4, but I can still imagine 
getting questions about it.




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