Improved installation routine #384
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
addressed utPLSQL#383
1 similar comment
2 similar comments
1 similar comment
jgebal
requested changes
Jul 3, 2017
| @@ -1,4 +1,9 @@ | |||
| create table plsql_profiler_runs | |||
| declare | |||
There was a problem hiding this comment.
I'm still thinking if we should make scripts rerunable or drop objects on uninstall.
Also, since we modify script provided from Oracle, should we split it into one script per object?
There was a problem hiding this comment.
I think we shouldn't drop it. Could you explain why do you want to split it?
There was a problem hiding this comment.
We use a pattern of one script for one object so this script is the only one breaking the pattern
There was a problem hiding this comment.
but its an oracle object, not our... But I can fix it tonight
jgebal
reviewed
Jul 3, 2017
source/core/coverage/proftab.sql
Outdated
| l_tab_exist number; | ||
| begin | ||
| select count(*) into l_tab_exist from | ||
| (select table_name from all_tables where table_name = 'PLSQL_PROFILER_RUNS' and owner in (user,'PUBLIC') |
There was a problem hiding this comment.
We cannot have any table with owner = 'PUBLIC'
jgebal
reviewed
Jul 3, 2017
source/core/coverage/proftab.sql
Outdated
| l_tab_exist number; | ||
| begin | ||
| select count(*) into l_tab_exist from | ||
| (select table_name from all_tables where table_name = 'PLSQL_PROFILER_DATA' and owner in (user,'PUBLIC'); |
jgebal
reviewed
Jul 3, 2017
source/core/coverage/proftab.sql
Outdated
| l_seq_exist number; | ||
| begin | ||
| select count(*) into l_seq_exist from | ||
| (select sequence_name from all_sequences where sequence_name = 'PLSQL_PROFILER_RUNNUMBER' and sequence_owner in (user,'PUBLIC'); |
Fixed syntax errors. Fixed bug when deploying to different user schema
2 similar comments
jgebal
approved these changes
Jul 4, 2017
Pazus
added a commit
that referenced
this pull request
Jul 18, 2017
### Documentation #399 Documentation now refers to [migration](https://github.com/utPLSQL/utPLSQL-v2-v3-migration) project #386 Documentation now refers to a valid object name: `ut_file_mapping` #362 Install and Uninstall scripts are now much more readable #361 Install guide now provides snippet on how to download latest release on Windows ### Installation #396 Added override user/password/tablespace for install_headless #384 Installation is now smooth even if profiler tables already exist ### Internal improvements #388 Improved reporting from RunTest #363 Fixed publishing of release documentation history ### Improvements and fixes #407 Fixed rare issue with `ORA-22813: operand value exceeds system limits` #403 Stack trace is now properly parsed on all machines #397 The `--%disabled` annotation on suite level is now reporting all tests as disabled #395 Coverage reporting is now properly filtering test packages on that use suitepath #390 Line of code for failed expectation is now also shown for unit tests owned by other users #380 Line no of failed test is now properly reported when using `ut.fail` #375 Annotation parameter list can now have spaces before/after brackets #373 Warnings in documentation reporter are now properly numbered #372 Documentation reporter is now providing a timing information for each test #370 by xUnit reporter now displays name of the package/procedure if suite/test has no description #369 Fixed errors with multi-byte characters in conversion from/to clob
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
addressed #383
addressed #382