Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

W-11599455 try scope revamp and ref duke #2386

Open
wants to merge 10 commits into
base: v4.4
Choose a base branch
from

Conversation

dukesphere
Copy link
Contributor

Writer's Quality Checklist

Before merging your PR, did you:

  • Run spell checker
  • Run link checker to check for broken xrefs
  • Check for orphan files
  • Perform a local build and do a final visual check of your content, including checking for:
    • Broken images
    • Dead links
    • Correct rendering of partials if they are used in your content
    • Formatting issues, such as:
      • Misnumbered ordered lists (steps) or incorrectly nested unordered lists
      • Messed up tables
      • Proper indentation
      • Correct header levels
  • Receive final review and signoff from:
    • Technical SME
    • Product Manager
    • Editor or peer reviewer
    • Reporter, if this content is in response to a reported issue (internal or external feedback)
  • If applicable, verify that the software actually got released

@dukesphere dukesphere added the DO NOT MERGE (YET) This is a work in progress. Do not merge nor review these changes yet label Oct 19, 2022
@dukesphere dukesphere requested a review from a team as a code owner October 19, 2022 18:38
@dukesphere dukesphere self-assigned this Oct 19, 2022
@dukesphere dukesphere changed the title W 11599455 try scope revamp and ref duke W-11599455 try scope revamp and ref duke Oct 19, 2022
@dukesphere
Copy link
Contributor Author

W-11599455

Copy link
Contributor Author

@dukesphere dukesphere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

edits

modules/ROOT/pages/try-scope-concept.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/try-scope-concept.adoc Outdated Show resolved Hide resolved

The Try scope treats child operations as a transaction when the Transactional Action (`transactionalAction`) is set to `ALWAYS_BEGIN` or `BEGIN_OR_JOIN`. It can be configured in the following ways:
//TODO: DO WE HAVE A LIST OF COMPONENTS AND OPERATIONS THAT SUPPORT TRANSACTIONS
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, sort of, see "The only components that can define the transaction type are message sources (For example, jms:listener and vm:listener) and the Try scope." in https://docs.mulesoft.com/mule-runtime/4.4/transaction-management#transaction-types


== Error Handling with the Try Scope
The Try scope enables transaction management and error handling for processing within the scope. xref:transaction-management.adoc[Transactions] force a group of processors to succeed or fail as a unit. The scope accepts xref:xa-transactions.adoc[Extended Architecture (XA)] and, by default, xref:single-resource-transaction.adoc[Local (LOCAL)] transactions.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: move and consolidate details about the transaction types into the Transaction section.

Rolls back any transactions, then executes and uses that result to re-throw the existing error, causing its container Try scope's execution to fail.
[source,xml,linenums]
----
<try>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With implicit defaults for the transaction type and action (minus doc:id attribute):

With the attributes...

modules/ROOT/pages/try-scope-concept.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/try-scope-concept.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/try-scope-concept.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/try-scope-concept.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/try-scope-concept.adoc Outdated Show resolved Hide resolved
@@ -1,92 +0,0 @@
= Try Scope XML Reference
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved to main page since it's so short.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE (YET) This is a work in progress. Do not merge nor review these changes yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant