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

Skip to content

Updated install script.It now fails on invalid sources after installation#42

Merged
rlove merged 2 commits intoutPLSQL:version3from
jgebal:feature/adding_failure_on_sources_installation
Aug 26, 2016
Merged

Updated install script.It now fails on invalid sources after installation#42
rlove merged 2 commits intoutPLSQL:version3from
jgebal:feature/adding_failure_on_sources_installation

Conversation

@jgebal
Copy link
Copy Markdown
Member

@jgebal jgebal commented Aug 23, 2016

No description provided.

@rlove
Copy link
Copy Markdown
Contributor

rlove commented Aug 24, 2016

Been thinking about this, and what if someone install into an existing schema that already contains invalid objects.

Maybe we should change the query to something like

 select count(1)
    into l_cnt
    from user_errors where name like 'UT%';

@jgebal
Copy link
Copy Markdown
Member Author

jgebal commented Aug 24, 2016

Good idea. I'll fix it

@jgebal
Copy link
Copy Markdown
Member Author

jgebal commented Aug 26, 2016

I did more thinking around the install validation.
First of all, we should keep the 'reporting' select in sync with the 'validation' select.

I'm not sure if we should really take into consideration the fact that the schema where utplsql is getting installed is dirty (contains invalid objects).

The ideal solution would be to only lookup exceptions for objects that are part of utplsql installation.

@rlove rlove merged commit 83624eb into utPLSQL:version3 Aug 26, 2016
@rlove rlove removed the in progress label Aug 26, 2016
@jgebal jgebal deleted the feature/adding_failure_on_sources_installation branch August 26, 2016 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants