[ 
https://issues.apache.org/jira/browse/BEAM-4283?focusedWorklogId=109308&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-109308
 ]

ASF GitHub Bot logged work on BEAM-4283:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 06/Jun/18 08:38
            Start Date: 06/Jun/18 08:38
    Worklog Time Spent: 10m 
      Work Description: echauchot commented on a change in pull request #5464: 
[BEAM-4283] Write Nexmark execution times to bigquery
URL: https://github.com/apache/beam/pull/5464#discussion_r193334164
 
 

 ##########
 File path: 
sdks/java/nexmark/src/main/java/org/apache/beam/sdk/nexmark/Main.java
 ##########
 @@ -74,22 +96,89 @@ void runAll(OptionT options, NexmarkLauncher 
nexmarkLauncher) throws IOException
           appendPerf(options.getPerfFilename(), configuration, perf);
           actual.put(configuration, perf);
           // Summarize what we've run so far.
-          saveSummary(null, configurations, actual, baseline, start);
+          saveSummary(null, configurations, actual, baseline, start, options);
         }
       }
+      if (options.getExportSummaryToBigQuery()){
+        savePerfsToBigQuery(options, actual, null);
+      }
     } finally {
       if (options.getMonitorJobs()) {
         // Report overall performance.
-        saveSummary(options.getSummaryFilename(), configurations, actual, 
baseline, start);
+        saveSummary(options.getSummaryFilename(), configurations, actual, 
baseline, start, options);
         saveJavascript(options.getJavascriptFilename(), configurations, 
actual, baseline, start);
       }
     }
-
     if (!successful) {
       throw new RuntimeException("Execution was not successful");
     }
   }
 
+  @VisibleForTesting
+  static void savePerfsToBigQuery(
+      NexmarkOptions options,
+      Map<NexmarkConfiguration, NexmarkPerf> perfs,
+      @Nullable FakeBigQueryServices fakeBigQueryServices) {
+    Pipeline pipeline = Pipeline.create(options);
 
 Review comment:
   Also an extra point is that the data volume to insert into BQ is at maximum 
12 (queries) x 3 fields for each nexmark run; thus I don't think there is any 
performance/speed concern here.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 109308)
    Time Spent: 6h  (was: 5h 50m)

> Export nexmark execution times to bigQuery
> ------------------------------------------
>
>                 Key: BEAM-4283
>                 URL: https://issues.apache.org/jira/browse/BEAM-4283
>             Project: Beam
>          Issue Type: Sub-task
>          Components: examples-nexmark
>            Reporter: Etienne Chauchot
>            Assignee: Etienne Chauchot
>            Priority: Major
>          Time Spent: 6h
>  Remaining Estimate: 0h
>
> Nexmark only outputs the results collection to bigQuery and prints in the 
> console the execution times. To supervise Nexmark execution times, we need to 
> store them as well per runner/query/mode



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to