IMSMWU/RClickhouse

Tables with datetime columns designated as Nullable() prompt error upon appending

Closed this issue · 1 comments

This bug is previously mentioned here: #75 (comment). I raise it again in case of omission.

I made the following changes in my personal repo for more than 4 months, and everything is fine since then. I am not familiar with the github or the automatic testing process so I have to leave the formal fix to you.

// line#366
// the master branch now
      return vecToScalar<ColumnDateTime, const std::time_t>(v);

// what I changed
      return vecToScalar<ColumnDateTime, const std::time_t>(v, nullCol);

Wish to see it fixed soon.

Hi @zylsun,
thank you very much for raising this issue. You are right, the same Bug which occurred for ColumnDate and was fixed in c2952bf also occurs with ColumnDateTime.
b49a033 is the fix for this issue.
Thanks!