ClickHouse.Net icon indicating copy to clipboard operation
ClickHouse.Net copied to clipboard

InvalidOperationException в функции BulkInsert

Open AntonAlekseevAA opened this issue 6 years ago • 1 comments

При попытке передать в метод пустую последовательность в качестве данных для вставки происходит исключение InvalidOperationException. Это исключение выкидывает оригинальная библиотека Clickhouse.Ado, это ее поведение в случае с пустой последовательностью. Было бы удобно, если бы метод BulkInsert мог производить валидацию входных данных, это избавило бы от необходимости каждый раз при вызове метода предварительно выполнять эту проверку.

AntonAlekseevAA avatar Jul 23 '19 14:07 AntonAlekseevAA

Дело в том, что для определения количества элементов в колекции, необходимо оббежать IEnumerable и весь его смысл тогда теряется. В текущей реализации оббегание коллекции отложено вплоть до передачи в кликхаус. Возможно, стоит рассмотреть замену IEnumerable на ICollection.

ilyabreev avatar Jul 23 '19 16:07 ilyabreev