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

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

                Author: ASF GitHub Bot
            Created on: 05/Jun/20 18:26
            Start Date: 05/Jun/20 18:26
    Worklog Time Spent: 10m 
      Work Description: kennknowles commented on pull request #11820:
URL: https://github.com/apache/beam/pull/11820#issuecomment-639689286


   PTAL. The first commit is unchanged, the second is a total rewrite (or 
unwrite, if you like).
   
    - Modified each `SqlQuery2`, etc, class to have just the amount of 
variation necessary
    - Made each test class run as `@Enclosed` with an inner class for each 
dialect. I don't love this use of inheritance, but it seems to be how JUnit 
wants things. It has a clearer test signal than using `@Parameterized` across 
the dialects, and allows separate `@Ignore` for broken/unsupported features.
   
   Did not do:
   
    - Jenkins job to publish benchmarks; planning that as a followup
    - Any other refactor to make suites more manageable


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


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

    Worklog Id:     (was: 441969)
    Time Spent: 2h 10m  (was: 2h)

> Add ZetaSQL Nexmark variant
> ---------------------------
>
>                 Key: BEAM-10093
>                 URL: https://issues.apache.org/jira/browse/BEAM-10093
>             Project: Beam
>          Issue Type: New Feature
>          Components: testing-nexmark
>            Reporter: Kenneth Knowles
>            Assignee: Kenneth Knowles
>            Priority: P2
>          Time Spent: 2h 10m
>  Remaining Estimate: 0h
>
> Most queries will be identical, but best to simply stay decoupled, so this is 
> a copy/paste/modify job.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to