Skip to content

Commit

Permalink
Fix TestSetTablePropertyProcedure
Browse files Browse the repository at this point in the history
  • Loading branch information
ZacBlanco committed Dec 20, 2024
1 parent 5819e85 commit f0a29a0
Showing 1 changed file with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -71,14 +71,14 @@ public void testSetTablePropertyProcedurePositionalArgs()
Table table = loadTable(tableName);
table.refresh();

assertEquals(table.properties().size(), 7);
assertEquals(table.properties().size(), 8);
assertEquals(table.properties().get(propertyKey), null);

assertUpdate(format("CALL system.set_table_property('%s', '%s', '%s', '%s')", TEST_SCHEMA, tableName, propertyKey, propertyValue));
table.refresh();

// now the table property read.split.target-size should have new value
assertEquals(table.properties().size(), 8);
assertEquals(table.properties().size(), 9);
assertEquals(table.properties().get(propertyKey), propertyValue);
}
finally {
Expand All @@ -99,15 +99,15 @@ public void testSetTablePropertyProcedureNamedArgs()
Table table = loadTable(tableName);
table.refresh();

assertEquals(table.properties().size(), 7);
assertEquals(table.properties().size(), 8);
assertEquals(table.properties().get(propertyKey), null);

assertUpdate(format("CALL system.set_table_property(schema => '%s', key => '%s', value => '%s', table_name => '%s')",
TEST_SCHEMA, propertyKey, propertyValue, tableName));
table.refresh();

// now the table property read.split.target-size should have new value
assertEquals(table.properties().size(), 8);
assertEquals(table.properties().size(), 9);
assertEquals(table.properties().get(propertyKey), propertyValue);
}
finally {
Expand All @@ -129,14 +129,14 @@ public void testSetTablePropertyProcedureUpdateExisting()
Table table = loadTable(tableName);
table.refresh();

assertEquals(table.properties().size(), 7);
assertEquals(table.properties().size(), 8);
assertEquals(table.properties().get(propertyKey), "4");

assertUpdate(format("CALL system.set_table_property('%s', '%s', '%s', '%s')", TEST_SCHEMA, tableName, propertyKey, propertyValue));
table.refresh();

// now the table property commit.retry.num-retries should have new value
assertEquals(table.properties().size(), 7);
assertEquals(table.properties().size(), 8);
assertEquals(table.properties().get(propertyKey), propertyValue);
}
finally {
Expand Down

0 comments on commit f0a29a0

Please sign in to comment.