[
https://issues.apache.org/jira/browse/IGNITE-12120?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17000670#comment-17000670
]
Sunny Chan commented on IGNITE-12120:
-------------------------------------
I have also added an extra patch for CassandraSessionImpl which also seems to
be a warning rather than an error - this is due to the fact that when the
exception occurs is throw it will get retried by
GridCacheWriteBehindStore::updateStore and therefore it is not strictly an
"error". But I am happy to restore it if we have decided that it should be an
error.
> Change log level in GridCacheWritebehindStore
> ---------------------------------------------
>
> Key: IGNITE-12120
> URL: https://issues.apache.org/jira/browse/IGNITE-12120
> Project: Ignite
> Issue Type: Bug
> Components: cache
> Affects Versions: 2.7.5
> Reporter: Sunny Chan
> Priority: Trivial
> Time Spent: 10m
> Remaining Estimate: 0h
>
> In the
> [GridCacheWriteBehindStore|https://github.com/apache/ignite/blob/7e73098d4d6e3d5f78326cb11dac7e083a2312dd/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/store/GridCacheWriteBehindStore.java#L893],
> when the updateStore failed to write to underlying store, it logs this as
> error:
> {{LT.error(log, e, "Unable to update underlying store: " + store);}}
> After this line logged the error, it would return false so that it would
> retry the store (by returning false).
> While later on in the updatStore function, when the writeCache overflows, it
> would log this:
> {{log.warning("Failed to update store (value will be lost as current buffer
> size is greater " + …}}
> then it will remove the failed entry.
> I think the severity of the log messages is not right, as the fail update
> would still be retried.
> So I propose to change the log severity level so that the first one would be
> a warn, and second one would be error
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
