Hi Igor Drozdov,

Taking care of absolute path and relative path correctly make sense,
Thanks for investigating the issue!

Thanks,
moon

On Wed, Dec 14, 2016 at 6:33 AM Igor Drozdov <[email protected]> wrote:

> Hello,
>
> I'm investigating issue
> https://issues.apache.org/jira/browse/ZEPPELIN-1797 (first case, "Cannot
> fetch dependencies for com.databricks:spark-csv_2.10:1.3.0")
>
> The reason is than
> org.apache.zeppelin.dep.Booter#newRepositorySystemSession method is unable
> to use absolute path for local repo. It creates broken path for it.
>
> Underlying exception is "Specified destination directory cannot be
> created:
> C:\dev\projects\zeppelin\zeppelin-interpreter\..\C:\Users\IGOR_D~1\AppData\Local\Temp\ZeppelinLTest_1481724731009\test-repo\com\databricks\spark-csv_2.10\1.3.0".
> As you can see, it appends absolute path to the end of ZEPPELIN_HOME.
>
> Huge amount of tests passes absolute path to local repo to
> DependencyResolver.
>
> I suggest modification to this method to handle absolute paths properly:
> When we pass absolute path it should use it for local repo
> When we pass relative path it should resolve it as relative to
> ZEPPELIN_HOME (current behavior)
>
> What do you think? Is it safe?
>
> Thank you
> Igor Drozdov
>

Reply via email to