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

Add option to disable views export from database #338

Closed
paplorinc opened this issue Feb 4, 2013 · 5 comments
Closed

Add option to disable views export from database #338

paplorinc opened this issue Feb 4, 2013 · 5 comments
Milestone

Comments

@paplorinc
Copy link

@paplorinc paplorinc commented Feb 4, 2013

As I have mentioned in #337 (comment) I would like to have an option for the Maven config to disable any other exports, but database tables (eg. views, triggers, etc.), if possible.

Thanks,
Pap Lőrinc

@paplorinc paplorinc mentioned this issue Feb 4, 2013
@timowest
Copy link
Member

@timowest timowest commented Feb 4, 2013

Ok, this is accessible via the TABLE_TYPE column and can also be specified in the getTables calls.

Maybe the default could be TABLE and VIEW, and via skipViews VIEW handling could be disabled.

@timowest
Copy link
Member

@timowest timowest commented Feb 5, 2013

The export scope can now also be controlled via the exportTables (default: true) and exportViews (default: true) options.

@paplorinc
Copy link
Author

@paplorinc paplorinc commented Feb 5, 2013

Thank you Timo, beautiful work :)!

Btw, an insignificant remark: types.toArray(new String[types.size()]) is slightly better than types.toArray(new String[0])

timowest added a commit that referenced this issue Feb 5, 2013
@timowest
Copy link
Member

@timowest timowest commented Feb 20, 2013

Released in 3.0.0.BETA2

@timowest timowest closed this Feb 20, 2013
@khoran
Copy link

@khoran khoran commented Feb 13, 2014

This change has broken the generation of classes for table returning functions and user defined TYPEs (in postgres). There are more then just tables and views in the world of databases. Can you please add an option to return the original functionality of generating classes for everything? Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants