TopK optimization to store not more than k values per window

microsoft / Trill

Trill is a single-node query processor for temporal or streaming data.

MIT License

1.25k stars 132 forks source link

TopK optimization to store not more than k values per window #121

Open arunkm opened 4 years ago

peterfreiling commented 4 years ago

Please make sure to honor all style cop (Roslyn analyzers) and VS style guidelines (should be red lines in your environment). If you are using VS and not seeing them let's figure out why. If not please find a way to enforce them locally.

peterfreiling commented 4 years ago

I realize the old code could use better documentation but the new code you are adding is not trivial and needs to be properly commented/documented. Please take time to comment thoroughly, and improve existing code when appropriate.