narrowizard commented on code in PR #8547:
URL:
https://github.com/apache/incubator-devlake/pull/8547#discussion_r2297073016
##########
backend/plugins/jira/tasks/epic_collector.go:
##########
@@ -98,6 +117,10 @@ func CollectEpics(taskCtx plugin.SubTaskContext)
errors.Error {
query.Set("startAt", fmt.Sprintf("%v",
reqData.Pager.Skip))
Review Comment:
Based on the information provided in the Atlassian developer changelog, the
new `api/3/search/jql` endpoint has changed the way data is paginated.
Specifically, random page access has been replaced with a continuation token
API. This means that you won't be able to fetch multiple pages simultaneously
using parallel threads. The `startAt` parameter has been replaced with
`nextPageToken`.
Given these changes, simply updating the API endpoint without adjusting the
pagination logic will result in incomplete data retrieval. The existing code
that relies on the `startAt` parameter for pagination will no longer work as
expected. It's crucial to implement the new nextPageToken mechanism to handle
pagination properly and ensure that all data can be fetched correctly.
> We’ll replace random page access with a continuation token API. This means
you won’t be able to get multiple pages at the same time with parallel threads.
startAt parameter will be replaced with nextPageToken.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
