arina-ielchiieva commented on a change in pull request #2042: DRILL-7672: Make
metadata type required when reading from / writing into Drill Metastore
URL: https://github.com/apache/drill/pull/2042#discussion_r399689046
##########
File path:
metastore/iceberg-metastore/src/test/java/org/apache/drill/metastore/iceberg/components/tables/TestIcebergTablesMetastore.java
##########
@@ -297,6 +302,7 @@ public void testOverwrite() {
.workspace(tableInfo.workspace())
.tableName(tableInfo.name())
.metadataKey("dir0")
+ .metadataType(MetadataType.TABLE.name())
Review comment:
In `TableMetadataUnit` most of the data is stored as string to be suitable
for storage (for example, in the file), since this is unit test I had to create
`TableMetadataUnit` manually and thus indicate metadata type string value, in
the code we usually do not do this.
----------------------------------------------------------------
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
