DalgoT4D / webapp

GNU Affero General Public License v3.0
5 stars 35 forks source link

Added a new row, yet it says no records synced, no bytes synced #970

Closed nairabhishek73 closed 21 hours ago

nairabhishek73 commented 2 days ago

https://jam.dev/c/0be40176-343e-448d-9e58-b84b818b86dc

fatchat commented 2 days ago

@mdshamoon

in the airbyte UI

http://localhost:8000/workspaces/4564267a-b0d4-456e-9cfd-dce8a3731138/connections/d4ccdb75-b31c-4f5e-b67e-224e6d2353cc/job-history

job 553 shows

these are returned in the dalgo api response as well, just under different field names:

image

instead of using recordsSynced and bytesSynced we should use recordsEmitted and bytesEmitted

BUT let's also find out what these other two fields mean, are we supposed to ignore them? from the words i feel like we care more about "synced" than "emitted"

fatchat commented 2 days ago

ok this is explained in https://github.com/airbytehq/airbyte/issues/15333

so yes we need to update our fields

fatchat commented 1 day ago

fixed; checked ui-testing-6 on staging, and on production arghyam, stir, kef, sneha

fatchat commented 1 day ago

@nairabhishek73

Image

nairabhishek73 commented 21 hours ago

Hi works!