jprdonnelly/impala
1
0
Fork 0
mirror of https://github.com/apache/impala.git synced 2025-12-25 02:03:09 -05:00
Code Issues Projects Releases Wiki Activity

IMPALA-5836: Improvements to Eclipse frontend configuration.

Browse Source 
Having recently gone through setting up Eclipse for frontend debugging,
I'm amending the process a tad to make it smoother for new developers.

* Adds the JDK sources package to the developer bootstrap
  process. These are useful if you need to point Eclipse to the source
  of Java libraries and are harmless to everyone else.
* Defaults to "downloadSource" for the "mvn eclipse:eclipse" plugin.
  This was being set explicitly in the Wiki instructions, but
  there's no reason not to set it on by default for everyone.
* Adds a "launcher" file for debugging the FE. Instead of creating
  a debug configuration manually, it's possible to share these across
  developers by checking them in. I checked it in via a mechanism
  that "mvn eclipse:eclipse" has to create additional files, to
  avoid literring the directory for non-Eclipse users.
* Changes the output directory used by Eclipse's compiler to
  a different one than Maven uses. Because we start Impala
  from the commandline, it's easy to start Impala and then
  change a line of Java, which forces Eclipse to kick in and
  replace the .class files. When this happens, the existing Impala
  starts complaining about class not found, because the underlying
  files have been overwritten. It's preferable to separate the
  directories and rely on Maven for the compiled Java.

Change-Id: Ia723fbf706cf409a8fb6b5ff0297c2b1ff7c9590
Reviewed-on: http://gerrit.cloudera.org:8080/7803
Reviewed-by: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Tested-by: Impala Public Jenkins
This commit is contained in:
Philip Zeyliger
2017-08-24 09:44:04 -07:00
committed by Impala Public Jenkins
parent 51c7fcd5dc
commit b524a5ce69
2 changed files with 41 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
bin/bootstrap_development.sh
View File

@@ -122,7 +122,7 @@ if [[ $DISTRIB_RELEASE = 14.04 ]]
then
JDK_VERSION=7
fi
apt-get --yes install openjdk-${JDK_VERSION}-jdk
apt-get --yes install openjdk-${JDK_VERSION}-jdk openjdk-${JDK_VERSION}-source
SET_JAVA_HOME="export JAVA_HOME=/usr/lib/jvm/java-${JDK_VERSION}-openjdk-amd64"
echo "$SET_JAVA_HOME" >> "${IMPALA_HOME}/bin/impala-config-local.sh"
eval "$SET_JAVA_HOME"