qa_server 5.0.3 → 5.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d3407df755ae3f2b2868030cdd5f7c5f633b25bd
-  data.tar.gz: 7f9d4d161ce0914c80a858f459022a89ec85cfe7
+  metadata.gz: 91196c2e5b8c6f31cb7c3e38e6fd7567cae793f8
+  data.tar.gz: 3fa8f94c8f584a7e9110d5a046bc5abf234e9f28
 SHA512:
-  metadata.gz: c0380e8bfde89a4d92ba7c12e6379e71f4b4a7a87ff17e117c2834f7864cab187f9c133f4d34c1eb04453e71e13b9809c9da623f4a868def05661d4f8a6a0833
-  data.tar.gz: 9e327af739bcacc9ec9f61f17178b97dbd57c0e5918d7e24b8522e3cccf2e0b746913dd603c7a4cbcd9764ae56e38df9f62b9d35026522f356c7a8a1c70af36b
+  metadata.gz: 46dd223510421a7a72f1c1f507ea2e33b2dd1064595fafd783b17f9f555056cb44b5f6e744afd1b08dcbbbabe77fa309e73f5e7fe4d1c84280efa69dc443b45a
+  data.tar.gz: bcecafa888be4eeaf4dc6bfb44acd6b4b70b4b239e8de64b12138945e6c8930b268ad40487d14da36d8e8ab7a7779df2f472c9c1e70efbf659ca5c7d3d17efc9

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+### 5.1.0 (2019-12-10)
+
+* allow suppression of performance data gathering
+
 ### 5.0.3 (2019-12-3)
 
 * bug fix - use correct parameters for search query

data/app/prepends/prepended_linked_data/find_term.rb

@@ -3,6 +3,8 @@ module PrependedLinkedData::FindTerm
   # Override Qa::Authorities::LinkedData::FindTerm#find method
   # @return [Hash] single term results in requested format
   def find(id, request_header: {}, language: nil, replacements: {}, subauth: nil, format: nil, performance_data: false) # rubocop:disable Metrics/ParameterLists
+    return super if QaServer.config.suppress_performance_gathering
+
     start_time_s = Time.now.to_f
     request_header = build_request_header(language: language, replacements: replacements, subauth: subauth, format: format, performance_data: performance_data) if request_header.empty?
     saved_performance_data = performance_data || request_header[:performance_data]
@@ -33,6 +35,8 @@ module PrependedLinkedData::FindTerm
 
     # Override to append performance history record id into the URL to allow access to the record in RDF::Graph
     def load_graph(url:)
+      return super if QaServer.config.suppress_performance_gathering
+
       access_start_dt = Time.now.utc
 
       url += "&phid=#{@phid}"
@@ -44,8 +48,10 @@ module PrependedLinkedData::FindTerm
       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
+    # Temporary override to fix bug.  Remove when QA Issue #271 is fixed and a new release is cut
     def performance_data?
+      return super if QaServer.config.suppress_performance_gathering
+
       @performance_data == true
     end
 end

data/app/prepends/prepended_linked_data/search_query.rb

@@ -3,6 +3,8 @@ module PrependedLinkedData::SearchQuery
   # Override Qa::Authorities::LinkedData::SearchQuery#search method
   # @return [String] json results for search query
   def search(query, request_header: {}, language: nil, replacements: {}, subauth: nil, context: false, performance_data: false) # rubocop:disable Metrics/ParameterLists
+    return super if QaServer.config.suppress_performance_gathering
+
     start_time_s = Time.now.to_f
     request_header = build_request_header(language: language, replacements: replacements, subauth: subauth, context: context, performance_data: performance_data) if request_header.empty?
     saved_performance_data = performance_data || request_header[:performance_data]
@@ -33,6 +35,8 @@ module PrependedLinkedData::SearchQuery
 
     # Override to append performance history record id into the URL to allow access to the record in RDF::Graph
     def load_graph(url:)
+      return super if QaServer.config.suppress_performance_gathering
+
       access_start_dt = Time.now.utc
 
       url += "&phid=#{@phid}"

data/app/prepends/prepended_rdf/rdf_graph.rb

@@ -12,6 +12,8 @@ module PrependedRdf::RdfGraph
   #   Set set graph name of each loaded statement
   # @return [void]
   def load(url, graph_name: nil, **options) # rubocop:disable Metrics/MethodLength, Metrics/AbcSize
+    return super if QaServer.config.suppress_performance_gathering
+
     raise TypeError, "#{self} is immutable" if immutable?
     phid, real_url = parse_phid(url)
     ph_record = QaServer::PerformanceHistory.find(phid)

data/lib/generators/qa_server/templates/config/initializers/qa_server.rb

@@ -56,4 +56,9 @@ QaServer.config do |config|
   # config.navmenu_extra_leftitems = [
   #   { label: 'Your Menu Item Label', url: 'http://your.menu.item/url' }
   # ]
+
+  # Performance data is gathered on every incoming query.  If load is high, this can have a negative performance
+  # impact and may need to be suppressed.  Performance stats will not be gathered when this config is true.
+  # @param [Boolean] do not gather performance data when true (defaults to false for backward compatibitily)
+  # config.suppress_performance_gathering = false
 end

data/lib/qa_server/configuration.rb

@@ -114,5 +114,13 @@ module QaServer
         f.puts('action, http request, load graph, normalization, TOTAL, data size, authority')
       end
     end
+
+    # Performance data is gathered on every incoming query.  If load is high, this can have a negative performance
+    # impact and may need to be suppressed.  Performance stats will not be gathered when this config is true.
+    # @param [Boolean] do not gather performance data when true (defaults to false for backward compatibitily)
+    attr_writer :suppress_performance_gathering
+    def suppress_performance_gathering
+      @suppress_performance_gathering ||= false
+    end
   end
 end

data/lib/qa_server/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module QaServer
-  VERSION = '5.0.3'
+  VERSION = '5.1.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: qa_server
 version: !ruby/object:Gem::Version
-  version: 5.0.3
+  version: 5.1.0
 platform: ruby
 authors:
 - E. Lynette Rayle
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-12-04 00:00:00.000000000 Z
+date: 2019-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails