search

catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..
https://chromium.googlesource.com/catapult
BSD 3-Clause "New" or "Revised" License
1.91k stars 563 forks source link

Dashboard - Row indexes value? #4504

Closed simonhatch closed 6 years ago

simonhatch commented 6 years ago

Should we be indexing this?

@anniesullie @benshayden @eakuefner

benshayden commented 6 years ago

I think Row.value can be de-indexed.

eakuefner commented 6 years ago

@benshayden and I saw this yesterday as well; let's stop indexing it.

simonhatch commented 6 years ago

Sweet, the built-in index for Row is 13.5 TB

simonhatch commented 6 years ago

Update, these are used in projection queries over rows and must be indexed. Likely can't remove the index here unless we want to remove the projection query as well.