flipp-oss / deimos

Framework to work with Kafka, Avro and ActiveRecord
Other
59 stars 22 forks source link

GUILD-574: Fix KafkaSource crashing on imports when import hooks are … #94

Closed DeeChau closed 3 years ago

DeeChau commented 3 years ago

…disabled

Pull Request Template

Description

Return ActiveRecord::Import::Result.new([], 0, [], []) when the kafka_config import is not set or the result set is empty. Fixes an issue where KafkaSource crashes on bulk-imports if import hooks are disabled.

Fixes #73

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist: