fix: use rpc method and responseStatusCode

[makeavish]

No description provided.

[request-info]

We would appreciate it if you could provide us with more info about this issue/pr!

[ankitnayan]

@makeavish please make sure the left menu filters in the trace-filter page and the tag filtering at the top are not using old columns.

[makeavish]

@ankitnayan This PR only updates getFilteredSpans API and doesn't affect any other API.
Deprecation of old columns will be taken up separately: #1967

[ankitnayan]

@ankitnayan This PR only updates getFilteredSpans API and doesn't affect any other API. Deprecation of old columns will be taken up separately: #1967

I wanted you to check if the other apis (getFilters and getAggregate) also need to be updated to use new columns and that they should not be using deprecated column anymore.

[makeavish]

@ankitnayan Already checked that, only httpCode is being used in other APIs.
So it's causing some duplicacy as we already cover httpCode in responseStatusCode but it doesn't cause any bugs. So can be removed later.

[ankitnayan]

@ankitnayan Already checked that, only httpCode is being used in other APIs. So it's causing some duplicacy as we already cover httpCode in responseStatusCode but it doesn't cause any bugs. So can be removed later.

okay...please create an issue for that...bugs might surface later

[makeavish]

Yes this issue was created for this objective: #1967

@ankitnayan Already checked that, only httpCode is being used in other APIs. So it's causing some duplicacy as we already cover httpCode in responseStatusCode but it doesn't cause any bugs. So can be removed later.

okay...please create an issue for that...bugs might surface later

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication