ygerzhedovich commented on code in PR #1743:
URL: https://github.com/apache/ignite-3/pull/1743#discussion_r1124299997
##########
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");
Review Comment:
```suggestion
throw new IllegalArgumentException("Distribution is not
specified");
```
##########
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");
Review Comment:
```suggestion
throw new IllegalArgumentException("Name is not specified");
```
--
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]
