szehon-ho commented on code in PR #4709:
URL: https://github.com/apache/iceberg/pull/4709#discussion_r978107679


##########
docs/spark-queries.md:
##########
@@ -394,3 +394,14 @@ 
spark.read.format("iceberg").load("db.table.files").show(truncate = false)
 // Hadoop path table
 
spark.read.format("iceberg").load("hdfs://nn:8020/path/to/table#files").show(truncate
 = false)
 ```
+
+You can also inspect Iceberg metadata tables with the time travel feature:

Review Comment:
   1. I think we tend to avoid second person 'you' in the docs, and just state 
that something can be done.
   
   2. Also what do you think about making a separate section (Time Travel with 
Metadata Tables) and showing both the DF and the SQL way ?  Otherwise people 
may assume its only possible via dataframe.
   
   3.  Also this is unrelated but I realized that this section (Inspecting with 
DataFrames) has the wrong heading level, it should be under the level of 
(Inspecting Tables), as that level is about all metadata tables.  Can you help 
fix it as well?



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