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

Mention Java as out of the box supported language #635

Merged
merged 1 commit into from
Dec 7, 2020

Conversation

honnix
Copy link
Member

@honnix honnix commented Dec 7, 2020

@EngHabu PTAL thanks.

@honnix
Copy link
Member Author

honnix commented Dec 7, 2020

Thanks for approving! Could you please help me merge it? It seems I don't have permission on this repo.

@ArgentFalcon ArgentFalcon merged commit e40f43a into master Dec 7, 2020
@honnix honnix deleted the out-of-the-box-java branch December 7, 2020 18:23
@kanterov
Copy link
Contributor

kanterov commented Dec 8, 2020

What about Scala. I think it's supported as well. API is designed specifically to have good UX when using Scala, and there are extensions specific for Scala.

@honnix
Copy link
Member Author

honnix commented Dec 8, 2020

This is complicated, unless we say JVM based languages, we will also miss something. But I'm OK adding Scala specifically.

@EngHabu
Copy link
Contributor

EngHabu commented Dec 8, 2020

Maybe we worded something like JVM based languages (e.g. Java, Scala... etc.) ?

@kumare3
Copy link
Contributor

kumare3 commented Dec 9, 2020

Not really all jvm languages right? As you can use the Java bridge but it may not be a native experience

@honnix
Copy link
Member Author

honnix commented Dec 9, 2020

Yeah I think it is reasonable to just list Java and Scala as first class citizens of JVM family.

eapolinario pushed a commit to eapolinario/flyte that referenced this pull request Dec 20, 2022
Signed-off-by: Kevin Su <[email protected]>


Signed-off-by: Yee Hing Tong <[email protected]>
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.

5 participants