[ 
https://issues.apache.org/jira/browse/BEAM-7455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16852075#comment-16852075
 ] 

Valentyn Tymofieiev commented on BEAM-7455:
-------------------------------------------

avroio_test.py is a unit test, and I think an integration test may give 
additional coverage, especially if runners use a different implementation of 
Avro IO than we have in the SDK. [~chamikara] what is your take on this - do we 
need an integration test for Avro IO for Beam runners, or you think that unit 
test coverage is sufficient here?

Also cc: [~kasiak] [~kamilwu] who are working on BQ IO ITs for visibility. 

> Improve Avro IO integration test coverage on Python 3.
> ------------------------------------------------------
>
>                 Key: BEAM-7455
>                 URL: https://issues.apache.org/jira/browse/BEAM-7455
>             Project: Beam
>          Issue Type: Sub-task
>          Components: io-python-avro
>            Reporter: Valentyn Tymofieiev
>            Assignee: Frederik Bode
>            Priority: Major
>
> It seems that we don't have an integration test for Avro IO on Python 3:
> fastavro_it_test [1] depends on both avro and fastavro, however avro package 
> currently does not work with Beam on Python 3, so we don't have an 
> integration test that exercises Avro IO on Python 3. 
> We should add an integration test for Avro IO that does not need both 
> libraries at the same time, and instead can run using either library. 
> [~frederik] is this something you could help with? 
> cc: [~chamikara] [~Juta]
> [1] 
> https://github.com/apache/beam/blob/master/sdks/python/apache_beam/examples/fastavro_it_test.py



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

Reply via email to