Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[INLONG-895][Doc] Improve HTTP report documentation #896

Merged
merged 2 commits into from
Nov 30, 2023
Merged

[INLONG-895][Doc] Improve HTTP report documentation #896

merged 2 commits into from
Nov 30, 2023

Conversation

gosonzhang
Copy link
Contributor

Added an introduction document on how HTTP reporting is completed in InLong

@@ -3,9 +3,28 @@ title: HTTP Report
sidebar_position: 3
---

## HTTP report process introduction
InLong processes HTTP report through the DataProxy node. The data source regularly obtains the access point list from the Manager, then selects available HTTP DataProxy nodes from the access point list according to its own policies, and then uses the HTTP protocol for data production; due to the performance of HTTP report due to problems such as low proportion of valid data and easy loss of request messages, it is recommended that users use TCP to report data as much as possible. The overall HTTP report process is shown below:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

due to the performance of HTTP report due to problems such as low proportion of valid data and easy loss of request messages

Suggestion :
due to the issues of low performance, low percentage of effective data, and easy loss of request messages in HTTP reporting.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@gosonzhang gosonzhang merged commit c6079e5 into apache:master Nov 30, 2023
1 check passed
@gosonzhang gosonzhang deleted the INLONG-895 branch November 30, 2023 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improve][Doc] Improve HTTP report documentation
4 participants