Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16126 )
Change subject: KUDU-3090 Support backing up ownership info ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/16126/7/java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala File java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala: http://gerrit.cloudera.org:8080/#/c/16126/7/java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala@747 PS7, Line 747: def validateTablesMatch(tableA: String, tableB: String): Unit = { > I'm not sure what you mean Do you mean it was changed between a full backup and a subsequent incremental backup? Metadata changes like that aren't super relevant to test because we always use the metadata from the last incremental backup when restoring. -- To view, visit http://gerrit.cloudera.org:8080/16126 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I963db0a36cd4b7f080944ed46fc4119b1e055143 Gerrit-Change-Number: 16126 Gerrit-PatchSet: 7 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 08 Jul 2020 14:06:50 +0000 Gerrit-HasComments: Yes
