Clint Wylie

Results 50 comments of Clint Wylie

It looks like if we change this to call `parser.parseStmtList()` instead of `parser.parseStmt()` we can parse lists of statements correctly, and then check for root being `SqlNodeList` in `DruidPlanner` ~here...

>It looks like if we change this to call parser.parseStmtList() instead of parser.parseStmt() we can parse lists of statements correctly, and then check for root being SqlNodeList in DruidPlanner ~here...

>šŸ‘ Might be good to add a link to the original SIEVE paper: https://www.usenix.org/conference/nsdi24/presentation/zhang-yazhuo for those interested yea totally, I have it linked in javadocs for `StorageLocation` and was planning...

> Initially our thinking was to focus on partial fetches from S3, so we only have to fetch the columns actually needed for a query. From my PoV (@clintropolis may...

>@clintropolis - how do segment metadata queries work that touch lots of segments and are system-generated? We wouldn't want to fetch all the segments to build sql schema on broker....

>Curious how is SegmentDescriptor used in AcquireSegmentAction? I thought it's enough to just use DataSegment? `SegmentDescriptor` isn't strictly used by the `AcquireSegmentAction`, its more of a bit of tracking convenience...

i think the only hesitation i have here is it feels odd that this is the only array function that allows numeric inputs, i wonder if we should just allow...

> @clintropolis Thanks for rerunning the jobs. Before the rerun there were two failed tasks, and now there’s only one left. The error still looks like an environment issue… really...

rather than a historical server level configuration, this feels like something that should be handled as part of load rules so that different strategies could be done on a per...

>I also see that you have rendered isLazyLoadOnStart() introduced in https://github.com/apache/druid/pull/6988 at SegmentLocalCacheManager obsolete This was an accident, fixed in #18637. Also, your understanding of how the virtual storage feature...