csimplestring / delta-go

Native Delta Lake Implementation in Go
38 stars 8 forks source link

Fix issue #36 #37

Closed giulianopz closed 1 year ago

giulianopz commented 1 year ago

Declare the error returned from Next() outside of for-loop scope for it to be checked by the if-statement after the for-loop.

giulianopz commented 1 year ago

Hi @csimplestring,

the PR was approved but it's still not merged. There are also some dependabot PRs not yet merged. May I ask you if this project is still actively maintained? Just asking b/c I would like to contribute more to the project.

csimplestring commented 1 year ago

@giulianopz thanks for your PR. I just forget to merge yesterday.

Regarding this project, I am still maintaining it but there is less activity now due to the lack of maintenance from the original Scala project.

The Delta community had a new proposal called "Delta Kernel" which aims to provide a unified api implementation for Delta Lake transaction protocol. I love that idea and my next step will be focusing on that.

The current project is not widely used as I expected, because it is not trivial to integrate with other data processing framework in Go ecosystem. I hope the Delta Kernel api can speed up this.

I also checked the current progress of their kernel api in Java, and I think most of this project code can be reused, which is a god news.

I really like your motivation and I will ping you when I start working on it!

giulianopz commented 1 year ago

Thanks, @csimplestring. I was not aware of this proposal. I will keep an eye on it. For sure, ping me when the new project starts.