fix hadoop dependency with rubix-presto

qubole / rubix

Cache File System optimized for columnar formats and object stores

Apache License 2.0

183 stars 74 forks source link

fix hadoop dependency with rubix-presto #72

Closed pvam closed 7 years ago

pvam commented 7 years ago

This fixes Rubix master UT.

pvam commented 7 years ago

Spoke offline. Dependency with other modules is not the best way to do this. Not yet sure why UT is failing with H2.6 dependency where the yarn method exists.

pvam commented 7 years ago

It should be fixed now. Will merge after approval.

abhishekdas99 commented 7 years ago

Did we test this in presto cluster ? If everything is fine, @pvam can we update this with the result. Then I can approve this .. Also get a approval from @stagraqubole as its Presto related. Lets get this done so that we can go ahead with the new release of Rubix

pvam commented 7 years ago

Opened #73, closing this. With #73, we dont dependency on yarn anymore we assume the property is populated via rubix client config.