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

[Bug][SDK] Adjusted frame length exceeds occurred when reporting data through the HTTP protocol #7766

Closed
1 of 2 tasks
g0715158 opened this issue Apr 3, 2023 · 5 comments · Fixed by #8418
Closed
1 of 2 tasks
Assignees
Milestone

Comments

@g0715158
Copy link

g0715158 commented Apr 3, 2023

What happened

When using dataproxy-sdk to report data through the http protocol, the following screenshots report errors:

The scenario description is as follows (SDK only reports one piece of data per startup process):

  1. This exception will be reported for the first piece of data reported after the creation of the data stream

  2. If the data is not reported for a long time, the first piece of data will report this exception when the data is reported again

Guess the reason for this problem is due to the DataProxy's channel management policy configuration for netty

image

What you expected to happen

Normal reporting of messages

How to reproduce

repair this bug

Environment

version:1.6.0

InLong version

1.6.0

InLong Component

InLong DataProxy

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct

@g0715158 g0715158 added the type/bug Something is wrong label Apr 3, 2023
@dockerzhang
Copy link
Contributor

@gosonzhang PTAL, thanks.

@healchow healchow added this to the 1.7.0 milestone Apr 7, 2023
@healchow healchow changed the title [Bug]An error occurred when reporting data through the http protocol using dataproxy sdk [Bug][DataProxySDK] Adjusted frame length exceeds occurred when reporting data through the HTTP protocol Apr 7, 2023
@gosonzhang
Copy link
Contributor

Thanks @g0715158, got it!

@gosonzhang
Copy link
Contributor

Feedback on the current situation of this problem, I plan to optimize the implementation of the Source side [1] first, and use netty to handle HTTP data access. This issue is temporarily reserved, and it will be tested and verified after integration

  1. [Improve][DataProxy] Optimize the implementation logic of the Source #7950

@dockerzhang dockerzhang modified the milestones: 1.7.0, 1.8.0 May 11, 2023
@gosonzhang
Copy link
Contributor

Judging from this error message, there are several situations:

  1. There is an error in the protocol of the message report, such as TCP reporting to the HTTP port, or the packet format is wrong, resulting in an error in recognition;
  2. The format of the message report is correct, but an overlong packet is sent

I found new similar problems in my environment, and I feel that there are still problems, and I need to continue to locate them

@gosonzhang
Copy link
Contributor

This problem should be caused by [1] changes

When the InLong-SDK obtains the access information of the DataProxy reporting node from the Manager, it needs to carry the reporting protocol that the SDK needs to adopt. If the information is not carried, the Manager will return all the reporting ports supported by the DataProxy; in this case, the SDK will Reporting failure occurs because a reporting node with inconsistent protocols is selected.

I will fix this problem, set different reporting protocols according to the created Sender, and store the cached information separately

  1. [Improve][SDK] Add protocolType field in DataProxy config #6190

@healchow healchow changed the title [Bug][DataProxySDK] Adjusted frame length exceeds occurred when reporting data through the HTTP protocol [Bug][SDK] Adjusted frame length exceeds occurred when reporting data through the HTTP protocol Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment