ChristopherRabotin / bungiesearch

UNMAINTAINED CODE -- Elasticsearch-dsl-py django wrapper with mapping generator
BSD 3-Clause "New" or "Revised" License
67 stars 20 forks source link

allow clear index to have timeout override too #141

Closed diwu1989 closed 8 years ago

diwu1989 commented 8 years ago

clear index command can have the timeout override too, rebuild index command will inherit this option too because it uses the same options as clear index

ChristopherRabotin commented 8 years ago

What's the point of adding the flag to optparse without actually using that flag?

NullSoldier commented 8 years ago

It writes it into options.timeout, which is then passed down into all sub commands called and is used there.

ChristopherRabotin commented 8 years ago

Yes, you're right! Sorry, I had forgotten how the subcommand was called and didn't check (mostly because I was just waking up).