Re: Webui and absolute links

2017-08-06 Thread Jörg Schad
Maybe code can express my intentions more clearly:
https://github.com/apache/drill/pull/896

On Sun, Aug 6, 2017 at 5:36 PM, Jörg Schad  wrote:

> Looking at it, it seems to be a separate issue to me.
>
> That issue is about external references, I care about the internal links.
> These currently use absolute addressing at many places, which leads to the
> problem in the DC/OS UI that css files etc cannot be loaded:
>
>
> On Sun, Aug 6, 2017 at 5:28 PM, Pritesh Maker  wrote:
>
>> Hi
>>
>> Could this be a duplicate of this JIRA - DRILL-5699: Drill Web UI Page
>> Source Has Links To External Sites
>>
>> It’s in progress with this PR – https://github.com/apache/drill/pull/891
>>
>> Pritesh
>>
>> On 8/6/17, 4:28 PM, "Jörg Schad"  wrote:
>>
>> Hi,
>> I am just creating a drill package for DC/OS / Apache Mesos.
>>
>> The one problem I am facing is the UI integration. Unfortunately the
>> drill
>> UI uses static links (
>> e.g.,
>> 
>> 
>> ) .
>> This does not work nicely with the DC/OS UI, which would reference the
>> drill UI via https:///service/drill/.
>>
>> Are there any objections against a PR making the all the links
>> relative (
>> e.g.,
>> 
>> 
>> ).
>>
>>
>> Thank you!
>> Joerg
>>
>>
>>
>


Re: Webui and absolute links

2017-08-06 Thread Jörg Schad
Looking at it, it seems to be a separate issue to me.

That issue is about external references, I care about the internal links.
These currently use absolute addressing at many places, which leads to the
problem in the DC/OS UI that css files etc cannot be loaded:


On Sun, Aug 6, 2017 at 5:28 PM, Pritesh Maker  wrote:

> Hi
>
> Could this be a duplicate of this JIRA - DRILL-5699: Drill Web UI Page
> Source Has Links To External Sites
>
> It’s in progress with this PR – https://github.com/apache/drill/pull/891
>
> Pritesh
>
> On 8/6/17, 4:28 PM, "Jörg Schad"  wrote:
>
> Hi,
> I am just creating a drill package for DC/OS / Apache Mesos.
>
> The one problem I am facing is the UI integration. Unfortunately the
> drill
> UI uses static links (
> e.g.,
> 
> 
> ) .
> This does not work nicely with the DC/OS UI, which would reference the
> drill UI via https:///service/drill/.
>
> Are there any objections against a PR making the all the links
> relative (
> e.g.,
> 
> 
> ).
>
>
> Thank you!
> Joerg
>
>
>


Webui and absolute links

2017-08-06 Thread Jörg Schad
Hi,
I am just creating a drill package for DC/OS / Apache Mesos.

The one problem I am facing is the UI integration. Unfortunately the drill
UI uses static links (
e.g.,


) .
This does not work nicely with the DC/OS UI, which would reference the
drill UI via https:///service/drill/.

Are there any objections against a PR making the all the links relative (
e.g.,


).


Thank you!
Joerg