[#22465] ASH: Ignore ASH calls while fetching active session history

Summary:
Due to a refactor in de9fd5d  ActiveSessionHistory calls seem to include the ASH calls while fetching samples from the TServer, even if `ignore_ash_and_perform_calls` is set.

This diff fixes the regression.
Jira: DB-11382

Test Plan: wait_states-itest

Reviewers: asaha

Reviewed By: asaha

Subscribers: ybase, yql

Differential Revision: https://phorge.dev.yugabyte.com/D35227

src/yb/integration-tests/wait_states-itest.cc

@@ -462,6 +462,9 @@ void WaitStateTestCheckMethodCounts::VerifyCountsUnlocked() {
   // It is acceptable that some calls may not have populated their aux_info yet.
   // This probability should be very low.
   ASSERT_LE(method_counts_[""], 2 * num_ash_calls * kProbNoMethod);
+
+  // We ignore ASH calls from showing up during the pull.
+  ASSERT_EQ(wait_state_code_counts_["kDumpRunningRpc_WaitOnReactor"], 0);
 }
 
 bool WaitStateTestCheckMethodCounts::IsDone() {

src/yb/tserver/pg_client_service.cc

@@ -1390,6 +1390,7 @@ class PgClientServiceImpl::Impl {
     for (const auto& call : conn.calls_in_flight()) {
       if (ShouldIgnoreCall(req, call)) {
         VLOG(3) << "Ignoring " << call.wait_state().DebugString();
+        continue;
       }
       auto* wait_state = resp->add_wait_states();
       wait_state->CopyFrom(call.wait_state());