[GitHub] [trafficcontrol] mitchell852 removed a comment on pull request #5489: Non-"CRUD-er" DSRs

[GitBox <gi...@apache.org>]

mitchell852 removed a comment on pull request #5489:
URL: https://github.com/apache/trafficcontrol/pull/5489#issuecomment-796274391


   > I was able to get `orderby` and `sortOrder` to work. I even tried specifically ordering by `createdAt`. Worked fine.
   
   yeah i can't get the `sortOrder` to stick when i do GET /api/4.0/deliveryservice_requests?orderby=createdAt&sortOrder=desc&limit=10 in your branch, i get this query:
   
   ```
   SELECT
   	a.username AS author,
   	e.username AS lastEditedBy,
   	s.username AS assignee,
   	r.assignee_id,
   	r.author_id,
   	r.change_type,
   	r.created_at,
   	r.id,
   	r.last_edited_by_id,
   	r.last_updated,
   	r.deliveryservice,
   	r.status
   FROM deliveryservice_request r
   JOIN tm_user a ON r.author_id = a.id
   LEFT OUTER JOIN tm_user s ON r.assignee_id = s.id
   LEFT OUTER JOIN tm_user e ON r.last_edited_by_id = e.id
   WHERE (
   	CAST(r.deliveryservice->>'tenantId' AS BIGINT) = ANY(CAST(:accessibleTenants AS BIGINT[]))
   )
   ORDER BY r.created_at
   LIMIT 10
   ```
   
   ^^ see? no DESC attached to order by in the query but in master i get it.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org