ygerzhedovich commented on code in PR #1743:
URL: https://github.com/apache/ignite-3/pull/1743#discussion_r1124300502
##########
modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/framework/TestBuilders.java:
##########
@@ -311,6 +311,18 @@ private static class TableBuilderImpl extends
AbstractTableBuilderImpl<TableBuil
/** {@inheritDoc} */
@Override
public TestTable build() {
+ if (distribution == null) {
+ throw new IllegalArgumentException("distribution is not
specified");
+ }
+
+ if (name == null) {
+ throw new IllegalArgumentException("name is not specified");
+ }
+
+ if (columns.isEmpty()) {
+ throw new IllegalArgumentException("table must contain at
least one column");
Review Comment:
```suggestion
throw new IllegalArgumentException("Table must contain at
least one column");
```
--
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]
