{"issue":{"milestone":{"open_issues":1,"title":"0.9.3","created_at":"2011-12-30T02:11:03Z","creator":{"login":"dynamobi-build","id":1123017,"url":"https://api.github.com/users/dynamobi-build","avatar_url":"https://secure.gravatar.com/avatar/cb215b3d786be20761071a562a56af64?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"cb215b3d786be20761071a562a56af64"},"state":"open","due_on":null,"closed_issues":18,"number":7,"description":null,"url":"https://api.github.com/repos/dynamobi/dynamobi/milestones/7"},"title":"[FRG-397] Improve error message when interval field value does not match precision","created_at":"2011-12-30T03:11:31Z","user":{"login":"dynamobi-build","id":1123017,"url":"https://api.github.com/users/dynamobi-build","avatar_url":"https://secure.gravatar.com/avatar/cb215b3d786be20761071a562a56af64?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"cb215b3d786be20761071a562a56af64"},"updated_at":"2011-12-30T03:11:36Z","state":"closed","labels":[{"name":"FRG","url":"https://api.github.com/repos/dynamobi/dynamobi/labels/FRG","color":"1f8f0d"},{"name":"SQL Validator","url":"https://api.github.com/repos/dynamobi/dynamobi/labels/SQL+Validator","color":"19b54d"}],"id":2685742,"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"closed_at":"2011-12-30T03:11:33Z","html_url":"https://github.com/dynamobi/dynamobi/issues/378","body":"<b><i>[reporter=\"jhyde\", created=\"Thu, 13 May 2010 15:11:11 -0500 (GMT-05:00)\"]</i></b>\nSQL:2008 specification part 2, 4.6.3 Intervals says:\n<br/>\n\n<br/>\nWithin a value of type interval, the first field is constrained only by the <interval leading field precision> of\n<br/>\nthe associated <interval qualifier>. Table 6, \"Valid values for fields in INTERVAL values\", specifies the constraints\n<br/>\non subsequent field values.\n<br/>\n\n<br/>\nThus:\n<br/>\n* INTERVAL '90' SECOND is legal\n<br/>\n* INTERVAL '120' SECOND is legal\n<br/>\n* INTERVAL '86400' SECOND is legal\n<br/>\n* INTERVAL '1:30' MINUTE TO SECOND is legal\n<br/>\n* INTERVAL '1:90' SECOND is illegal\n<br/>\n* INTERVAL '1:-30' SECOND is illegal\n<br/>\n* INTERVAL '1:59.999' SECOND is illegal\n<br/>\n* INTERVAL '1:59:59' HOUR TO SECOND is legal\n<br/>\n* INTERVAL '1:59:61' HOUR TO SECOND is illegal\n<br/>\n* INTERVAL '1:61:59' HOUR TO SECOND is illegal\n<br/>\n\n<br/>\nIn particular,\n<br/>\n\n<br/>\n> VALUES INTERVAL '120' SECOND;\n<br/>\n\n<br/>\ngives\n<br/>\n\n<br/>\n> Error: from line 1, column 8 to line 1, column 28: Illegal interval literal format '120' for INTERVAL SECOND\n<br/>\n\n<br/>\nbut it should succeed.","number":378,"url":"https://api.github.com/repos/dynamobi/dynamobi/issues/378","comments":3,"assignee":null},"comment":{"created_at":"2011-12-30T03:11:35Z","user":{"login":"dynamobi-build","id":1123017,"url":"https://api.github.com/users/dynamobi-build","avatar_url":"https://secure.gravatar.com/avatar/cb215b3d786be20761071a562a56af64?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"cb215b3d786be20761071a562a56af64"},"updated_at":"2011-12-30T03:11:35Z","id":3309531,"body":"<b><i>[author=\"jhyde\", created=\"Thu, 13 May 2010 15:40:24 -0500 (GMT-05:00)\"]</i></b>\nEric, I agree with your interpretation of the spec. I think we should keep the default leading precision at 2, per the spec.\n<br/>\n\n<br/>\nI would change the error message to\n<br/>\n\n<br/>\nInterval field value '120' exceeds precision 2 for SECOND field\n<br/>\n\n<br/>\nso that people have a clue that they can change the precision. I am downgrading this to minor bug.","url":"https://api.github.com/repos/dynamobi/dynamobi/issues/comments/3309531"},"action":"created"}