qa_server 2.2.3 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a526113eb2d57af9e670446c76d9c92e33cdb1c8
-  data.tar.gz: b3f9253e899f8ff605632952f8ef5990a17e9147
+  metadata.gz: c54a8298febf924d051837fef371b727c32912f7
+  data.tar.gz: 5c038a7a516bd50834670fd393ee215086708b73
 SHA512:
-  metadata.gz: d19ffc6e5baf8ebf1d742fb6edc06fc9e620eadd8182a8632653e576fcee0d673b65f50eba97e0c345494b0d2970deb2f174f9583f003bd324934045c888c831
-  data.tar.gz: afe7fa02263b9b375da202bc87dfdb27d7f790653840a398d28630cfbd3f75d40c944485179260a94dd8a6561fa179a367763607954415cccd9feef86f91958f
+  metadata.gz: a36a6e1d5072d3978a18a34b84bf338b9d318c7db8948a6fb8bbb3ae2506714e8281b4dfa42faae61f5410a3379d38c2e2c2077d848c14a8895ce9dfb7193fb8
+  data.tar.gz: 1a5df12683d02f30c8bccc9f96d721faaffe06484223fe3ae543dc94f2948c11c164fa93a9833c57b0d523b898de54b19a02b344d5a56f9018d27f2a1b3a8304

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+### 3.0.0 (2019-10-09)
+
+* refactor performance data and graphs to include stats by action as well as by authority and time period
+* adds pagination start_record parameter for LD4P cached data
+* adds RDA Registry authority
+* splits LCNAF into 2 authorities for LD4P cached data focused locnames and real world object focused locnames_rwo
+* adds wikidata authority - NOTE: The first pass on fetching through this authority is non-performant.
+
 ### 2.2.3 (2019-09-11)
 
 * do not process and store performance data if it wasn't returned in the result

data/app/assets/stylesheets/qa_server/_authorities.scss

