-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update Flink version to 1.16.1 #70
Conversation
@shuttie would you be so kind to review and merge this PR? |
The problem is that I don't work on Findify anymore, so I'm not the maintainer and cannot merge PRs anymore. I've contacted Findify folks about the future of this project and will keep you posted on it. I'm OK to spend some time supporting the project (and flink-scala-api also) on community grounds, but as the project was originally made under the Findify umbrella, there should be some steps taken from their side. |
I see, thanks for clarification. Hopefully it will move forward. I think we need to probe an idea of making the findify/flink-scala-api as new official Scala API in the org.apache.flink, i.e. move the entire project to Apache umbrella. |
I don't think it's necessary or even desirable to bring back any form of official Scala API to Flink. It's perfectly fine if it's a third party wrapper, but if Findify doesn't want to maintain it, then it should be transferred to a new github org created for that purpose. |
Let's wait for a Findify answer and a feedback from @shuttie. P.S. I am already tired to see abandoned Scala projects since I started to use Scala 8 years ago. It is a already a cemetery of different Scala libraries related to Big Data, Data Science, because they were so to speak 3-rd party. I do not want to do one more experiment with Flink Scala wrapper. The only viable way for future Scala application in Flink is to have Scala wrapper as part of org.apache.flink. Still main Flink modules will be Scala-free. Module Dependency should be only from Scala wrapper to Flink Java modules, opposite dependency is forbidden. |
@shuttie did you get any response from Findify? I am wondering whether we should just fork this repo and continue ourselves. |
@novakov-alexey no honestly I'm not willing to commit to maintain a wrapper. Mostly because I don't think I know enough about Flink to do it correctly. To give an example I found an issue recently #71 and I have no idea how to fix it. That said I'd be happy to give a hand if someone else wants to take ownership. I totally understand what you're saying about dying wrappers, which is why I'm not willing to take ownership of a module if I'm not sure I will be able and willing to keep it alive long term. |
As a main author of this thing, I have no problem continuing to maintain the project on community grounds. The problem is about project rights - noone can at this moment merge PRs, and I'm trying to contact anyone from Findify regarding this. |
@shuttie any news on moving to a community Github organization? |
I spotted a start https://github.com/flink-extended/flink-scala-api |
Thanks for the activity. |
No description provided.