ptupitsyn commented on pull request #282: URL: https://github.com/apache/ignite-3/pull/282#issuecomment-906211847
@AMashenkov > We have no internal package in ignite-api module Let's create it. I don't suggest to move `TupleImpl` to another module. > I mean do we need 2 wrappers over the Row: 1-st SchemaAware immutable tuple for using Client handler/protocol, > but wrap to a 2-nd mutable adapter (which is not schema aware) when returning to a user. > > This will make code clean, as for now we have to check if tuple is schema aware and then check if the schema is not 'null'. What do you suggest? `SchemaAware` exists for perf reasons, as you noted above. How do we make the code cleaner and keep the efficient tuple serialization approach? -- 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]
