-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Druid doesn't build with OpenJDK (on Ubuntu 12.04) #14
Comments
What error do you get? Would be nice if it built. |
@octavore may have some feedback for you. It was a fresh VM and there may have been issues with the maven version, we're not sure. |
I had issues as well, switching to the sun (oracle) jdk fixed it.. I seem to recall finding where I think the issue might be fixed in openjdk but not yet trickled into the packaged version in ubuntu.. Here's a snippet from build.sh output where it fails: [INFO] Building druid-common 0.1.0-SNAPSHOT
Caused by: org.apache.maven.plugin.CompilationFailureException: Compilation failure
|
I got it to build as follows on Ubuntu 12.04, by installing OpenJDK7 and pointing my JAVA_HOME to it: sudo apt-get install openjdk-7-jdk maven It's related to these: https://bugs.openjdk.java.net/show_bug.cgi?id=100167 |
I just pushed a fix that I think will work around this issue. If it doesn't, well, the code still works as is and has a lot more spots where it's specifying generics, but oh well ;). |
Not working... |
Ok, actually read the links jeremybarnes posted and I think that removing the Maps.newTreeMap() calls and just using the plain old "new TreeMap<>()" invocation will fix the problem. Pushed a fix, hopefully it works. |
Correct druid_inabox package install.
* RowBasedIndexedTable: Add specialized index types for long keys. (apache#10430) * RowBasedIndexedTable: Add specialized index types for long keys. Two new index types are added: 1) Use an int-array-based index in cases where the difference between the min and max values isn't too large, and keys are unique. 2) Use a Long2ObjectOpenHashMap (instead of the prior Java HashMap) in all other cases. In addition: 1) RowBasedIndexBuilder, a new class, is responsible for picking which index implementation to use. 2) The IndexedTable.Index interface is extended to support using unboxed primitives in the unique-long-keys case, and callers are updated to use the new functionality. Other key types continue to use indexes backed by Java HashMaps. * Fixup logic. * Add tests. * Web console: switch to switches instead of checkboxes (apache#10454) * switch to switches * add img alt * add relative * change icons * update snapshot * Fix the offset setting in GoogleStorage#get (apache#10449) * Fix the offset in get of GCP object * upgrade compute dependency * fix version * review comments * missed * Fix the task id creation in CompactionTask (apache#10445) * Fix the task id creation in CompactionTask * review comments * Ignore test for range partitioning and segment lock Co-authored-by: Gian Merlino <gianmerlino@gmail.com> Co-authored-by: Vadim Ogievetsky <vadim@ogievetsky.com> Co-authored-by: Abhishek Agarwal <1477457+abhishekagarwal87@users.noreply.github.com>
I'm sure this is not a big deal to anyone but it might warrant a note in the docs - or maybe Maven can catch it?
The text was updated successfully, but these errors were encountered: