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.

Sign up for GitHub

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

Jump to bottom

[IDEA] : we have to add UUID type as well to this list #212

Open
bheemreddy181-zz opened this issue Jun 10, 2019 · 3 comments
Open

[IDEA] : we have to add UUID type as well to this list #212

bheemreddy181-zz opened this issue Jun 10, 2019 · 3 comments

Comments

@bheemreddy181-zz
Copy link

UUID is also a type where people are referring to for Primary Keys https://medium.com/@dylanwolff/uuid-primary-keys-with-postgres-and-rom-958492e223ea

schemacrawler-additional-lints/src/main/java/io/github/mbarre/schemacrawler/utils/LintUtils.java

Line 85 in de95837

public static final boolean isSqlTypeIntegerBased(int javaSqlType) {
do we want to make this method generic and change the lint name ?
@bheemreddy181-zz
Copy link
Author

@adriens any thoughts , i can make a PR

@adriens
Copy link
Collaborator

adriens commented Jun 26, 2019

Hi @bheemreddy181 , sorry for late answer I'm very busy and cannot spend as much time as required on this projetc. ..so does @mbarre .

Yes, I think this could be a good Idea to add UUID as a PK type. Agree @mbarre ?

@adriens
Copy link
Collaborator

adriens commented Feb 21, 2021

See #267

@adriens adriens pinned this issue Feb 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bheemreddy181-zz @adriens