|
87 KB
This issue is verified on Build no.: ads-windows-x64-18.5.0-dev-8.
This issue is verified on Build no.: ads-windows-x64-18.5.0-dev-12.
This issue is verified on Build no.: ads-windows-x64-18.5.0-dev-12.
Code Reviewed.
svn# 55539, 55530, 55519
Code Reviewed.
svn# 55539, 55530, 55519
svn #55613 - Added check constraint to ERM also.
svn #55613 - Added check constraint to ERM also.
While doing area functional testing, I noticed that the check selection is available before version 7.2. I checked the documentation and it looks like the check constraint was added in 7.2. Would you please put a check in for that and not display it in the constraint drop down if earlier than 7.2?
See VerticaConstraintsNode.java
While doing area functional testing, I noticed that the check selection is available before version 7.2. I checked the documentation and it looks like the check constraint was added in 7.2. Would you please put a check in for that and not display it in the constraint drop down if earlier than 7.2?
See VerticaConstraintsNode.java
Hi Tom
Added Check constraint condition visible for Vertica 7.2 and greater, But as we not able database below Vertica 7.2, we couldn't able to test it.
These changes added at revision: 55614
Hi Tom
Added Check constraint condition visible for Vertica 7.2 and greater, But as we not able database below Vertica 7.2, we couldn't able to test it.
These changes added at revision: 55614
Hi Pankaj,
Why did you create the Cassandra version variable for Vertica? There is already a Vertica version variable listed around line 8000 in ConnectionProperties.java. Please fix this.
Just an observation, the check constraint was implemented in Vertica 7.2.3. We don't have this resolution in ADS so we can just leave it at 7.2.
Thanks, Tom
Hi Pankaj,
Why did you create the Cassandra version variable for Vertica? There is already a Vertica version variable listed around line 8000 in ConnectionProperties.java. Please fix this.
Just an observation, the check constraint was implemented in Vertica 7.2.3. We don't have this resolution in ADS so we can just leave it at 7.2.
Thanks, Tom
Hi Tom
Removed Extra added variable from ConnectionProperties.java.
Thanks
Pankaj
Hi Tom
Removed Extra added variable from ConnectionProperties.java.
Thanks
Pankaj
Issue #14703 |
Closed |
Fixed |
Resolved |
Completion |
No due date |
Fixed Build 18.5 |
No time estimate |
This issue is verified on Build no.: ads-windows-x64-18.5.0-dev-8.