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

Allowed memory size on tag 1.5.0 #132

Open
babenko-alex-pdffiller opened this issue Oct 9, 2023 · 3 comments
Open

Allowed memory size on tag 1.5.0 #132

babenko-alex-pdffiller opened this issue Oct 9, 2023 · 3 comments
Labels

Comments

@babenko-alex-pdffiller
Copy link

After the update to 1.5.0, they started getting an error Allowed memory size of 536870912 bytes exhausted (tried to allocate 266338312 bytes)
we get the error along with "Warning: A non-numeric value encountered","file":"\/app\/vendor\/packaged\/thrift\/src\/Protocol\/TCompactProtocol.php:107
"Deprecated: Implicit conversion from float-string "257931e25eba638" to int loses precision", "file":"/app/vendor/packaged/thrift/src/Protocol/TCompactProtocol.php:107"

OS: GNU/Linux, PHP: 8.2

@sergeyklay
Copy link
Collaborator

Hi @paillechat,

I suspect that the changes made in TextCodec.php as part of pull request #130, particularly around the handling of 128-bit trace IDs, might be related to this problem. Would you have any insights into this? Additionally, if you have the bandwidth, would you be able to take a look and possibly address this issue? Your expertise would be greatly appreciated.

@paillechat
Copy link

Hi @babenko-alex-pdffiller,
Please provide a more detailed stack trace, it will help identify the problem, thanks!

@babenko-alex-pdffiller
Copy link
Author

babenko-alex-pdffiller commented Oct 10, 2023

Hi @paillechat,
Full trace of one request:
"message":"Error: Allowed memory size of 536870912 bytes exhausted (tried to allocate 266338312 bytes)","class":"Symfony\\Component\\ErrorHandler\\Error\\OutOfMemoryError","code":0,"file":"\/app\/vendor\/packaged\/thrift\/src\/Protocol\/TCompactProtocol.php:123"

"message":"Warning: A non-numeric value encountered","class":"ErrorException","code":0,"file":"\/app\/vendor\/packaged\/thrift\/src\/Protocol\/TCompactProtocol.php:107","line":0,"trace":"\/app\/vendor\/packaged\/thrift\/src\/Protocol\/TCompactProtocol.php:736; \/app\/vendor\/packaged\/thrift\/src\/Base\/TBase.php:357; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Thrift\/Agent\/Zipkin\/Span.php:186; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Sender\/UdpSender.php:280; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Sender\/UdpSender.php:248; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Sender\/UdpSender.php:114; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Sender\/UdpSender.php:93; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Reporter\/RemoteReporter.php:43; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Tracer.php:290;

"message":"Deprecated: Implicit conversion from float-string \"257931e25eba638\" to int loses precision","class":"ErrorException","code":0,"file":"\/app\/vendor\/packaged\/thrift\/src\/Protocol\/TCompactProtocol.php:107","line":0,"trace":"\/app\/vendor\/packaged\/thrift\/src\/Protocol\/TCompactProtocol.php:736; \/app\/vendor\/packaged\/thrift\/src\/Base\/TBase.php:357; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Thrift\/Agent\/Zipkin\/Span.php:186; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Sender\/UdpSender.php:280; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Sender\/UdpSender.php:248; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Sender\/UdpSender.php:114; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Sender\/UdpSender.php:93; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Reporter\/RemoteReporter.php:43; \/app\/vendor\/jonahgeorge\/jaeger-client-php\/src\/Jaeger\/Tracer.php:290;

@sergeyklay sergeyklay added the bug label Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants