func (cache *Cache) startExpirationProcessing() - Never Exits and Leaves residual Goroutines. about ttlcache

[ReneKroon]

@zkynet Can you test this code? Its merged to master but not an official release yet.

[zveinn]

Completely forgot to report back!

It seems that did it. Thanks man!

[ReneKroon]

Good call, i'll fix it in the upcoming 2 weeks i think.

I guess adding a Close method would work?

[zveinn]

Yep, that should do it!

However, maybe "close" is not the right description. Somehow close feels like a non-permanent move. Maybe it's just me ?

Since you already have:
NewChache()

Maybe one of these two:
DeleteCache()
RemoveCache()

[zveinn]

Actually, now that I think about it. Do you feel like getting some help on the package ?

I would not mind implementing a fix for you. I can make some adjustments and then deliver them in PR's.

[ReneKroon]

A pull request is always welcome. i think 'RemoveCache' is a good name but i also like to check if we can use some common standard interface. For example, Closer reads:

type Closer
Closer is the interface that wraps the basic Close method.

The behavior of Close after the first call is undefined. Specific implementations may document their own behavior.

type Closer interface {
    Close() error
}

But in this case i'm happy with either solution.

[zveinn]

Yep, that makes perfect sense. I will take a look this week and see if I can't get this done :)

[zveinn]

Sweet, I was about to get my hand dirty tonight, but since you already did I'll do a testing session in stead. It'll happen in about 6-7 hours.

Good job.