Misha

Results 19 comments of Misha

> Thanks a lot for the contribution 🎉 Curious if the change in the PR solves the bug. Also, is the result correct (i.e. there are 500 rows ingested after...

> That shouldn't happen - output of a query like the following in durable storage should produce multiple partitions (which it isn't doing so at the moment) > > ```sql...

> Can you please verify if that patch fixes the issue? Hi @LakshSingla, thanks for the suggestion, it actually does fix the problem, although WDYT about the scan stage which...

> Thanks for the changes @nozjkoitop. I'll review them shortly. @LakshSingla Could I remind you to have a look on that please?

> Can you please add a test in the `MSQSelectTest` or `MSQInsertTest` to validate that the PR fails before with the exception you are seeing and passes after this patch?...

Hey @LakshSingla, just a kindly reminder about this PR :)

> @nozjkoitop I think @LakshSingla's comment was to only make changes that affect MSQ, since the bug is seen only there. I think your changes might affect other queries as...

Hi @LakshSingla I've checked the latest druid and described issue is still there, although now there is a NPE on the ui, but provided patch still resolves an issue, so...

Issue is still there in v32, merged master to solve conflicts