zhangshenghang commented on code in PR #9869:
URL: https://github.com/apache/seatunnel/pull/9869#discussion_r2355766881


##########
docs/en/connector-v2/source/Redis.md:
##########
@@ -252,21 +253,44 @@ connector will generate data as the following:
 | ---- | ----------- | ------- |
 | 200  | get success | true    |
 
-when you assign format is `text`, connector will do nothing for upstream data, 
for example:
+when you assign format is `text`, you can choose to specify the schema 
information or not. 
 
-upstream data is the following:
-
-```json
-{"code":  200, "data":  "get success", "success":  true}
+For example, upstream data is the following:
 
+```text
+200#get success#true
 ```
 
+If you do not assign data schema connector will treat the upstream data as the 
following:
+
+| content                                                  |
+| -------------------------------------------------------- |
+| 200#get success#true |
+
+If you assign data schema, you should also assign the option `schema` and 
`field_delimiter` as following:
+
+```hocon
+field_delimiter = "#"
+schema {
+    fields {
+        code = int
+        data = string
+        success = boolean
+    }
+}
+
+```

Review Comment:
   I have a question. Text format has been applied to Redis here. However, it 
is still not supported in HTTP. Do we need to modify the HTTP connector 
separately in the future?
   
   
https://github.com/apache/seatunnel/blob/02c7eb3177989bcd50ba6c1059862c1586d3fa39/docs/en/connector-v2/source/Http.md?plain=1#L176C1-L177C1



-- 
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]

Reply via email to