newClient should panic when failed to connect to a db

[cruvie]

Issue tracker is used for reporting bugs and discussing new features. Please use
stackoverflow for supporting issues.

newClient should panic when failed to connect to a db

Expected Behavior

panic when failed to connect to a db

Current Behavior

nothing happened

Possible Solution

panic when failed to connect to a db

Steps to Reproduce

	client := redis.NewClient(&redis.Options{
		Addr:     "127.0.0.1:7923",
		Password: "123456",
		DB:       0,
	})
	if client == nil {
		t.Error("client is nil")
	}

Context (Environment)

it leaves init error to runtime error

newClient should panic when failed to connect to a db

Detailed Description

Possible Implementation

[LINKIWI]

Panicking on initialization due to a single connection error would be extremely weird behavior. go-redis establishes connections lazily on-demand when commands are issued, and any connection errors are correctly surfaced there. If you want to ensure that a connection is valid after creating a client, consider issuing a Ping immediately, and panicking at your application layer if that fails.

[Kamalesh-Seervi]

can I work on this