eventuate-tram / eventuate-tram-sagas

Sagas for microservices
Other
1k stars 228 forks source link

ReactiveSqlQueryRow expects boolean fields to be Int #100

Open rrrship opened 10 months ago

rrrship commented 10 months ago

When migrating from the non-reactive Eventuate stack to reactive, then all the examples and SQLs had the saga_instance fields as boolean, which means the reactive stack won't work as expected, because it expects these fields to be Integer, having 0 or 1 values.

Changing the getBoolean method to just handle boolean types fixed the issue. I can provide a PR if this change makes sense?

cer commented 9 months ago

Sorry for the delay. Yes, please provide a PR.