Fixed not null columns

Author: alex268

table/src/main/java/tech/ydb/table/description/TableDescription.java

@@ -128,7 +128,7 @@ public static class Builder {
         private TableTtl ttlSettings = TableTtl.notSet();
         private final List<ChangefeedDescription> changefeeds = new ArrayList<>();
 
-        public Builder withStoreType(StoreType storeType) {
+        public Builder setStoreType(StoreType storeType) {
             this.storeType = storeType;
             return this;
         }

table/src/main/java/tech/ydb/table/impl/BaseSession.java

@@ -91,6 +91,7 @@
 import tech.ydb.table.values.ListValue;
 import tech.ydb.table.values.StructValue;
 import tech.ydb.table.values.TupleValue;
+import tech.ydb.table.values.Type;
 import tech.ydb.table.values.Value;
 import tech.ydb.table.values.proto.ProtoType;
 import tech.ydb.table.values.proto.ProtoValue;
@@ -208,6 +209,9 @@ private static YdbTable.ColumnMeta buildColumnMeta(TableColumn column) {
         if (column.getFamily() != null) {
             builder.setFamily(column.getFamily());
         }
+        if (column.getType().getKind() != Type.Kind.OPTIONAL) {
+            builder.setNotNull(true);
+        }
         return builder.build();
     }
 
@@ -739,10 +743,10 @@ private static TableDescription mapDescribeTable(
         TableDescription.Builder description = TableDescription.newBuilder();
         switch (result.getStoreType()) {
             case STORE_TYPE_ROW:
-                description = description.withStoreType(TableDescription.StoreType.ROWS);
+                description = description.setStoreType(TableDescription.StoreType.ROWS);
                 break;
             case STORE_TYPE_COLUMN:
-                description = description.withStoreType(TableDescription.StoreType.COLUMNS);
+                description = description.setStoreType(TableDescription.StoreType.COLUMNS);
                 break;
             case UNRECOGNIZED:
             case STORE_TYPE_UNSPECIFIED:

table/src/test/java/tech/ydb/table/integration/AlterTableTest.java

@@ -74,6 +74,7 @@ public void alterTableTest() {
 
         TableDescription description = describeResult.getValue();
 
+        Assert.assertEquals(TableDescription.StoreType.ROWS, description.getStoreType());
         Assert.assertEquals(1, description.getColumnFamilies().size());
         Assert.assertEquals(DEFAULT_FAMILY, description.getColumnFamilies().get(0).getName());
 

table/src/test/java/tech/ydb/table/integration/RenameTablesTest.java

@@ -1,12 +1,13 @@
 package tech.ydb.table.integration;
 
 import java.util.Arrays;
+
 import org.junit.Assert;
 import org.junit.ClassRule;
 import org.junit.Test;
+
 import tech.ydb.core.Result;
 import tech.ydb.core.Status;
-
 import tech.ydb.table.SessionRetryContext;
 import tech.ydb.table.description.TableColumn;
 import tech.ydb.table.description.TableDescription;
@@ -53,8 +54,8 @@ public class RenameTablesTest {
     public void testRenameTables() {
         TableDescription tableDescription = TableDescription.newBuilder()
                 .addNullableColumn("id", PrimitiveType.Uint64)
-                .addNullableColumn("code", PrimitiveType.Text)
-                .addNullableColumn("size", PrimitiveType.Float)
+                .addNonnullColumn("code", PrimitiveType.Text)
+                .addNonnullColumn("size", PrimitiveType.Float)
                 .addNullableColumn("created", PrimitiveType.Timestamp)
                 .addNullableColumn("data", PrimitiveType.Json)
                 .setPrimaryKey("id")