Discussion: https://github.com/kubeflow/pipelines/pull/5311#pullrequestreview-613890507
It is hard to scale when there are many artifacts to show on Artifact list. We should implement something that API can accept pageSize and pageToken like listPipelines.
It might require update on backend API, or we can implement this during KFPv2 implementation, because it touches metadata store communication.
/kind bug
/area frontend
@Bobgy
Hello!
There are two old issues about this, so let's link to them:
Moreover, to fix this I suggest we rely on existing pagination support of the backend, thus making the the UI backwards compatible.
GetArtifacts, GetExecutions, and GetContexts APIs since 0.23.0 (release notes)GetExecutionsByContext andGetArtifactsByContext APIs since 0.25.1 (release notes)Therefore, it's just a matter of
In the end, there will be no breaking changes, no backend API changes, or different metadata store communication. What do you think?
Yes, the original plan was to support pagination when backend supports that. And the backend support has already been released as @elikatsis pointed out.
Suggest close either this issue or the existing issue, so that we do not have two threads for the same topic
Thank you @elikatsis and @Bobgy for the reference! It makes sense to change client since metadata has already supported pagination. I will close this issue here. Since we have linked this issue to the previous ones, we can find the reference for our decision on making client changes for supporting pagination.