waterlx edited a comment on issue #474: Fix typo in the message for refreshing 
table metadata
URL: https://github.com/apache/incubator-iceberg/pull/474#issuecomment-533981704
 
 
   "Refreshing table metadata from new version: aaa" indicates that Iceberg is 
refreshing the table metadata maintained in `TableOperations` (so called 
"currently loaded one", that is, the value of `currentMetadataLocation` in 
`BaseMetastoreTableOperations`) to use the value fetched from the metastore, 
that is, `newLocation` in `refreshFromMetadataLocation()`.
   
   "from new version" still cause confusions IMHO (make me feel like new 
version followed is the source version we are refreshing from, but actually it 
is the target version we are updating currentMetadataLocation to), but which is 
trivial. 

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

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

Reply via email to