fix bugs

[azurewang]

fix receive large packet
fix _get_byte8
fix check large query

[agentzh]

Thank you for your patches!

It'll be even more appreciated if you can

1. add corresponding test cases to the existing test suite,
2. remove unrelated minor changes from the patch, and
3. use more descriptive commit log messages than "modified: /path/to/some/file".

Thanks!
-agentzh

[azurewang]

修改的git log的描述
实现了支持大于16Mb的query。
添加的large.t测试集
测试返回超大的insert_id
测试发送大于16Mb的query
测试读取返回的大于16Mb的行

[agentzh]

Perfect! Thank you for your contribution! I'll look into your patches soon :)

[agentzh]

Your test case "TEST 2: large query" in t/large.t is failing on my side. I'm getting the following error:

send() failed (32: Broken pipe)

Do you have any ideas? Is it passing on your side? Do you have any special configurations on the mysqld side?

[agentzh]

And I'm getting the same error while running your "TEST 3: large row" in t/large.t:

t/large.t .. 1/3 
#   Failed test 'TEST 3: large row - response_body - response is expected'
#   at /home/agentzh/git/lua-resty-mysql/../test-nginx/lib/Test/Nginx/Socket.pm line 955.
#          got: 'insert data error:failed to send query: broken pipe
# '
#     expected: '1
# '

#   Failed test 'TEST 3: large row - pattern "[error]" should not match any line in error.log but matches line "2013/03/27 18:11:35 [error] 25278\#0: *1 send() failed (32: Broken pipe), client: 127.0.0.1, server: localhost, request: \"GET /t HTTP/1.1\", host: \"localhost\""'
#   at /home/agentzh/git/lua-resty-mysql/../test-nginx/lib/Test/Nginx/Socket.pm line 878.

[azurewang]

应该是mysqld端设置的最大包大小不够大。我检查我的环境，有可能在测试脚本里加一下设置环境变量。
在 2013-3-28 上午9:12，"Yichun Zhang" notifications@github.com写道：

And I'm getting the same error while running your "TEST 3: large row" in
t/large.t:

t/large.t .. 1/3

Failed test 'TEST 3: large row - response_body - response is expected'

at /home/agentzh/git/lua-resty-mysql/../test-nginx/lib/Test/Nginx/Socket.pm line 955.

got: 'insert data error:failed to send query: broken pipe

'

expected: '1

'

Failed test 'TEST 3: large row - pattern "[error]" should not match any line in error.log but matches line "2013/03/27 18:11:35 [error] 25278#0: *1 send() failed (32: Broken pipe), client: 127.0.0.1, server: localhost, request: "GET /t HTTP/1.1", host: "localhost""'

at /home/agentzh/git/lua-resty-mysql/../test-nginx/lib/Test/Nginx/Socket.pm line 878.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/6#issuecomment-15563098
.

[azurewang]

你确定打过补丁在跑的测试？

[azurewang]

我这边在mysqld设置max-allowed-packet=200m
使用我打过补丁的代码三个测试都正常通过。
在 2013-3-28 上午10:21，"azure wang" azure1st@gmail.com写道：

你确定打过补丁在跑的测试？

[azurewang]

我的mysqld是debian6.0上apt-get安装的mysql-server。配置文件只增加了上面说的那一项。

在 2013-3-28 上午11:26，"azure wang" azure1st@gmail.com写道：

我这边在mysqld设置max-allowed-packet=200m
使用我打过补丁的代码三个测试都正常通过。

在 2013-3-28 上午10:21，"azure wang" azure1st@gmail.com写道：

你确定打过补丁在跑的测试？

[azurewang]

你可以把你的mysqld信息告诉我一下，我按照你的mysqld环境在测试一下。
在 2013-3-28 上午11:40，"azure wang" azure1st@gmail.com写道：

我的mysqld是debian6.0上apt-get安装的mysql-server。配置文件只增加了上面说的那一项。

在 2013-3-28 上午11:26，"azure wang" azure1st@gmail.com写道：

我这边在mysqld设置max-allowed-packet=200m
使用我打过补丁的代码三个测试都正常通过。

在 2013-3-28 上午10:21，"azure wang" azure1st@gmail.com写道：

你确定打过补丁在跑的测试？

[azurewang]

测试三，我这没打补丁时，得到的行数是错误的，应该是一行，得到了两行。行里的数据也是错误的。没有出现管道破裂的情况啊。很奇怪。
在 2013-3-28 上午11:42，"azure wang" azure1st@gmail.com写道：

你可以把你的mysqld信息告诉我一下，我按照你的mysqld环境在测试一下。
在 2013-3-28 上午11:40，"azure wang" azure1st@gmail.com写道：

我的mysqld是debian6.0上apt-get安装的mysql-server。配置文件只增加了上面说的那一项。

在 2013-3-28 上午11:26，"azure wang" azure1st@gmail.com写道：

我这边在mysqld设置max-allowed-packet=200m
使用我打过补丁的代码三个测试都正常通过。

在 2013-3-28 上午10:21，"azure wang" azure1st@gmail.com写道：

你确定打过补丁在跑的测试？

[agentzh]

Hello!

2013/3/27 AzureWang notifications@github.com

我这边在mysqld设置max-allowed-packet=200m
使用我打过补丁的代码三个测试都正常通过。

What happens if you use the default mysqld setup?

I think the tests should pass with the default mysqld configuration.

-agentzh

[agentzh]

Hello!

On Wed, Mar 27, 2013 at 7:21 PM, AzureWang notifications@github.com wrote:

你确定打过补丁在跑的测试？

Yes, of course. I was on your fork's master.

Regards,
-agentzh

[agentzh]

Hello!

2013/3/27 AzureWang notifications@github.com

你可以把你的mysqld信息告诉我一下，我按照你的mysqld环境在测试一下。

I'm using Fedora 17's mysqld (everything updated from the official yum),
with the default configuration.

-agentzh

[azurewang]

我修改了测试的代码，你出现的错误是由于使用了mysqld5.5导致的，我写测试脚本的时候使用的是5.1，mysqld5.5如果发送的SQL超过max-allowed-packet(1M)就出现broken pipe, mysqld5.1会明确告诉你Got a packet bigger than 'max_allowed_packet' bytes.
mysqld5.5错误处理倒退了。现在Test3我这边使用mysqld5.5可以通过了。Test2由于是测试超过16M的SQL，不修改mysqld的max-allowed-packet 就无法通过测试。不知道在写测试脚本的时候能不能通过获取服务器的环境变量来判断是否运行这个测试。
mysqld5.5默认max-allowed-packet =1M 这个值有点小啊， 呵呵。反正是不能满足我们的业务。超过16M的情况虽然非常小，但是不能排除有这个可能性。所以我选择了支持大于16M的SQL。

[agentzh]

Okay, I get it now :)