[GitHub] [incubator-seatunnel] liugddx commented on a diff in pull request #4372: [Feature][Connector] http connector support pagable

["liugddx (via GitHub)" <gi...@apache.org>]

liugddx commented on code in PR #4372:
URL: https://github.com/apache/incubator-seatunnel/pull/4372#discussion_r1149958674


##########
docs/en/connector-v2/source/Http.md:
##########
@@ -289,6 +293,32 @@ Http {
 }
 ```
 
+### page options
+
+## Example
+
+simple:
+
+```hocon
+Http {
+  url = "https://tyrantlucifer.com/api/getDemoData"
+  pageing = {
+    pageNoField = "pageNo",
+    pageNo = "1-2,4,8-10",

Review Comment:
   > The disadvantage of only providing the page range is that it cannot self-increase to traverse all data sets, and users need to clearly know the number of pages they want to synchronize. But the advantage is that there is no requirement for the structure of the user response body.
   
   You can get the specified key through `jsonpath`, such as ``` jsonPath.get("*totalPage")``` . `totalPage` can be used as a parameter
   
   



-- 
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: commits-unsubscribe@seatunnel.apache.org

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