[jira] [Commented] (VCL-898) Update utils.pm::insert_request to add an entry to the log table

["Josh Thompson (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/VCL-898?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705027#comment-14705027 ] 

Josh Thompson commented on VCL-898:
-----------------------------------

I prefer the idea of modifying the request table to accept NULL for the logid. That way, reload reservations can continue to stay out of the log and sublog tables.

> Update utils.pm::insert_request to add an entry to the log table
> ----------------------------------------------------------------
>
>                 Key: VCL-898
>                 URL: https://issues.apache.org/jira/browse/VCL-898
>             Project: VCL
>          Issue Type: Sub-task
>          Components: vcld (backend)
>            Reporter: Andy Kurth
>            Assignee: Andy Kurth
>             Fix For: 2.5
>
>
> A constraint was (_correctly_) added to _request.logid_ which causes *utils.pm::insert_request* to fail.  *insert_request* is called from:
> * *image.pm::setup_capture_base_image*
> * *utils.pm::insert_reload_request*
> *insert_request* does not currently attempt to insert an entry to the log table before inserting into the request table but it does accept a _log.id_ argument.  However, _log.id_ is specified as 0 in both locations where it is called.  Since there is no entry in the log table with id=0, the following error is generated when attempting to insert into the request table:
> {panel}
> Cannot add or update a child row: a foreign key constraint fails (`vcl`.`request`, CONSTRAINT `request_ibfk_18` FOREIGN KEY (`logid`) REFERENCES `log` (`id`) ON UPDATE CASCADE)
> {panel}
> *insert_request* either needs to be updated to first add a row to the log table or the request table needs to be modified to allow NULL logid values.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)