Skip to content

Conversation

@911432
Copy link
Contributor

@911432 911432 commented Mar 31, 2023

Fixed typos for Flink and Hive release version.
I want to fix the Flink version to 1.17.0 and Hive version to 3.1.3 but I don't know if iceberg supports it.

@hililiwei
Copy link
Contributor

nit: You don't need to start a new PR every time you update, just push the changes to your branch.

@Fokko Fokko self-requested a review April 10, 2023 22:25
```

Start the Flink SQL client. There is a separate `flink-runtime` module in the Iceberg project to generate a bundled jar, which could be loaded by Flink SQL client directly. To build the `flink-runtime` bundled jar manually, build the `iceberg` project, and it will generate the jar under `<iceberg-root-dir>/flink-runtime/build/libs`. Or download the `flink-runtime` jar from the [Apache repository](https://repo.maven.apache.org/maven2/org/apache/iceberg/iceberg-flink-runtime-1.16/{{% icebergVersion %}}/).
Start the Flink SQL client. There is a separate `flink-runtime` module in the Iceberg project to generate a bundled jar, which could be loaded by Flink SQL client directly. To build the `flink-runtime` bundled jar manually, build the `iceberg` project, and it will generate the jar under `<iceberg-root-dir>/flink-runtime/build/libs`. Or download the `flink-runtime` jar from the [Apache repository](https://repo.maven.apache.org/maven2/org/apache/iceberg/iceberg-flink-runtime-1.17/{{% icebergVersion %}}/).
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This link [https://repo.maven.apache.org/maven2/org/apache/iceberg/iceberg-flink-runtime-1.17] doesn't exist yet.
But I think someone will make it.

@911432
Copy link
Contributor Author

911432 commented Apr 21, 2023

I couldn't make a manual that fits the changing iceberg.
So I'll just change the link and upload it again.

@911432 911432 closed this Apr 21, 2023
@911432 911432 deleted the patch-1 branch April 21, 2023 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants