[jira] [Commented] (PHOENIX-3223) Add hadoop classpath to PQS classpath

2016-09-01 Thread Alicia Ying Shu (JIRA)

[ 
https://issues.apache.org/jira/browse/PHOENIX-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15455938#comment-15455938
 ] 

Alicia Ying Shu commented on PHOENIX-3223:
--

[~elserj] LGTM. Thanks.

> Add hadoop classpath to PQS classpath
> -
>
> Key: PHOENIX-3223
> URL: https://issues.apache.org/jira/browse/PHOENIX-3223
> Project: Phoenix
>  Issue Type: Bug
>Reporter: Sergio Peleato
>Assignee: Josh Elser
> Fix For: 4.9.0, 4.8.1
>
>
> Follow-on from PHOENIX-2369: running on Microsoft's Azure Data Lake Store 
> nets some ClassNotFoundExceptions on the corresponding classes. The library 
> is available via the Hadoop installation, but we miss it because PQS isn't 
> adding the {{hadoop classpath}} like {{sqlline.py}} is (only 
> {{$HADOOP_CONF_DIR}}, oops).
> Alicia's patch on PHOENIX-2369 already did the calculation work to get the 
> classpath; just need to simply add it to PQS' classpath now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (PHOENIX-3223) Add hadoop classpath to PQS classpath

2016-08-30 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/PHOENIX-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450598#comment-15450598
 ] 

Hudson commented on PHOENIX-3223:
-

FAILURE: Integrated in Jenkins build Phoenix-4.8-HBase-1.2 #2 (See 
[https://builds.apache.org/job/Phoenix-4.8-HBase-1.2/2/])
PHOENIX-3223 Add `hadoop classpath` to PQS classpath when available (elserj: 
rev 008883a5805c117fbd2209d911b336b44109eb9c)
* (edit) bin/queryserver.py


> Add hadoop classpath to PQS classpath
> -
>
> Key: PHOENIX-3223
> URL: https://issues.apache.org/jira/browse/PHOENIX-3223
> Project: Phoenix
>  Issue Type: Bug
>Reporter: Sergio Peleato
>Assignee: Josh Elser
> Fix For: 4.9.0, 4.8.1
>
>
> Follow-on from PHOENIX-2369: running on Microsoft's Azure Data Lake Store 
> nets some ClassNotFoundExceptions on the corresponding classes. The library 
> is available via the Hadoop installation, but we miss it because PQS isn't 
> adding the {{hadoop classpath}} like {{sqlline.py}} is (only 
> {{$HADOOP_CONF_DIR}}, oops).
> Alicia's patch on PHOENIX-2369 already did the calculation work to get the 
> classpath; just need to simply add it to PQS' classpath now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (PHOENIX-3223) Add hadoop classpath to PQS classpath

2016-08-30 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/PHOENIX-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450579#comment-15450579
 ] 

Hudson commented on PHOENIX-3223:
-

FAILURE: Integrated in Jenkins build Phoenix-master #1376 (See 
[https://builds.apache.org/job/Phoenix-master/1376/])
PHOENIX-3223 Add `hadoop classpath` to PQS classpath when available (elserj: 
rev 93a9c9187f63578b9da9864e06c0df3231c9f61c)
* (edit) bin/queryserver.py


> Add hadoop classpath to PQS classpath
> -
>
> Key: PHOENIX-3223
> URL: https://issues.apache.org/jira/browse/PHOENIX-3223
> Project: Phoenix
>  Issue Type: Bug
>Reporter: Sergio Peleato
>Assignee: Josh Elser
> Fix For: 4.9.0, 4.8.1
>
>
> Follow-on from PHOENIX-2369: running on Microsoft's Azure Data Lake Store 
> nets some ClassNotFoundExceptions on the corresponding classes. The library 
> is available via the Hadoop installation, but we miss it because PQS isn't 
> adding the {{hadoop classpath}} like {{sqlline.py}} is (only 
> {{$HADOOP_CONF_DIR}}, oops).
> Alicia's patch on PHOENIX-2369 already did the calculation work to get the 
> classpath; just need to simply add it to PQS' classpath now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (PHOENIX-3223) Add hadoop classpath to PQS classpath

2016-08-30 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/PHOENIX-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450377#comment-15450377
 ] 

ASF GitHub Bot commented on PHOENIX-3223:
-

Github user asfgit closed the pull request at:

https://github.com/apache/phoenix/pull/205


> Add hadoop classpath to PQS classpath
> -
>
> Key: PHOENIX-3223
> URL: https://issues.apache.org/jira/browse/PHOENIX-3223
> Project: Phoenix
>  Issue Type: Bug
>Reporter: Sergio Peleato
>Assignee: Josh Elser
> Fix For: 4.9.0, 4.8.1
>
>
> Follow-on from PHOENIX-2369: running on Microsoft's Azure Data Lake Store 
> nets some ClassNotFoundExceptions on the corresponding classes. The library 
> is available via the Hadoop installation, but we miss it because PQS isn't 
> adding the {{hadoop classpath}} like {{sqlline.py}} is (only 
> {{$HADOOP_CONF_DIR}}, oops).
> Alicia's patch on PHOENIX-2369 already did the calculation work to get the 
> classpath; just need to simply add it to PQS' classpath now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (PHOENIX-3223) Add hadoop classpath to PQS classpath

