if use too much count option, use range will cause use too much memory.

oliver006 / elasticsearch-test-data

Generate and upload test data to Elasticsearch for performance and load testing

MIT License

257 stars 124 forks source link

Closed konghui closed 5 years ago

konghui commented 5 years ago

Range will cause use too much memory even cause kill by oomkiller, use xrange instead

Outpox commented 5 years ago

Hey there, Just to let you know, this change breaks the compatibility with python3. I've had to revert it for my code to work.

oliver006 commented 5 years ago

My apologies, i was not aware of the subtle differences between the two and didn't know that Python3 does no longer support xrange.

I think it's ok to expect people to use Python3 at this point (py2 EOL is in about 5 minutes or so) so I'll revert this.

konghui commented 5 years ago

I am sorry. I will add the compatible code for python3.

Outpox commented 5 years ago

It's alright guys, thanks for answering so quickly! :)