"not initialized" error on any redis operation

uppal.samar · 2014-09-08T19:04:15+00:00

Hello! On Sun, Sep 7, 2014 at 11:40 PM, rvsw wrote: > Thanks for providing the replace_filter module in nginx. It has been very > useful to me. >...

"not initialized" error on any redis operation

uppal.samar

Hello,

The below code seems to be connecting to redis just fine (no error output). But on trying any redis (get or set etc) operation I keep getting "not initialized" message. Strangely, the same code was working fine earlier, so I suspect there is something else going on here. Thanks for your help.

location / {

default_type "text/plain";

content_by_lua '

local redis = require "resty.redis"

local red = redis:new()

red:set_timeout(1000)

local ok, err = red:connect("127.0.0.1", 6379)

if not ok then

ngx.say("failed to connect: ", err)

return

end

ngx.req.read_body()

local body_data = ngx.req.get_body_data()

ok, err = red.lpush("master", body_data)

if not ok then

ngx.say("failed to push to list", err);

end

manchev

Hello!

On Mon, Sep 8, 2014 at 2:04 PM, Samarjit Uppal <uppal...@gmail.com> wrote:

Hello,

The below code seems to be connecting to redis just fine (no error output). But on trying any redis (get or set etc) operation I keep getting "not initialized" message. Strangely, the same code was working fine earlier, so I suspect there is something else going on here. Thanks for your help.

location / {
default_type "text/plain";
content_by_lua '
local redis = require "resty.redis"
local red = redis:new()

red:set_timeout(1000)

local ok, err = red:connect("127.0.0.1", 6379)
if not ok then
ngx.say("failed to connect: ", err)
return
end

ngx.req.read_body()
local body_data = ngx.req.get_body_data()
ok, err = red.lpush("master", body_data)

This should definitely read red:lpush.

if not ok then
ngx.say("failed to push to list", err);
end

.

Best,
Vladislav

uppal.samar

Yes, seems like it should. But I get the error "not initialized". Weird, because red:connect("127.0.0.1", 6379) does not throw any error. Not sure whats going on.

On Monday, September 8, 2014 4:48:39 PM UTC+5:30, Vladislav Manchev wrote:

Hello!

On Mon, Sep 8, 2014 at 2:04 PM, Samarjit Uppal <uppal...@gmail.com> wrote:

Hello,

The below code seems to be connecting to redis just fine (no error output). But on trying any redis (get or set etc) operation I keep getting "not initialized" message. Strangely, the same code was working fine earlier, so I suspect there is something else going on here. Thanks for your help.

location / {
default_type "text/plain";
content_by_lua '
local redis = require "resty.redis"
local red = redis:new()

red:set_timeout(1000)

local ok, err = red:connect("127.0.0.1", 6379)
if not ok then
ngx.say("failed to connect: ", err)
return
end

ngx.req.read_body()
local body_data = ngx.req.get_body_data()
ok, err = red.lpush("master", body_data)

This should definitely read red:lpush.

if not ok then
ngx.say("failed to push to list", err);
end

.

Best,
Vladislav

uppal.samar

Ok, so I modified the redis.lua file and found that it was failing in the _do_cmd func. Seems like the _do_cmd gets called whenever we try to send a redis command. So no sock was available? does that mean a tcp connection failure?

local function _do_cmd(self, ...)

local args = {...}

local sock = self.sock

if not sock then

return nil, "not initialized in _do_cmd"

end

On Monday, September 8, 2014 4:48:39 PM UTC+5:30, Vladislav Manchev wrote:

Hello!

On Mon, Sep 8, 2014 at 2:04 PM, Samarjit Uppal <uppal...@gmail.com> wrote:

Hello,

The below code seems to be connecting to redis just fine (no error output). But on trying any redis (get or set etc) operation I keep getting "not initialized" message. Strangely, the same code was working fine earlier, so I suspect there is something else going on here. Thanks for your help.

location / {
default_type "text/plain";
content_by_lua '
local redis = require "resty.redis"
local red = redis:new()

red:set_timeout(1000)

local ok, err = red:connect("127.0.0.1", 6379)
if not ok then
ngx.say("failed to connect: ", err)
return
end

ngx.req.read_body()
local body_data = ngx.req.get_body_data()
ok, err = red.lpush("master", body_data)

This should definitely read red:lpush.

if not ok then
ngx.say("failed to push to list", err);
end

.

Best,
Vladislav

luis.gasca

double check that you are calling red:lpush(), not red.lpush(). Note the dot vs the colon

On 09/08/2014 02:05 PM, Samarjit Uppal wrote:

Ok, so I modified the redis.lua file and found that it was failing in the _do_cmd func. Seems like the _do_cmd gets called whenever we try to send a redis command. So no sock was available? does that mean a tcp connection failure?

local function _do_cmd(self, ...)

local args = {...}

local sock = self.sock

if not sock then

return nil, "not initialized in _do_cmd"

end

On Monday, September 8, 2014 4:48:39 PM UTC+5:30, Vladislav Manchev wrote:

Hello!

On Mon, Sep 8, 2014 at 2:04 PM, Samarjit Uppal <uppal...@gmail.com> wrote:

Hello,

The below code seems to be connecting to redis just fine (no error output). But on trying any redis (get or set etc) operation I keep getting "not initialized" message. Strangely, the same code was working fine earlier, so I suspect there is something else going on here. Thanks for your help.

location / {

default_type "text/plain";

content_by_lua '

local redis = require "resty.redis"

local red = redis:new()

red:set_timeout(1000)

local ok, err = red:connect("127.0.0.1", 6379)

if not ok then

ngx.say("failed to connect: ", err)

return

end

ngx.req.read_body()

local body_data = ngx.req.get_body_data()

ok, err = red.lpush("master", body_data)

This should definitely read red:lpush.

if not ok then

ngx.say("failed to push to list", err);

end

.

uppal.samar

Argghhh!!!! That was it, had to use a colon. Thanks a lot :)

On Monday, September 8, 2014 5:46:50 PM UTC+5:30, Luis Gasca wrote:

double check that you are calling red:lpush(), not red.lpush(). Note the dot vs the colon

On 09/08/2014 02:05 PM, Samarjit Uppal wrote:

Ok, so I modified the redis.lua file and found that it was failing in the _do_cmd func. Seems like the _do_cmd gets called whenever we try to send a redis command. So no sock was available? does that mean a tcp connection failure?

local function _do_cmd(self, ...)

local args = {...}

local sock = self.sock

if not sock then

return nil, "not initialized in _do_cmd"

end

On Monday, September 8, 2014 4:48:39 PM UTC+5:30, Vladislav Manchev wrote:

Hello!

On Mon, Sep 8, 2014 at 2:04 PM, Samarjit Uppal <uppal...@gmail.com> wrote:

Hello,

The below code seems to be connecting to redis just fine (no error output). But on trying any redis (get or set etc) operation I keep getting "not initialized" message. Strangely, the same code was working fine earlier, so I suspect there is something else going on here. Thanks for your help.

location / {

default_type "text/plain";

content_by_lua '

local redis = require "resty.redis"

local red = redis:new()

red:set_timeout(1000)

local ok, err = red:connect("127.0.0.1", 6379)

if not ok then

ngx.say("failed to connect: ", err)

return

end

ngx.req.read_body()

local body_data = ngx.req.get_body_data()

ok, err = red.lpush("master", body_data)

This should definitely read red:lpush.

if not ok then

ngx.say("failed to push to list", err);

end

.