2016-08-30 Thread Josh Elser (JIRA)

[ 
https://issues.apache.org/jira/browse/PHOENIX-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15450361#comment-15450361
 ] 

Josh Elser commented on PHOENIX-3223:
-

Thanks [~sergey.soldatov]! Will commit shortly. I've tested this locally and it 
worked as expected for me.

> Add hadoop classpath to PQS classpath
> -
>
> Key: PHOENIX-3223
> URL: https://issues.apache.org/jira/browse/PHOENIX-3223
> Project: Phoenix
>  Issue Type: Bug
>Reporter: Sergio Peleato
>Assignee: Josh Elser
> Fix For: 4.9.0, 4.8.1
>
>
> Follow-on from PHOENIX-2369: running on Microsoft's Azure Data Lake Store 
> nets some ClassNotFoundExceptions on the corresponding classes. The library 
> is available via the Hadoop installation, but we miss it because PQS isn't 
> adding the {{hadoop classpath}} like {{sqlline.py}} is (only 
> {{$HADOOP_CONF_DIR}}, oops).
> Alicia's patch on PHOENIX-2369 already did the calculation work to get the 
> classpath; just need to simply add it to PQS' classpath now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (PHOENIX-3223) Add hadoop classpath to PQS classpath

2016-08-30 Thread Sergey Soldatov (JIRA)

[ 
https://issues.apache.org/jira/browse/PHOENIX-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449996#comment-15449996
 ] 

Sergey Soldatov commented on PHOENIX-3223:
--

LGTM. [~elserj] nice work :)

> Add hadoop classpath to PQS classpath
> -
>
> Key: PHOENIX-3223
> URL: https://issues.apache.org/jira/browse/PHOENIX-3223
> Project: Phoenix
>  Issue Type: Bug
>Reporter: Sergio Peleato
>Assignee: Josh Elser
> Fix For: 4.9.0, 4.8.1
>
>
> Follow-on from PHOENIX-2369: running on Microsoft's Azure Data Lake Store 
> nets some ClassNotFoundExceptions on the corresponding classes. The library 
> is available via the Hadoop installation, but we miss it because PQS isn't 
> adding the {{hadoop classpath}} like {{sqlline.py}} is (only 
> {{$HADOOP_CONF_DIR}}, oops).
> Alicia's patch on PHOENIX-2369 already did the calculation work to get the 
> classpath; just need to simply add it to PQS' classpath now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (PHOENIX-3223) Add hadoop classpath to PQS classpath

2016-08-30 Thread Josh Elser (JIRA)

[ 
https://issues.apache.org/jira/browse/PHOENIX-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449987#comment-15449987
 ] 

Josh Elser commented on PHOENIX-3223:
-

[~sergey.soldatov], [~aliciashu], either of you have a chance to look at this? 
Pretty simple change.

For got to mention originally, thanks to [~speleato] for reporting this one in 
the first place.

> Add hadoop classpath to PQS classpath
> -
>
> Key: PHOENIX-3223
> URL: https://issues.apache.org/jira/browse/PHOENIX-3223
> Project: Phoenix
>  Issue Type: Bug
>Reporter: Josh Elser
>Assignee: Josh Elser
> Fix For: 4.9.0, 4.8.1
>
>
> Follow-on from PHOENIX-2369: running on Microsoft's Azure Data Lake Store 
> nets some ClassNotFoundExceptions on the corresponding classes. The library 
> is available via the Hadoop installation, but we miss it because PQS isn't 
> adding the {{hadoop classpath}} like {{sqlline.py}} is (only 
> {{$HADOOP_CONF_DIR}}, oops).
> Alicia's patch on PHOENIX-2369 already did the calculation work to get the 
> classpath; just need to simply add it to PQS' classpath now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (PHOENIX-3223) Add hadoop classpath to PQS classpath

2016-08-30 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/PHOENIX-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15449984#comment-15449984
 ] 

ASF GitHub Bot commented on PHOENIX-3223:
-

GitHub user joshelser opened a pull request:

https://github.com/apache/phoenix/pull/205

PHOENIX-3223 Add `hadoop classpath` to PQS classpath when available



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/joshelser/phoenix PHOENIX-3223-pqs-classpath

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/phoenix/pull/205.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #205


commit d65593b194f6bb89bc0e69e744d2c0e92971cdb8
Author: Josh Elser 
Date:   2016-08-30T19:58:50Z

PHOENIX-3223 Add `hadoop classpath` to PQS classpath when available




> Add hadoop classpath to PQS classpath
> -
>
> Key: PHOENIX-3223
> URL: https://issues.apache.org/jira/browse/PHOENIX-3223
> Project: Phoenix
>  Issue Type: Bug
>Reporter: Josh Elser
>Assignee: Josh Elser
> Fix For: 4.9.0, 4.8.1
>
>
> Follow-on from PHOENIX-2369: running on Microsoft's Azure Data Lake Store 
> nets some ClassNotFoundExceptions on the corresponding classes. The library 
> is available via the Hadoop installation, but we miss it because PQS isn't 
> adding the {{hadoop classpath}} like {{sqlline.py}} is (only 
> {{$HADOOP_CONF_DIR}}, oops).
> Alicia's patch on PHOENIX-2369 already did the calculation work to get the 
> classpath; just need to simply add it to PQS' classpath now.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)