AMashenkov commented on a change in pull request #452:
URL: https://github.com/apache/ignite-3/pull/452#discussion_r754101900
##########
File path:
modules/schema/src/main/java/org/apache/ignite/internal/schema/Column.java
##########
@@ -178,6 +181,15 @@ public Object defaultValue() {
return defValSup.get();
}
+ /**
+ * Get no default value flag: {@code true} if column hasn't default value,
{@code false} - otherwise.
+ *
+ * @return {@code true} if column hasn't default value, {@code false} -
otherwise.
Review comment:
So, this method looks useless. If column default value wasn't configured
then
For 'nullable' column - the default is implicit `null`.
For 'non-null' column - explicit null lead to invalid configuration error
and implicit `null` means "no default wasn't set".
--
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]
