<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>