<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
<br>
<div class="moz-cite-prefix">On 04/11/2018 10:34 AM, Austin
Macdonald wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CA+Kyt=pK4bFup2a_rWHj61xQ5cycqZ10Ma6nSqboHZ9er=iM=A@mail.gmail.com">
<div dir="ltr">
<div><font face="arial, helvetica, sans-serif"><span
style="font-size:12.8px">From our checkin meeting, there
was an MVP doc question that needed some discussion:</span></font></div>
<b
style="font-family:arial,helvetica,sans-serif;font-size:12.8px"><br>
Publications:</b><span
style="font-family:arial,helvetica,sans-serif;font-size:12.8px"> </span><a
href="https://pulp.plan.io/projects/pulp/wiki/Pulp_3_Minimum_Viable_Product#Publications"
target="_blank"
style="font-family:arial,helvetica,sans-serif;font-size:12.8px"
moz-do-not-send="true">https://pulp.pla<wbr>n.io/projects/pulp/wiki/Pulp_3<wbr>_Minimum_Viable_Product#Public<wbr>ations</a><br>
<ul
style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration-style:initial;text-decoration-color:initial">
<ul>
<li><i
style="font-family:arial,helvetica,sans-serif;font-size:12.8px">As
a user, As an authenticated user, I can delete
publications.</i></li>
<ul>
<li><i style="font-size:12.8px"><font face="arial,
helvetica, sans-serif">asynchronously with a lock on
the repository version. </font></i></li>
<li><i style="font-size:12.8px"><font face="arial,
helvetica, sans-serif">prevented if associated with
a distribution.</font></i></li>
<li><i style="font-size:12.8px"><font face="arial,
helvetica, sans-serif">single object only.</font></i></li>
</ul>
</ul>
</ul>
<blockquote
style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration-style:initial;text-decoration-color:initial;margin:0px
0px 0px 40px;border:none;padding:0px">In the code, Publication
deletes are synchronous, not asynchronous like the MVP docs
says. I think the code is correct, so we should remove this
line. If we do not remove this line, we should write a story
to make this call async.<br>
<br>
In the code, Publication deletes are not blocked by
association to distributions. Should write a story or remove
this line?<br>
</blockquote>
</div>
</blockquote>
<br>
I propose we leave the publication delete synchronous (non locking)
for now. The code should be setting the Distribution.publication =
NULL and this seems like a reasonable thing. If we make publication
DELETE asynchronous (with locking), we'd need to also make setting
the Distribution.publication asynchronous (with locking) to prevent
race conditions. This all seems more complicated than necessary.<br>
<br>
<blockquote type="cite"
cite="mid:CA+Kyt=pK4bFup2a_rWHj61xQ5cycqZ10Ma6nSqboHZ9er=iM=A@mail.gmail.com">
<div dir="ltr">
<blockquote
style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration-style:initial;text-decoration-color:initial;margin:0px
0px 0px 40px;border:none;padding:0px"><font face="arial,
helvetica, sans-serif"><span style="font-size:12.8px"><br>
</span></font>"single object only". What does that mean? If
it means 1 publication at a time, that is how all our objects
work, so I think we can delete this line.</blockquote>
</div>
</blockquote>
<br>
+0 delete that line.<br>
<br>
<blockquote type="cite"
cite="mid:CA+Kyt=pK4bFup2a_rWHj61xQ5cycqZ10Ma6nSqboHZ9er=iM=A@mail.gmail.com">
<div dir="ltr"><br>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Pulp-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Pulp-dev@redhat.com">Pulp-dev@redhat.com</a>
<a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/pulp-dev">https://www.redhat.com/mailman/listinfo/pulp-dev</a>
</pre>
</blockquote>
<br>
</body>
</html>