@@ -1,3 +1,6 @@
-td.table_subheading {
+th.table_subheading, td.table_subheading {
   background-color: lightgrey;
 }
+th.table_subheading {
+  text-align: center;
+}

data/app/assets/stylesheets/qa_server/_check-status.scss

@@ -73,6 +73,11 @@ td.position {
   background-color: #ffffff;
 }
 
+.status-not-supported {
+  text-align: center;
+  background-color: #F3EEEE;
+}
+
 label.horizontal-list {
   padding-right: 15px;
 }

data/app/assets/stylesheets/qa_server/_monitor-status.scss

@@ -20,11 +20,11 @@ div.performance-data-section-visible {
   display: block;
 }
 
-div.right-menu-section {
+div.performance-graph-menu-section {
   padding-top: 50px;
 }
 
-ul.right-menu {
+ul.time-period-menu, ul.action-menu {
   list-style-type: none;
   li.clickable {
     display: inline;

data/app/models/concerns/qa_server/performance_history_data_keys.rb

@@ -4,6 +4,10 @@
 module QaServer
   module PerformanceHistoryDataKeys
     ALL_AUTH = :all_authorities
+    SEARCH = :search
+    FETCH = :fetch
+    ALL_ACTIONS = :all_actions
+
     STATS = :stats
     FOR_DATATABLE = :datatable_stats
 
@@ -17,13 +21,22 @@ module QaServer
     BY_MONTH = :month
 
     LOW_LOAD = :low_load_ms
+    LOW_RETR = :low_retrieve_ms
+    LOW_GRPH = :low_load_graph_ms
     LOW_NORM = :low_normalization_ms
+    LOW_ACTN = :low_action_request_ms
     LOW_FULL = :low_full_request_ms
     AVG_LOAD = :avg_load_ms
+    AVG_RETR = :avg_retrieve_ms
+    AVG_GRPH = :avg_load_graph_ms
     AVG_NORM = :avg_normalization_ms
+    AVG_ACTN = :avg_action_request_ms
     AVG_FULL = :avg_full_request_ms
-    HIGH_LOAD = :max_load_ms
-    HIGH_NORM = :max_normalization_ms
-    HIGH_FULL = :max_full_request_ms
+    HIGH_LOAD = :high_load_ms
+    HIGH_RETR = :high_retrieve_ms
+    HIGH_GRPH = :high_load_graph_ms
+    HIGH_NORM = :high_normalization_ms
+    HIGH_ACTN = :high_action_request_ms
+    HIGH_FULL = :high_full_request_ms
   end
 end

data/app/models/qa_server/performance_history.rb

@@ -16,15 +16,13 @@ module QaServer
       include QaServer::PerformanceHistoryDataKeys
 
       # Save a scenario result
-      # @param run_id [Integer] the run on which to gather statistics
-      # @param result [Hash] the scenario result to be saved
-      def save_result(dt_stamp:, authority:, action:, size_bytes:, load_time_ms:, normalization_time_ms:) # rubocop:disable Metrics/ParameterLists
-        create(dt_stamp: dt_stamp,
+      # @param authority [String] name of the authority
+      # @param action [Symbol] type of action being evaluated (e.g. :fetch, :search)
+      # @return ActveRecord::Base for the new performance history record
+      def create_record(authority:, action:)
+        create(dt_stamp: Time.now.getlocal,
                authority: authority,
-               action: action,
-               size_bytes: size_bytes,
-               load_time_ms: load_time_ms,
-               normalization_time_ms: normalization_time_ms)
+               action: action)
       end
 
       # Performance data for a day, a month, a year, and all time for each authority.
@@ -32,34 +30,38 @@ module QaServer
       # @returns [Hash] performance statistics for the past 24 hours
       # @example
       #   { all_authorities:
-      #     { datatable_stats:
-      #       { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }
-      #     }
-      #     { day:
-      #       { 0: { hour: '1400', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         1: { hour: '1500', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         2: { hour: '1600', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         ...,
-      #         23: { hour: 'NOW', load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }}
-      #       }
-      #     }
-      #     { month:
-      #       { 0: { day: '07-15-2019', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         1: { day: '07-16-2019', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         2: { day: '07-17-2019', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         ...,
-      #         29: { day: 'TODAY', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }}
-      #       }
-      #     }
-      #     { year:
-      #       { 0: { month: '09-2019', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         1: { month: '10-2019', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         2: { month: '11-2019', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
-      #         ...,
-      #         11: { month: '08-2019', stats: { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }}
-      #       }
-      #     }
-      #     { AGROVOC_LD4L_CACHE: ... # same data for each authority  }
+      #     { search:
+      #       {
+      #         datatable_stats:
+      #           { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5,
+      #             retrieve_10th_ms: 12.3, graph_load_10th_ms: 12.3, normalization_10th_ms: 4.2, full_request_10th_ms: 16.5,
+      #             retrieve_90th_ms: 12.3, graph_load_90th_ms: 12.3, normalization_90th_ms: 4.2, full_request_90th_ms: 16.5 }
+      #         day:
+      #           { 0: { hour: '1400', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             1: { hour: '1500', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             2: { hour: '1600', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             ...,
+      #             23: { hour: 'NOW', retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }}
+      #           },
+      #         month:
+      #           { 0: { day: '07-15-2019', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             1: { day: '07-16-2019', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             2: { day: '07-17-2019', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             ...,
+      #             29: { day: 'TODAY', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }}
+      #           },
+      #         year:
+      #           { 0: { month: '09-2019', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             1: { month: '10-2019', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             2: { month: '11-2019', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }},
+      #             ...,
+      #             11: { month: '08-2019', stats: { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }}
+      #           }
+      #       },
+      #       fetch: { ... # same data as for search_stats }
+      #       all: { ... # same data as for search_stats }
+      #     },
+      #     AGROVOC_LD4L_CACHE: { ... # same data for each authority  }
       #   }
       def performance_data(datatype: :datatable)
         return if datatype == :none
@@ -87,27 +89,33 @@ module QaServer
         end
 
         def data_for_authority(authority_name: nil, datatype:)
-          data = {}
-          data[FOR_DATATABLE] = data_table_stats(authority_name) if calculate_datatable?(datatype)
-          if calculate_graphdata?(datatype)
-            data[FOR_DAY] = graph_data_service_class.average_last_24_hours(authority_name)
-            data[FOR_MONTH] = graph_data_service_class.average_last_30_days(authority_name)
-            data[FOR_YEAR] = graph_data_service_class.average_last_12_months(authority_name)
+          action_data = {}
+          [:search, :fetch, :all_actions].each do |action|
+            data = {}
+            data[FOR_DATATABLE] = data_table_stats(authority_name, action) if calculate_datatable?(datatype)
+            if calculate_graphdata?(datatype)
+              data[FOR_DAY] = graph_data_service_class.average_last_24_hours(authority_name: authority_name, action: action)
+              data[FOR_MONTH] = graph_data_service_class.average_last_30_days(authority_name: authority_name, action: action)
+              data[FOR_YEAR] = graph_data_service_class.average_last_12_months(authority_name: authority_name, action: action)
+            end
+            action_data[action] = data
           end
-          data
+          action_data
         end
 
         # Get statistics for all available data.
         # @param [String] auth_name - limit statistics to records for the given authority (default: all authorities)
         # @returns [Hash] performance statistics for the datatable during the expected time period
         # @example
-        #   { load_avg_ms: 12.3, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5, etc. }
-        def data_table_stats(auth_name)
+        #   { retrieve_avg_ms: 12.3, graph_load_avg_ms: 2.1, normalization_avg_ms: 4.2, full_request_avg_ms: 16.5,
+        #     retrieve_10th_ms: 12.3, graph_load_10th_ms: 12.3, normalization_10th_ms: 4.2, full_request_10th_ms: 16.5,
+        #     retrieve_90th_ms: 12.3, graph_load_90th_ms: 12.3, normalization_90th_ms: 4.2, full_request_90th_ms: 16.5 }
+        def data_table_stats(auth_name, action)
           records = records_for_last_24_hours(auth_name) ||
                     records_for_last_30_days(auth_name) ||
                     records_for_last_12_months(auth_name) ||
                     all_records(auth_name)
-          stats_calculator_class.new(records).calculate_stats(avg: true, low: true, high: true)
+          stats_calculator_class.new(records, action: action).calculate_stats(avg: true, low: true, high: true)
         end
 
         def expected_time_period

data/app/prepends/prepended_linked_data/find_term.rb

@@ -3,16 +3,58 @@ module PrependedLinkedData::FindTerm
   # Override Qa::Authorities::LinkedData::FindTerm#find method
   # @return [Hash] single term results in requested format
   def find(id, language: nil, replacements: {}, subauth: nil, format: nil, jsonld: false, performance_data: false) # rubocop:disable Metrics/ParameterLists
+    start_time_s = Time.now.to_f
+
     saved_performance_data = performance_data
     performance_data = true
-    full_results = super
-    return full_results unless full_results.is_a?(Hash) && full_results.key?(:performance)
-    QaServer::PerformanceHistory.save_result(dt_stamp: Time.now.getlocal,
-                                             authority: authority_name,
-                                             action: 'fetch',
-                                             size_bytes: full_results[:performance][:fetched_bytes],
-                                             load_time_ms: (full_results[:performance][:fetch_time_s] * 1000),
-                                             normalization_time_ms: (full_results[:performance][:normalization_time_s] * 1000))
-    saved_performance_data ? full_results : full_results[:results]
+    ph_record = QaServer::PerformanceHistory.create_record(authority: authority_name, action: 'fetch')
+    @phid = ph_record.id
+    begin
+      full_results = super
+      update_performance_history_record(full_results, start_time_s)
+    rescue Exception => e # rubocop:disable Lint/RescueException
+      ph_record.destroy
+      raise e
+    end
+    saved_performance_data || !full_results.key?(:results) ? full_results : full_results[:results]
   end
+
+  private
+
+    def update_performance_history_record(full_results, start_time_s)
+      ph_record = QaServer::PerformanceHistory.find(@phid)
+      return ph_record.destroy unless full_results.is_a?(Hash) && full_results.key?(:performance)
+      ph_record.action_time_ms = (Time.now.to_f - start_time_s) * 1000
+      ph_record.size_bytes = full_results[:performance][:fetched_bytes]
+      ph_record.retrieve_plus_graph_load_time_ms = full_results[:performance][:fetch_time_s] * 1000
+      ph_record.normalization_time_ms = full_results[:performance][:normalization_time_s] * 1000
+      ph_record.save
+    end
+
+    # Override to append performance history record id into the URL to allow access to the record in RDF::Graph
+    def load_graph(url:)
+      access_start_dt = Time.now.utc
+
+      url += "&phid=#{@phid}"
+      @full_graph = graph_service.load_graph(url: url)
+
+      access_end_dt = Time.now.utc
+      @access_time_s = access_end_dt - access_start_dt
+      @fetched_size = full_graph.triples.to_s.size if performance_data?
+      Rails.logger.info("Time to receive data from authority: #{access_time_s}s")
+    end
+
+    # Temporary override to fix bug.  Remove when QA PR #273 is merged and a new release is cut
+    def normalize_results
+      normalize_start_dt = Time.now.utc
+
+      json = perform_normalization
+
+      normalize_end_dt = Time.now.utc
+      @normalize_time_s = normalize_end_dt - normalize_start_dt
+      @normalized_size = json.to_s.size if performance_data?
+      Rails.logger.info("Time to convert data to json: #{normalize_time_s}s")
+      json = append_performance_data(json) if performance_data?
+      json
+    end
 end

data/app/prepends/prepended_linked_data/search_query.rb

@@ -3,16 +3,44 @@ module PrependedLinkedData::SearchQuery
   # Override Qa::Authorities::LinkedData::SearchQuery#search method
   # @return [String] json results for search query
   def search(query, language: nil, replacements: {}, subauth: nil, context: false, performance_data: false) # rubocop:disable Metrics/ParameterLists
+    start_time_s = Time.now.to_f
+
     saved_performance_data = performance_data
     performance_data = true
-    full_results = super
-    return full_results unless full_results.is_a?(Hash) && full_results.key?(:performance)
-    QaServer::PerformanceHistory.save_result(dt_stamp: Time.now.getlocal,
-                                             authority: authority_name,
-                                             action: 'search',
-                                             size_bytes: full_results[:performance][:fetched_bytes],
-                                             load_time_ms: (full_results[:performance][:fetch_time_s] * 1000),
-                                             normalization_time_ms: (full_results[:performance][:normalization_time_s] * 1000))
-    saved_performance_data ? full_results : full_results[:results]
+    ph_record = QaServer::PerformanceHistory.create_record(authority: authority_name, action: 'search')
+    @phid = ph_record.id
+    begin
+      full_results = super
+      update_performance_history_record(full_results, start_time_s)
+    rescue Exception => e # rubocop:disable Lint/RescueException
+      ph_record.destroy
+      raise e
+    end
+    saved_performance_data || !full_results.key?(:results) ? full_results : full_results[:results]
   end
+
+  private
+
+    def update_performance_history_record(full_results, start_time_s)
+      ph_record = QaServer::PerformanceHistory.find(@phid)
+      return ph_record.destroy unless full_results.is_a?(Hash) && full_results.key?(:performance)
+      ph_record.action_time_ms = (Time.now.to_f - start_time_s) * 1000
+      ph_record.size_bytes = full_results[:performance][:fetched_bytes]
+      ph_record.retrieve_plus_graph_load_time_ms = full_results[:performance][:fetch_time_s] * 1000
+      ph_record.normalization_time_ms = full_results[:performance][:normalization_time_s] * 1000
+      ph_record.save
+    end
+
+    # Override to append performance history record id into the URL to allow access to the record in RDF::Graph
+    def load_graph(url:)
+      access_start_dt = Time.now.utc
+
+      url += "&phid=#{@phid}"
+      @full_graph = graph_service.load_graph(url: url)
+
+      access_end_dt = Time.now.utc
+      @access_time_s = access_end_dt - access_start_dt
+      @fetched_size = full_graph.triples.to_s.size if performance_data?
+      Rails.logger.info("Time to receive data from authority: #{access_time_s}s")
+    end
 end

data/app/prepends/prepended_rdf/rdf_graph.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+require 'rdf/reader'
+
+module PrependedRdf::RdfGraph
+  ##
+  # Loads RDF statements from the given file or URL into `self`.
+  #
+  # @param  [String, #to_s]          url
+  # @param  [Hash{Symbol => Object}] options
+  #   Options from {RDF::Reader.open}
+  # @option options [RDF::Resource] :graph_name
+  #   Set set graph name of each loaded statement
+  # @return [void]
+  def load(url, graph_name: nil, **options) # rubocop:disable Metrics/MethodLength, Metrics/AbcSize
+    raise TypeError, "#{self} is immutable" if immutable?
+    phid, real_url = parse_phid(url)
+    ph_record = QaServer::PerformanceHistory.find(phid)
+    start_time_s = Time.now.to_f
+
+    reader = RDF::Reader.open(real_url, { base_uri: real_url }.merge(options))
+
+    end_time_s = Time.now.to_f
+    ph_record.retrieve_time_ms = (end_time_s - start_time_s) * 1000
+    QaServer.config.performance_tracker.write "#{format('%.6f', end_time_s - start_time_s)}, " # read data
+
+    start_time_s = Time.now.to_f
+
+    if graph_name
+      statements = []
+      reader.each_statement do |statement|
+        statement.graph_name = graph_name
+        statements << statement
+      end
+      insert_statements(statements)
+      statements.size
+    else
+      insert_statements(reader)
+      nil
+    end
+
+    end_time_s = Time.now.to_f
+    ph_record.graph_load_time_ms = (end_time_s - start_time_s) * 1000
+    ph_record.save
+    QaServer.config.performance_tracker.write "#{format('%.6f', end_time_s - start_time_s)}, " # load graph
+  end
+
+  private
+
+    def parse_phid(url)
+      i = url.rindex('&phid=')
+      phid = url[(i + 6)..url.length]
+      adjusted_url = url[0..(i - 1)]
+      [phid, adjusted_url]
+    end
+end

data/app/presenters/concerns/qa_server/monitor_status/performance_datatable_behavior.rb

@@ -1,59 +1,84 @@
 # frozen_string_literal: true
 # This module provides access methods into the performance data hash.
 module QaServer::MonitorStatus
-  module PerformanceDatatableBehavior
+  module PerformanceDatatableBehavior # rubocop:disable Metrics/ModuleLength
     include QaServer::PerformanceHistoryDataKeys
 
-    def datatable_stats(authority_data)
-      authority_data[FOR_DATATABLE]
+    def datatable_search_stats(authority_data)
+      data_table_for(authority_data, SEARCH)
     end
 
-    def low_load(stats)
-      format_stat stats[LOW_LOAD]
+    def datatable_fetch_stats(authority_data)
+      data_table_for(authority_data, FETCH)
+    end
+
+    def datatable_all_actions_stats(authority_data)
+      data_table_for(authority_data, ALL_ACTIONS)
+    end
+
+    def low_retrieve(stats)
+      format_stat stats, LOW_RETR
+    end
+
+    def low_graph_load(stats)
+      format_stat stats, LOW_GRPH
     end
 
     def low_normalization(stats)
-      format_stat stats[LOW_NORM]
+      format_stat stats, LOW_NORM
     end
 
     def low_full_request(stats)
-      format_stat stats[LOW_FULL]
+      format_stat stats, LOW_ACTN
+    end
+
+    def high_retrieve(stats)
+      format_stat stats, HIGH_RETR
     end
 
-    def high_load(stats)
-      format_stat stats[HIGH_LOAD]
+    def high_graph_load(stats)
+      format_stat stats, HIGH_GRPH
     end
 
     def high_normalization(stats)
-      format_stat stats[HIGH_NORM]
+      format_stat stats, HIGH_NORM
     end
 
     def high_full_request(stats)
-      format_stat stats[HIGH_FULL]
+      format_stat stats, HIGH_ACTN
     end
 
-    def avg_load(stats)
-      format_stat stats[AVG_LOAD]
+    def avg_retrieve(stats)
+      format_stat stats, AVG_RETR
+    end
+
+    def avg_graph_load(stats)
+      format_stat stats, AVG_GRPH
     end
 
     def avg_normalization(stats)
-      format_stat stats[AVG_NORM]
+      format_stat stats, AVG_NORM
     end
 
     def avg_full_request(stats)
-      format_stat stats[AVG_FULL]
+      format_stat stats, AVG_ACTN
+    end
+
+    def datatable_data_style(stats)
+      return "status-not-supported" if unsupported_action?(stats)
+      "status-neutral"
     end
 
     def low_full_request_style(stats)
-      performance_style_class(stats, LOW_FULL)
+      performance_style_class(stats, LOW_ACTN)
     end
 
     def high_full_request_style(stats)
-      performance_style_class(stats, HIGH_FULL)
+      performance_style_class(stats, HIGH_ACTN)
     end
 
     def avg_full_request_style(stats)
-      performance_style_class(stats, AVG_FULL)
+      performance_style_class(stats, AVG_ACTN)
     end
 
     def performance_table_description
@@ -75,12 +100,21 @@ module QaServer::MonitorStatus
         QaServer.config.performance_datatable_default_time_period
       end
 
-      def format_stat(stat)
-        return '' if stat.nil?
-        format("%0.1f", stat)
+      def data_table_for(authority_data, action)
+        authority_data[action][FOR_DATATABLE]
+      end
+
+      def unsupported_action?(stats)
+        stats[AVG_ACTN].nan?
+      end
+
+      def format_stat(stats, idx)
+        return '' if stats[idx].nil? || unsupported_action?(stats)
+        format("%0.1f", stats[idx])
       end
 
       def performance_style_class(stats, stat_key)
+        return "status-not-supported" if unsupported_action?(stats)
         return "status-bad" if max_threshold_exceeded(stats, stat_key)
         return "status-unknown" if desired_threshold_not_met(stats, stat_key)
         "status-good"