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

Implement EXCLUDE constraint for PostgreSQL #98

Closed
wants to merge 1 commit into from

Conversation

KES777
Copy link
Contributor

@KES777 KES777 commented Oct 21, 2017

This pull request will allow to generate next statement

ALTER TABLE subnets ADD CONSTRAINT subnets_subnet_udx  EXCLUDE USING gist ( subnet inet_ops with && ) WHERE state = 0;

with next syntax:

$sqlt_table->add_constraint(
	name       =>  'subnets_subnet_udx',
	type       =>  'exclude',
	fields     =>  'subnet',
	expression =>  'subnet inet_ops with &&',
	options    =>  { using => 'gist', where => 'state = 0' },
);

@KES777 KES777 closed this Dec 28, 2017
@KES777 KES777 reopened this Dec 28, 2017
@KES777
Copy link
Contributor Author

KES777 commented Dec 28, 2017

Q: What is wrong with perl-5.8 here?
A: Travis-CI team answer:

The command that's failing is "perl Makefile.PL && make test" and it seems to be failing to move beyond some questions posed during the "make". What does it do? Please review the process here, since the issue appears to be coming from your code. You may also try reproducing the problem with our Docker images. https://docs.travis-ci.com/user/common-build-problems/#Troubleshooting-Locally-in-a-Docker-Image

@rabbiveesh
Copy link
Contributor

Hey, this is great, but it's incomplete + i can't push to your branch. I'm gonna close this and start a new PR on top of your commit, and i'll credit you in the changelog.

Thanks so much for contributing, even if there was a 6 year gap til it got picked up!

@KES777
Copy link
Contributor Author

KES777 commented Jan 1, 2024

@rabbiveesh Thanks for this.

@KES777 KES777 deleted the implement_exclude branch January 1, 2024 17:08
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.

None yet

2 participants