Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issue #36 #37

Conversation

giulianopz
Copy link
Contributor

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

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 giulianopz force-pushed the fix/err-in-for-loop-scope-not-checked branch from 9632701 to 61bd146 Compare September 13, 2023 20:18
@giulianopz
Copy link
Contributor Author

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 csimplestring merged commit 80cc2fa into csimplestring:master Sep 20, 2023
1 check passed
@csimplestring
Copy link
Owner

@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
Copy link
Contributor Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants