[jira] [Created] (IGNITE-19789) Sql. Introduce RowSchema for RowFactory

["Konstantin Orlov (Jira)" <ji...@apache.org>]

Konstantin Orlov created IGNITE-19789:
-----------------------------------------

             Summary: Sql. Introduce RowSchema for RowFactory
                 Key: IGNITE-19789
                 URL: https://issues.apache.org/jira/browse/IGNITE-19789
             Project: Ignite
          Issue Type: Improvement
          Components: sql
            Reporter: Konstantin Orlov


There are three way to create {{org.apache.ignite.internal.sql.engine.exec.RowHandler.RowFactory}}:

{code:java}
RowFactory<RowT> factory(IgniteTypeFactory typeFactory, RelDataType rowType);
RowFactory<RowT> factory(IgniteTypeFactory typeFactory, List<RelDataType> fieldTypes);
RowFactory<RowT> factory(Type... types);
{code}

The first two create unnecessary dependency on {{calcite}} library, the last one doesn't provide required type's parameters, like decimal scale, for instance.

Let's replace all three methods with a single {{RowFactory<RowT> factory(RowSchema schema)}}, where {{RowSchema}} is a class that should be introduced.

h4. Implementation Notes

Although {{org.apache.ignite.internal.schema.BinaryTupleSchemas}} might seem like a good candidate on the role of RowSchema, it doesn't have exhaustive type's support. Besides, introducing new type to binary tuple is much complex procedure since it will affect every module. Thus, it's better to implement distinct schema optimised for RowHandler|RowFactory use case.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)