Gian Merlino

Results 161 comments of Gian Merlino

Just raised #13206, which makes queries like this succeed instead of throwing an error (what we get with only #12944) or hanging (what we get before #12944).

I'm very interested in improving our error reporting, so I thank you for writing this up. I wanted to note that there is a similar concept to the proposed DruidException...

Hmm. I'm not sure if the dummy-dim-dropping code is the problem. It's only dropping dummy dims: ones that don't appear at all in the output. But `countryName` _should_ appear, because...

There are two better solutions that I didn't do in this patch due to them being more complex: 1) Improving the query execution logic so the `granularity` feature doesn't incur...

Thanks for the review @rohangarg. I pushed up some changes based on your comments.

> Can we go further? Any reason to create buckets that will be empty? If we're doing time-based grouping, and data is time ordered, we can create buckets on the...

We haven't had a chance to write a unit test demonstrating this problem yet. If you have one you could share, that would certainly accelerate the process, and would be...

Thanks for looking into it @somu-imply!

#12990 updated the docs to say that Java 17 is experimental. As of now I am not aware of anything that is broken on Java 17. I use it on...

Recently reported issue: https://github.com/apache/druid/issues/13277