Don't rely on index from future

rinsed-org / rb-snowflake-client

pure ruby snowflake (REST API) client

MIT License

9 stars 11 forks source link

Don't rely on index from future #71

Closed nampas closed 3 weeks ago

nampas commented 4 weeks ago

We're seeing this occasionally

TypeError no implicit conversion from nil to integer
[]=(/usr/local/bundle/gems/rb_snowflake_client-1.1.3/lib/ruby_snowflake/result.rb:19)
...
/usr/local/bundle/gems/rb_snowflake_client-1.1.3/lib/ruby_snowflake/client/threaded_in_memory_strategy.rb:35
...

I can't really see how we'd ever have a nil index. But maybe having the index not rely on the result of the future execution would be safer.

reidnimz commented 4 weeks ago

Yeah, looks right. Seems like it shouldn't be necessary - but can't hurt