It is also worth noting that this document is a snapshot rather than the long-term plan. As the SDK evolves, the annotations will almost certainly change with it (and will certainly expand).
+Aljoscha For streaming/batch execution separation, this is better served by configuration in the runner's build (e.g. specifying two separate executions in the pom.xml, one for streaming and one for batch). Given that the tests live in a separate module from the runner, this is likened to how RunnableOnService tests are currently executed by all of the runners. For sink, I think given the current implementations of sink there isn't a huge need; however, most sinks should be annotated with some form of superclass (although the implementation of sink requires side inputs, so this is also worth considering). +jb These would live on the tests proper, yes. On Sun, Jun 12, 2016 at 11:05 PM, Jean-Baptiste Onofré <j...@nanthrax.net> wrote: > Hi Thomas, > > it looks good to me. > > Just curious: the proposed annotations will be directly in the Java SDK > Test jar right ? > > Thanks, > Regards > JB > > > On 06/11/2016 01:34 AM, Thomas Groh wrote: > >> Hey Beamers! >> >> We have a lovely Capability Matrix ( >> http://beam.incubator.apache.org/capability-matrix/) which describes what >> runners can do, and what's in the model. However, right now we only have >> one way to specify that a test is useful to be executed in a runner, the >> RunnableOnService category. >> >> I've worked on a document to expand the number of annotations to be more >> in >> line with the capability matrix, which should help runner writers test >> more >> precisely with regards to the Beam model. The document is located at >> >> https://docs.google.com/document/d/1fICxq32t9yWn9qXhmT07xpclHeHX2VlUyVtpi2WzzGM/edit?usp=sharing >> , >> and I've added edit access for all of our committers. >> >> Feel free to take a look and leave any comments you may have, >> >> Thanks, >> >> Thomas >> >> > -- > Jean-Baptiste Onofré > jbono...@apache.org > http://blog.nanthrax.net > Talend - http://www.talend.com >
