Fixed Failing Build for Databricks 11.3 on branch 23.12 [databricks] #10435
Conversation
This reverts commit dacc6fe.
Signed-off-by: raza jafri <[email protected]>
razajafri
requested review from
jlowe,
revans2,
tgravescs,
GaryShen2008 and
NvTimLiu
as code owners
razajafri
changed the title
jenkins/databricks/build.sh
Outdated
| if [[ ! -d $HOME/apache-maven-3.6.3 ]]; then | ||
| wget https://archive.apache.org/dist/maven/maven-3/3.6.3/binaries/apache-maven-3.6.3-bin.tar.gz -P /tmp | ||
| tar xf /tmp/apache-maven-3.6.3-bin.tar.gz -C $HOME | ||
| sudo ln -s $HOME/apache-maven-3.6.3/bin/mvn /usr/local/bin/mvn | ||
| fi | ||
| sudo apt install -y maven rsync |
There was a problem hiding this comment.
We should not revert this part, deterministic Maven version is more robust either way regardless whether the apt package for Maven is back
|
While triaging an issue yesterday, I wasn't able to build the plugin for 23.12 or 24.04. It only worked after reverting the change.
AFAIK Databricks has reverted this for 11.3. That could change in the future |
|
build |
|
Why this PR targets branch-23.12? Do we need to fix it in 23.12? |
Yes, because databricks has rolled back the change for PythonRunner |
|
Since we don't plan on issuing another 23.12 release and given that the PythonRunner changes might be pulled back in in the near future, we are closing this issue |
Revert "PythonRunner Changes [databricks]"
fixes #10432