Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Do not open new transaction when getting called.#496

Merged
jgebal merged 3 commits intodevelopfrom
feature/keep_transactions_clean
Oct 18, 2017
Merged

Do not open new transaction when getting called.#496
jgebal merged 3 commits intodevelopfrom
feature/keep_transactions_clean

Conversation

@jgebal
Copy link
Copy Markdown
Member

@jgebal jgebal commented Oct 17, 2017

With this change all calls to ut.run are in autonomous transaction.
This way, utPLSQL does not create a new transaction if a transaction was not opened in the calling session.

Resolves #441

With this change all calls to `ut.run` are in autonomous transaction.
This way, utPLSQL does not create a new transaction if a transaction was not opened in the calling session.
@jgebal jgebal added this to the v3.1.0 milestone Oct 17, 2017
@jgebal jgebal requested review from Pazus, pesse and viniciusam October 17, 2017 22:10
@ghost ghost assigned jgebal Oct 17, 2017
@ghost ghost added the in progress label Oct 17, 2017
# Conflicts:
#	source/api/ut_runner.pkb
Copy link
Copy Markdown
Member

@Pazus Pazus left a comment

Choose a reason for hiding this comment

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

Great job. Approved.

Comment thread source/api/ut.pkb
ut_varchar2_list(sys_context('userenv', 'current_schema')), a_reporter, a_color_console,
a_coverage_schemes, a_source_file_mappings, a_test_file_mappings, a_include_objects, a_exclude_objects
run_autonomous(
a_paths, l_reporter, ut_utils.boolean_to_int(a_color_console), a_coverage_schemes, a_source_file_mappings, a_test_file_mappings,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Change argument to to _a_reporter

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

My bad, skip it.

@ghost ghost assigned Pazus Oct 18, 2017
@jgebal jgebal merged commit a3a0dc7 into develop Oct 18, 2017
@ghost ghost removed the in progress label Oct 18, 2017
@jgebal jgebal deleted the feature/keep_transactions_clean branch October 20, 2017 07:05
@jgebal jgebal modified the milestones: v3.1.0, v3.0.4 Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants