Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove code using hadoop class in pinoint commons to compile with jdk 1.6 #432

Closed
minwoo-jung opened this issue May 18, 2015 · 0 comments · Fixed by #438
Closed

Remove code using hadoop class in pinoint commons to compile with jdk 1.6 #432

minwoo-jung opened this issue May 18, 2015 · 0 comments · Fixed by #438

Comments

@minwoo-jung
Copy link
Member

Now test code in pipoint commons project uses hadoop class and is compiled by jdk 1.6.
After we upgraded hadoop version, it is need to Jdk 1.7 to be executed hadoop class.
therefore we remove code that use hadoop class in pinpoint common project.
minwoo-jung added a commit to minwoo-jung/pinpoint that referenced this issue May 18, 2015
@minwoo-jung
[pinpoint-apm#432] Remove code using hadoop class in pinoint commons …
93e3526 
…to compile with

jdk 1.6
minwoo-jung added a commit that referenced this issue May 18, 2015
@minwoo-jung
Merge pull request #436 from minwoo-jung/1.1.x
6d23c7d 
[#432] Remove code using hadoop class in pinoint commons to compile with jdk 1.6
Xylus added a commit to Xylus/pinpoint that referenced this issue May 19, 2015
@Xylus
pinpoint-apm#432 Removed hadoop dependency from Pinpoint commons test.
0756a31 
Pinpoint now uses HBase 1.0.x, which is compiled against jdk 1.7.
Pinpoint commons however is compiled with jdk 1.6 and would throw
UnsupportedClassVersionError when tests that use classes in Hadoop/HBase
library.
This commit removes these so such tests won't fail.
@Xylus Xylus mentioned this issue May 19, 2015
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove hadoop dependency from Pinpoint commons test. Xylus/pinpoint
1 participant
@minwoo-jung