Add record count to OneData File with Delta version upgrade.

[vamshigv]

Currently with delta version 2.0.2 we had remove recount count as the current version does not provide it. Add it back during upgrade.

[the-other-tim-brown]

We are now on Delta 2.4.X so this can be easily done. Labeling as a "good first issue" in case anyone wants to pick it up.

[gzagarwal]

We are now on Delta 2.4.X so this can be easily done. Labeling as a "good first issue" in case anyone wants to pick it up. as part of this change do we want to remove the record count from deltaActionsConverter as it was workaround

[gzagarwal]

Also the change is not present in the github code in main branch the code is changed in "core/src/main/java/io/onetable/delta/DeltaActionsConverter.java" https://github.com/apache/incubator-xtable/blob/main/core/src/main/java/org/apache/xtable/delta/DeltaActionsConverter.java

[the-other-tim-brown]

@gzagarwal The history may be hard to find but there is some method we can now call to get the row count if I am remembering correctly instead of computing it ourselves.

[gzagarwal]

@gzagarwal The history may be hard to find but there is some method we can now call to get the row count if I am remembering correctly instead of computing it ourselves.

Sure @the-other-tim-brown let me look into the xtable code and test case. if i find some thing then will ask to assign the jira.

[the-other-tim-brown]

@gzagarwal awesome! We're excited to have you start contributing!

[prithvijitguha]

Hi @the-other-tim-brown,

I really like this project! I wanted to take a look at something to pick up get used to things.

I came across this issue and I think it hasn't been closed yet. I found these two commits, so the previous method to get the count is the second(I think?) https://github.com/apache/incubator-xtable/commit/cdc0422cbe86107b6e752562f1b0395e9a5cc728

https://github.com/apache/incubator-xtable/commit/e48374c998a39b24deb68e44a4a765e9c4da5fb4

[the-other-tim-brown]

Hi @the-other-tim-brown,

I really like this project! I wanted to take a look at something to pick up get used to things.

I came across this issue and I think it hasn't been closed yet. I found these two commits, so the previous method to get the count is the second(I think?) cdc0422

e48374c

@prithvijitguha there should be a way to get the record count directly from the delta objects after upgrading. Take a look and raise a PR if you would like!

[Reactor11]

Hi, Is someone working on it? I am new to this project and would like to get started.

[the-other-tim-brown]

@Reactor11 I don't think anyone is so I have assigned you the issue. Let me know if you need any help getting started!

[Reactor11]

Hi @the-other-tim-brown - Thanks for assigning me this issues, I would need help in getting started! Let me know how can we connect for some time perhaps!