You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This release has a breaking change that I was planning on doing for some time, but I was not sure the best way to do it:
Now we expect to receive the information on the table name and the ID Columns of the table as an argument of the methods: Insert(), Update() and Delete().
Before this change we received this information on the ksql constructor which was awkward in two different ways:
It was necessary to create a new ksql instance for each table you were planning on using.
If you only wanted to use the Query* functions it was unnecessary to pass the table info during construction, making it possible to instantiate a ksql client whose 3 helper functions Insert, Update and Delete would not work because it was missing the table info.
This change was postponed until now because I wanted to be sure about this approach before making this breaking change. But I do believe this is necessary and that this is the simplest and more flexible solution we can get.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
This release has a breaking change that I was planning on doing for some time, but I was not sure the best way to do it:
Now we expect to receive the information on the table name and the ID Columns of the table as an argument of the methods:
Insert()
,Update()
andDelete()
.Before this change we received this information on the ksql constructor which was awkward in two different ways:
Query*
functions it was unnecessary to pass the table info during construction, making it possible to instantiate aksql
client whose 3 helper functionsInsert
,Update
andDelete
would not work because it was missing the table info.This change was postponed until now because I wanted to be sure about this approach before making this breaking change. But I do believe this is necessary and that this is the simplest and more flexible solution we can get.
This discussion was created from the release All the Imporant Features Implemented.
Beta Was this translation helpful? Give feedback.
All reactions