Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-08-09 Thread via GitHub
dongjoon-hyun commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2278617868 Let me close this because this PR is obsolete as of now, @jiangzho . :) -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-08-09 Thread via GitHub
dongjoon-hyun closed pull request #2: Operator 0.1.0 URL: https://github.com/apache/spark-kubernetes-operator/pull/2 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscr

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-08-06 Thread via GitHub
dongjoon-hyun commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2272129504 The following are merged too. - #28 - #29 Now, we can move on to the remaining modules, @jiangzho . - spark-operator-tests - spark-operator-docs -

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-07-28 Thread via GitHub
dongjoon-hyun commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2254902900 For the record, the following are merged. - #13 - #14 - #15 - #16 - #17 - #18 - #19 - #20 - #21 - #22 - #23 - #12 - #24

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-05-20 Thread via GitHub
dongjoon-hyun commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2120918277 For the record, `submission` module is merged finally today with SPARK-48326 (TODO). - https://github.com/apache/spark-kubernetes-operator/pull/10 From Spark r

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-05-03 Thread via GitHub
dongjoon-hyun commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2093722066 Hi, All. For the record, - We has been working on `submission` module and will merge with `Apache Spark 4.0.0-preview` as the first version. `Version-agnostic

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-26 Thread via GitHub
dongjoon-hyun commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2080135136 `Java API` module is merged. - #8 To @jiangzho , please make a PR for `spark-operator` and what you think it's tightly-related. -- This is an automated mes

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-22 Thread via GitHub
dongjoon-hyun commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2071471093 Hi, All. For the record, the following basic build and test PRs are merged. We are moving to API part. - #3 - #4 - #5 - #6 - #7 To @jian

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-19 Thread via GitHub
dongjoon-hyun commented on code in PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#discussion_r1572840033 ## config/checkstyle/checkstyle.xml: ## @@ -0,0 +1,195 @@ + + +https://checkstyle.org/dtds/configuration_1_3.dtd";> + + + + Review Comment: Pleas

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-19 Thread via GitHub
dongjoon-hyun commented on code in PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#discussion_r1572840033 ## config/checkstyle/checkstyle.xml: ## @@ -0,0 +1,195 @@ + + +https://checkstyle.org/dtds/configuration_1_3.dtd";> + + + + Review Comment: Pleas

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-19 Thread via GitHub
dongjoon-hyun commented on code in PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#discussion_r1572837826 ## build.gradle: ## @@ -1,3 +1,16 @@ +buildscript { + repositories { +maven { + url = uri("https://plugins.gradle.org/m2/";) +} + } +

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-19 Thread via GitHub
dongjoon-hyun commented on code in PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#discussion_r1572836874 ## build-tools/helm/spark-kubernetes-operator/values.yaml: ## @@ -0,0 +1,178 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or m

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-19 Thread via GitHub
dongjoon-hyun commented on code in PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#discussion_r1572836113 ## build-tools/helm/spark-kubernetes-operator/values.yaml: ## @@ -0,0 +1,178 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or m

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-19 Thread via GitHub
dongjoon-hyun commented on code in PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#discussion_r1572834233 ## .gitignore: ## @@ -16,3 +16,30 @@ build dependencies.lock **/dependencies.lock gradle/wrapper/gradle-wrapper.jar + +# Compiled source # +##

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-19 Thread via GitHub
dongjoon-hyun commented on code in PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#discussion_r1572833180 ## .github/workflows/build_and_test.yml: ## @@ -26,4 +26,20 @@ jobs: GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} with: c

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-19 Thread via GitHub
dongjoon-hyun commented on code in PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#discussion_r1572833649 ## .gitignore: ## @@ -16,3 +16,30 @@ build dependencies.lock **/dependencies.lock gradle/wrapper/gradle-wrapper.jar + +# Compiled source # +##

Re: [PR] Operator 0.1.0 [spark-kubernetes-operator]

2024-04-18 Thread via GitHub
jiangzho commented on PR #2: URL: https://github.com/apache/spark-kubernetes-operator/pull/2#issuecomment-2065538781 Rebase pick failed due to schedule change - attempted a non fast forward merge commit. I may do a final squash to make the history clear -- This is an automated message fr