search

druid-io / pydruid

A Python connector for Druid
Other
506 stars 194 forks source link

Add order to scan valid_parts #207

Open MSadeghzadehG opened 4 years ago

MSadeghzadehG commented 4 years ago

I didn't see any other parts that need to change. if there is, please mention.

MSadeghzadehG commented 4 years ago

PTAL @john-bodley @mistercrunch

JustFeng commented 4 years ago

thx

john-bodley commented 4 years ago

@MSadeghzadehG should the list contain all the scan-query properties?

MSadeghzadehG commented 4 years ago

@john-bodley the resultFormat, batch_size, and legacy are missing. and I didn't see metrics property in scan query parameters. and I think the batch_size should add to the list. what's your opinion?

MSadeghzadehG commented 4 years ago

@JustFeng you're welcome.