sanpwc commented on a change in pull request #611:
URL: https://github.com/apache/ignite-3/pull/611#discussion_r796522238
##########
File path:
modules/table/src/main/java/org/apache/ignite/internal/configuration/schema/ExtendedTableConfigurationSchema.java
##########
@@ -17,22 +17,22 @@
package org.apache.ignite.internal.configuration.schema;
+import java.util.UUID;
import org.apache.ignite.configuration.annotation.InternalConfiguration;
+import org.apache.ignite.configuration.annotation.InternalId;
import org.apache.ignite.configuration.annotation.NamedConfigValue;
import org.apache.ignite.configuration.annotation.Value;
import org.apache.ignite.configuration.schemas.table.TableConfigurationSchema;
-import org.apache.ignite.configuration.validation.Immutable;
/**
* Extended table configuration schema class.
*/
@InternalConfiguration
// TODO: IGNITE-15480 Add id's to columns in order to properly process column
renaming withing index context.
public class ExtendedTableConfigurationSchema extends TableConfigurationSchema
{
- /** Table id. String representation of {@link
org.apache.ignite.lang.IgniteUuid}. */
- @Value
- @Immutable
- public String id;
+ /** Table id. */
+ @InternalId
Review comment:
What will happen if @InternalId will be used with non UUID field?
--
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]
