<div dir="ltr">I’ve opened the following issues:<div><br></div><div><a href="https://pulp.plan.io/issues/3790">https://pulp.plan.io/issues/3790</a><br></div><div><a href="https://pulp.plan.io/issues/3801">https://pulp.plan.io/issues/3801</a> - requires breaking changes</div><div><br></div><div>Feel free to leave feedback on the issues.<br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 26, 2018 at 3:58 PM Jeff Ortel <<a href="mailto:jortel@redhat.com">jortel@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
<br>
<br>
<div class="m_4559170014005015576moz-cite-prefix">On 06/26/2018 08:48 AM, Dennis Kliban
wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">The user should be able to specify a page size at
request time. The user should also be able to specify which page
they are requesting. <br>
</div>
</blockquote>
<br>
+ 1<br>
<blockquote type="cite">
<div class="gmail_extra"><br>
<div class="gmail_quote">On Tue, Jun 26, 2018 at 9:31 AM, David
Davis <span dir="ltr"><<a href="mailto:daviddavis@redhat.com" target="_blank">daviddavis@redhat.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">I was looking at the pagination code this
morning and there were two things I wanted to discuss.
<div><br>
</div>
<div>First, there’s no way to override the number of
results per request. Instead, page size has to be
configured for the whole app. Allowing users to override
page size is trivial[0] so I wonder if we should enable
it.</div>
<div><br>
</div>
<div>The second topic is a bit more complex. We currently
use cursor based pagination where pages must be fetched
sequentially as opposed to the default DRF pagination
method of using page numbers. Cursors work great for
large data sets as you don’t have to figure out things
like the number of pages.</div>
<div><br>
</div>
<div>The first problem is that in Pulp we parallelize web
requests for things like fetching metadata. See our
Ansible plugin as an example[1]. If we want to support
things like syncing content from one Pulp server to
another, we probably have to use offset/page-based
pagination for certain endpoints. </div>
<div><br>
</div>
<div>Another consideration is Katello. In Katello’s UI
they show the number of pages and allow users to jump to
arbitrary pages or the last page. If we want Katello to
stop indexing Pulp data and instead query Pulp directly,
we’ll need to allow them to use page numbers somehow.</div>
<div><br>
</div>
<div>Thoughts?</div>
<div><br>
</div>
<div>[0] <a href="https://gist.github.com/daviddavis/56a0b86629cd675d57aac61583c01944" target="_blank">https://gist.github.com/daviddavis/56a0b86629cd675d57aac61583c01944</a><br clear="all">
<div>
<div dir="ltr" class="m_4559170014005015576m_2199165952748460416gmail_signature">
<div dir="ltr">
<div>
<div dir="ltr">
<div>
<div dir="ltr">
<div>[1] <a href="https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/tasks/synchronizing.py#L147-L171" target="_blank">https://github.com/pulp/pulp_ansible/blob/master/pulp_ansible/app/tasks/synchronizing.py#L147-L171</a></div>
<span class="m_4559170014005015576HOEnZb"><font color="#888888">
<div><br>
</div>
<div>David<br>
</div>
</font></span></div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
<br>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
<br>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset class="m_4559170014005015576mimeAttachmentHeader"></fieldset>
<br>
<pre>_______________________________________________
Pulp-dev mailing list
<a class="m_4559170014005015576moz-txt-link-abbreviated" href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a>
<a class="m_4559170014005015576moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/pulp-dev" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a>
</pre>
</blockquote>
<br>
</div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>