[
https://issues.apache.org/jira/browse/GORA-419?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14496449#comment-14496449
]
ASF GitHub Bot commented on GORA-419:
-------------------------------------
Github user hsaputra commented on a diff in the pull request:
https://github.com/apache/gora/pull/23#discussion_r28435053
--- Diff:
gora-accumulo/src/main/java/org/apache/gora/accumulo/store/AccumuloStore.java
---
@@ -718,12 +718,13 @@ else if (effectiveSchema.getType() == Type.MAP) {
}
}
- private int putMap(Mutation m, int count, Schema valueType, Object o,
Pair<Text, Text> col) throws GoraException {
+ private int putMap(Mutation m, int count, Schema valueType, Object o,
Pair<Text, Text> col,
--- End diff --
With the update, the Pair for col only used to get the name, could we just
use the fieldName instead for the Query?
I.E remove the col input param in putMap and putArray.
> putMap and putArray use column instead of field name to deleteByQuery
> ---------------------------------------------------------------------
>
> Key: GORA-419
> URL: https://issues.apache.org/jira/browse/GORA-419
> Project: Apache Gora
> Issue Type: Bug
> Components: gora-accumulo
> Affects Versions: 0.6
> Reporter: Gerhard Gossen
>
> In {{AccumuloStore.put(k, v)}} fields of type MAP or ARRAY are cleared first
> before they are set to the new value. This is done in the methods
> {{putMap}}/{{putArray}} using a call to {{deleteByQuery(q)}}. The name for
> fields to be deleted is taken from the current column. However,
> {{deleteByQuery}} tries to translate the field names of the query to column
> names again, which fails with a log message like
> {code}
> 2015-04-13 13:43:35.084 ERROR 16733 --- [ool-46-thread-1]
> o.a.gora.accumulo.store.AccumuloStore : Mapping not found for field: ol
> 2015-04-13 13:43:35.104 ERROR 16733 --- [ool-46-thread-1]
> o.a.gora.accumulo.store.AccumuloStore : Mapping not found for field: mk
> 2015-04-13 13:43:35.115 ERROR 16733 --- [ool-46-thread-1]
> o.a.gora.accumulo.store.AccumuloStore : Mapping not found for field: mtdt
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
