a panic about hdfs HOT 4 CLOSED

Hi @gyustorm, sorry for the delay responding.

I'm not sure I completely understand your bug report - can you make a PR with a test case that panics?

from hdfs.

Hi @colinmarc Your hdfs.FileReader has only one blockReader, the FileReader implements os.File.ReadAt, so if i launch goroutines to read different ranges of the same file from hadoop for speeding up, then it will come to thread unsafe problem.

type FileReader struct {
    client *Client
    name   string
    info   os.FileInfo

    blocks      []*hdfs.LocatedBlockProto
    blockReader *rpc.BlockReader
    offset      int64

    readdirLast string

    closed bool
}

from hdfs.

Ah, I understand. I don't make any guarantees about the thread safety of FileReaders, but maybe I should. Does it work if you try to have multiple FileReaders instead of just accessing the same one concurrently?

from hdfs.

Hmm, maybe it works. Thanks for your time.

from hdfs.