[pull] master from apache:master #38
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![pull[bot]](https://avatars.githubusercontent.com/in/12910?s=60&v=4){kind=small}
This pull request fixes # Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. - [ ] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) --- - [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6529 from avishnus/kubernetes. Closes #6529 2a82df5 [Cheng Pan] Update dev/dependencyList a485546 [Cheng Pan] Update pom.xml 1ef0c40 [Cheng Pan] Update dev/dependencyList 4b45b00 [Cheng Pan] Update dev/dependencyList d0c5174 [Cheng Pan] Update dev/dependencyList bc2f6b8 [Cheng Pan] Update dev/dependencyList f76a7f4 [avishnus] Upgrading kubernetes-client Lead-authored-by: Cheng Pan <[email protected]> Co-authored-by: avishnus <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
# 🔍 Description ## Issue References 🔗 This pull request fixes #6537 ## Describe Your Solution 🔧 Upgrade Flink from 1.19.0 to 1.19.1. ## Types of changes 🔖 - [ ] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ #### Behavior With This Pull Request 🎉 #### Related Unit Tests --- # Checklist 📝 - [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6538 from SteNicholas/KYUUBI-6537. Closes #6537 b39d308 [SteNicholas] [KYUUBI #6537] Upgrade Flink from 1.19.0 to 1.19.1 Authored-by: SteNicholas <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
# 🔍 Description ## Issue References 🔗 This pull request fixes #6534 ## Describe Your Solution 🔧 check null ahead. ## Types of changes 🔖 - [x] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 Pass GHA --- # Checklist 📝 - [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6540 from pan3793/client-npe. Closes #6534 6976cd3 [Cheng Pan] [KYUUBI #6534] Fix checkEngineConnectionAlive may throw NPE Authored-by: Cheng Pan <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
# 🔍 Description ## Issue References 🔗 This pull request fixes #6523 ## Describe Your Solution 🔧 `KyuubiServer` respects configuration properties provided as `--conf` cli options with highest priority. ## Types of changes 🔖 - [ ] Bugfix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ REST endpoint starts at `10099` port: ```shell bin/kyuubi run --conf kyuubi.frontend.rest.bind.port=9999 ... 2024-07-10 17:57:24.998 INFO main org.apache.kyuubi.server.KyuubiRestFrontendService: Service[KyuubiRestFrontendService] is started. 2024-07-10 17:57:24.999 INFO main org.apache.kyuubi.server.KyuubiRestFrontendService: Exposing REST endpoint at: http://0.0.0.0:10099 2024-07-10 17:57:24.999 INFO main org.apache.kyuubi.server.KyuubiServer: Service[KyuubiServer] is started. ``` #### Behavior With This Pull Request :tada: REST endpoint starts at `9999` port: ```shell bin/kyuubi run --conf kyuubi.frontend.rest.bind.port=9999 ... 2024-07-10 17:49:41.659 INFO main org.apache.kyuubi.server.KyuubiRestFrontendService: Service[KyuubiRestFrontendService] is started. 2024-07-10 17:49:41.660 INFO main org.apache.kyuubi.server.KyuubiRestFrontendService: Exposing REST endpoint at: http://0.0.0.0:9999 2024-07-10 17:49:41.660 INFO main org.apache.kyuubi.server.KyuubiServer: Service[KyuubiServer] is started. ``` #### Related Unit Tests `kyuubi-common/src/test/scala/org/apache/kyuubi/config/KyuubiConfSuite.scala` --- # Checklist 📝 - [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6530 from dnskr/kyuubi-supports-conf-args. Closes #6523 c7dbb30 [dnskr] Parse command and shift args 4977af1 [dnskr] [KYUUBI #6523] Support configuration with --conf options Authored-by: dnskr <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
… before picking jobs # 🔍 Description This is similar to #5310. We need to wait for Jetty Server started before picking jobs, otherwise, it may get the wrong local HTTP server port -1. This only affects Batch V2 ## Types of changes 🔖 - [x] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 Tested in internal workloads. --- # Checklist 📝 - [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6542 from pan3793/batch-wait-started. Closes #6542 1f62deb [Cheng Pan] fix ee1d05d [Cheng Pan] KyuubiBatchService should wait for HTTP server started before picking jobs Authored-by: Cheng Pan <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )