search

DeployHubProject / DeployHub-Pro

DeployHub Pro Pipeline Status Project
https://www.openmakesoftware.com/application-release-automation-for-continuous-delivery/
Other
11 stars 4 forks source link

optimize /dmadminweb/ReportsData?type=CompList #352

Open piyush94 opened 2 years ago

piyush94 commented 2 years ago

The below request is being sent when accessing any Endpoint from the UI. And due to no filter for endpoint ID in the request, this request is getting timed out as there are a lot of components in the database.

Kindly have a look.

curl 'http://deployhub:6172/dmadminweb/ReportsData?type=CompList' \
  -H 'Connection: keep-alive' \
  -H 'Accept: application/json, text/javascript, */*; q=0.01' \
  -H 'User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.45 Safari/537.36' \
  -H 'X-Requested-With: XMLHttpRequest' \
  -H 'Referer: http://deployhub:6172/dmadminweb/Home' \
  -H 'Accept-Language: en-US,en;q=0.9' \
  -H 'Cookie: JSESSIONID=3B4E61BEB56483BD624B316CAF77D376; p2=; loggedin=Y; p1=; admin=N; p3=1637665460719; token=eyJhbGciOiJSUzI1NiJ9.eyJzdWIiOiIxIiwianRpIjoiMWE0OTQzMzItNWNlMy00MTBlLThlOTAtMzk0ZTdhZDZiMmY4In0.WbRxvbaC3Ro6y5GBYqGi3INb3F92EDSoyRkgsNJajdY3RJ8nj7W3nEWnVL78tT66ooCsnEN-N1DWzvWgfwFuuCCPBtEoOYBlhotVvPvWtTYxm3T0cmos4_G-ClseEDvhDCl-7rBDWwLRAvZdAKmtE1aW935Mf12YY28KPG0PkrNewPb6QHZlSg36uNB_uyS540guZUcATXvwkyc2_Acx9Tc7g4CzkWE4nETrW929SbIQV1UffV1sMbVPbHcPy5qSHojhL6W1P2eNoSw9_qjwFu988XB4radR6-C2S6x5FMFhnxP6qE3wxLubbF6eMwUsxYEkUNInC_jfbyYTB1DGmg' \
  --compressed \
  --insecure
piyush94 commented 2 years ago

This request is impacting Components page as well in the UI.

sbtaylor15 commented 2 years ago

@piyush94 there are three possible choices:

  1. We can make the default filter to return just the latest 5 of each component.
  2. Add an archive feature to "hide" components that are not used. So the default will bring back "non-archived" components.
  3. Enable pagination bringing a page at a time from the db.

Do you use the table filter option to filter based on name or domain?

piyush94 commented 2 years ago

@sbtaylor15

  1. Pagination option sounds good in case of the Components page. I am not using any table filter at the moment.
  2. One doubt, why all the components are being fetched when viewing an Endpoint ?
sbtaylor15 commented 2 years ago

@piyush94 Can you please send a screenshot of the Endpoint where all of the components are fetched?

piyush94 commented 2 years ago

@sbtaylor15 It's happening for all the endpoints.