[
https://issues.apache.org/jira/browse/GEODE-8953?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17296289#comment-17296289
]
ASF GitHub Bot commented on GEODE-8953:
---------------------------------------
davebarnes97 commented on a change in pull request #6095:
URL: https://github.com/apache/geode/pull/6095#discussion_r588598937
##########
File path: geode-docs/developing/transactions/design_considerations.html.md.erb
##########
@@ -117,6 +118,21 @@ immediately after the commit.
A transaction disables expiration on any region entries affected
by the transaction.
+## <a id="transactions-nontransactions" class="no-quick-link"></a>Mixing
Transactions with Nontransactional Operations
+
+For best performance, non-transactional operations do not acquire the
exclusive locks used to check
+for conflicts in a transaction. A transaction operating on the same data as a
non-transactional actor
+is unable to detect the conflict caused by a non-transactional operation.
+
+A user application should adopt the policy of always using transactions or no
transactions at all
Review comment:
I've replaced this paragraph with:
If using transactions, an application should adopt the policy of designating
certain regions or sets of entries
exclusively for transactional puts, updates, and deletions, so transactional
entries will not be modified by non-transactional.
operations.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> User Guide - re-introduce transaction details regarding non-transactional
> changes
> ---------------------------------------------------------------------------------
>
> Key: GEODE-8953
> URL: https://issues.apache.org/jira/browse/GEODE-8953
> Project: Geode
> Issue Type: Bug
> Components: docs
> Affects Versions: 1.13.1
> Reporter: Dave Barnes
> Assignee: Dave Barnes
> Priority: Major
> Labels: pull-request-available
>
> Community member @alberto.gomez requests that we re-introduce some verbiage
> that was deleted in the fix for "GEODE-5509: Rewrite the docs on transaction."
> The passage to re-instate is:
> "If other, non-transactional sources update the keys the transaction is
> modifying, the changes may intermingle with this transaction’s changes. The
> other sources can include distributions from remote members, loading
> activities, and other direct cache modification calls from the same member.
> When this happens, after your commit finishes, the cache state may not be
> what you expected."
> [~eshu]Â concurs, providing the background explanation:
> To achieve best performance, non-transactional operations do not acquire
> DLock used to check conflicts in a transaction. So transaction will not be
> able to detect the conflict caused by a non transactional operation. It is
> expected that user application always uses transaction or no transaction at
> all, unless user knows that certain regions or set of entries will not be
> modified by operations outside of a transaction.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
