[GitHub] [druid] maytasm commented on issue #9724: Add integration tests for kafka ingestion

[GitBox <gi...@apache.org>]

maytasm commented on issue #9724: Add integration tests for kafka ingestion
URL: https://github.com/apache/druid/pull/9724#issuecomment-616025702
 
 
   Open question:
   For Kafka we have the above set of coverage for both when Kafka producer has transaction enabled and transaction disabled. This still take a long time with the parallelization of tests. I see two options...
   1) We can split the groups into two groups... kafka-index-transaction and kafka-index-non-transaction that can be run in parallel in travis. 
   2) We can disable some tests of either transaction or non-transaction. Specially, we can choose to run only one of the following classes: integration-tests/src/test/java/org/apache/druid/tests/indexer/ITKafkaIndexingServiceNonTransactionalSerializedTest.java or integration-tests/src/test/java/org/apache/druid/tests/indexer/ITKafkaIndexingServiceTransactionalSerializedTest.java as they are very similar and since they cannot be parallelized, they takes a long time to run (~20 mins each)

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org