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

sql过长会被截取 , 函数名 过长也会进行截取 源数据那边做了截取处理,可否不要截取 #14

Closed
12gameRensheng opened this issue Jul 30, 2021 · 12 comments

Comments

@12gameRensheng
Copy link
Contributor

如图所示 : sql过长会被截取 , 函数名 过长也会进行截取 源数据那边做了截取处理,可否不要截取
image

@longxinH
Copy link
Owner

在xhprof已经做过不限制字符长度的尝试,当出现很长的字符串的时候,会出现内存溢出,导致段错误。commit

@longxinH
Copy link
Owner

目前是限制512,可以适当的加大限制范围

@12gameRensheng
Copy link
Contributor Author

大佬你好 因为我不懂c代码 可以弄成一个配置吗 默认就是512 读取apm.ini 或者 php.ini里面的配置进行

@longxinH
Copy link
Owner

longxinH commented Aug 1, 2021

提议不错,我这边修改一下

@12gameRensheng
Copy link
Contributor Author

大佬 你修改代码时 能不能顺便把我提的 那个 每个函数里面也加一个作者 如果有读取处理
格式就统一按照php的注释规范
格式如下:* @author liangdabiao 如下图所示
#13

@12gameRensheng
Copy link
Contributor Author

管理ui界面 我愿意花时间完善下 作者的参数 和进行PR

@longxinH
Copy link
Owner

longxinH commented Aug 3, 2021

我看了代码,mysql没有截取的,看一下是不是php输出的时候截取了。

@12gameRensheng
Copy link
Contributor Author

12gameRensheng commented Aug 4, 2021

大佬 我目前测试当函数名超过502 扩展里面会自动截取处理。
例子如下:

  1. 创建一个测试文件
    image
  2. 执行代码
    image
  3. export.php 处理导入代码中 做下处理 将扩展里面传递的变量 $_apm_export 存储到本地的./年_月_日_时_分_秒.json 文件
    image
  4. 打开json文件 查看 函数t1的结尾数字对上了 但是函数z1的结尾数字对不上
    image
    image

longxinH added a commit that referenced this issue Aug 4, 2021
支持函数长度缓冲区设置 #14
@longxinH
Copy link
Owner

longxinH commented Aug 4, 2021

最新版本支持设置长度,apm.scratch_buf = 1024

@12gameRensheng
Copy link
Contributor Author

感谢 大佬 我现在试试

@12gameRensheng
Copy link
Contributor Author

感谢大佬 已经可以了
image

@longxinH
Copy link
Owner

longxinH commented Aug 4, 2021

apm.scratch_buf 不建议设置过大

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

No branches or pull requests

2 participants