AMashenkov commented on a change in pull request #353:
URL: https://github.com/apache/ignite-3/pull/353#discussion_r715702029
##########
File path:
modules/table/src/main/java/org/apache/ignite/internal/table/KeyValueViewImpl.java
##########
@@ -47,8 +47,8 @@
* @param valueMapper Value class mapper.
* @param tx The transaction.
*/
- public KVViewImpl(InternalTable tbl, SchemaRegistry schemaReg,
KeyMapper<K> keyMapper,
- ValueMapper<V> valueMapper, @Nullable Transaction tx) {
+ public KeyValueViewImpl(InternalTable tbl, SchemaRegistry schemaReg,
KeyMapper<K> keyMapper,
Review comment:
They are useless for binary views. These mappers describes how columns
should be mapped to user POJO fields.
They need for the examples of API usage.
--
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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
