chore: use filter in entities query for selections as well

[saxenakshitiz]

No description provided.

[github-actions]

Test Results

367 tests  +1   367 ✅ +1   9s ⏱️ -1s
 68 suites +1     0 💤 ±0 
 68 files   +1     0 ❌ ±0 

Results for commit 9eae09e. ± Comparison against base commit 77592de.

This pull request removes 31 and adds 17 tests. Note that renamed tests count towards both.

      long: 60
      string: "PT1M"
      valueType: LONG
      valueType: STRING
    alias: "numCalls"
    columnName: "SERVICE.numCalls"
    value {
    }
  columnIdentifier {
…

org.hypertrace.gateway.service.baseline.BaselineServiceImplTest ‑ [1] function: AVGRATE
arguments {
  columnIdentifier {
    columnName: "SERVICE.numCalls"
    alias: "numCalls"
  }
}
arguments {
  literal {
    value {
      valueType: STRING
      string: "PT1M"
    }
  }
}
alias: "numCalls"

org.hypertrace.gateway.service.baseline.BaselineServiceImplTest ‑ [2] function: AVGRATE
arguments {
  columnIdentifier {
    columnName: "SERVICE.numCalls"
    alias: "numCalls"
  }
}
arguments {
  literal {
    value {
      valueType: LONG
      long: 60
    }
  }
}
alias: "numCalls"

org.hypertrace.gateway.service.explore.ExploreServiceTest ‑ [10] aggregations-with-groupby
org.hypertrace.gateway.service.explore.ExploreServiceTest ‑ [11] aggregations-with-groupby-and-the-rest-deprecated
org.hypertrace.gateway.service.explore.ExploreServiceTest ‑ [12] group-by-with-group-limit
org.hypertrace.gateway.service.explore.ExploreServiceTest ‑ [13] grouped-time-aggregation-with-group-limit
org.hypertrace.gateway.service.explore.ExploreServiceTest ‑ [14] simple-aggregations
org.hypertrace.gateway.service.explore.ExploreServiceTest ‑ [15] time-aggregations-space-filtered
org.hypertrace.gateway.service.explore.ExploreServiceTest ‑ [1] grouped-time-aggregation-with-group-limit-with-null-and-nonnull-value-group
org.hypertrace.gateway.service.explore.ExploreServiceTest ‑ [2] aggregations-with-multiple-groupbys-and-the-rest-deprecated
…

♻️ This comment has been updated with latest results.

[skjindal93]

do we need this?

[saxenakshitiz]

removed this commented section

[skjindal93]

returns an unmodifiable map. The map is being modified on L328

[saxenakshitiz]

That's correct. Moved it back to HashMap