[jira] [Commented] (HIVE-16082) Allow user to change number of listener thread in LlapTaskCommunicator

2017-03-01 Thread Hive QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-16082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15891445#comment-15891445
 ] 

Hive QA commented on HIVE-16082:




Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12855507/HIVE-16082.1.patch

{color:red}ERROR:{color} -1 due to build exiting with an error

Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/3881/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/3881/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-3881/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Tests exited with: NonZeroExitCodeException
Command 'bash /data/hiveptest/working/scratch/source-prep.sh' failed with exit 
status 1 and output '+ date '+%Y-%m-%d %T.%3N'
2017-03-02 01:45:26.143
+ [[ -n /usr/lib/jvm/java-8-openjdk-amd64 ]]
+ export JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64
+ JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64
+ export 
PATH=/usr/lib/jvm/java-8-openjdk-amd64/bin/:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
+ 
PATH=/usr/lib/jvm/java-8-openjdk-amd64/bin/:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
+ export 'ANT_OPTS=-Xmx1g -XX:MaxPermSize=256m '
+ ANT_OPTS='-Xmx1g -XX:MaxPermSize=256m '
+ export 'MAVEN_OPTS=-Xmx1g '
+ MAVEN_OPTS='-Xmx1g '
+ cd /data/hiveptest/working/
+ tee /data/hiveptest/logs/PreCommit-HIVE-Build-3881/source-prep.txt
+ [[ false == \t\r\u\e ]]
+ mkdir -p maven ivy
+ [[ git = \s\v\n ]]
+ [[ git = \g\i\t ]]
+ [[ -z master ]]
+ [[ -d apache-github-source-source ]]
+ [[ ! -d apache-github-source-source/.git ]]
+ [[ ! -d apache-github-source-source ]]
+ date '+%Y-%m-%d %T.%3N'
2017-03-02 01:45:26.146
+ cd apache-github-source-source
+ git fetch origin
+ git reset --hard HEAD
HEAD is now at ba8de30 HIVE-14459: TestBeeLineDriver - migration and re-enable 
(Peter Vary via Zoltan Haindrich reviewed by Vihang Karajgaonkar)
+ git clean -f -d
Removing 
llap-server/src/java/org/apache/hadoop/hive/llap/io/decode/OrcAcidColumnVectorProducer.java
Removing 
llap-server/src/java/org/apache/hadoop/hive/llap/io/decode/OrcAcidEncodedDataConsumer.java
Removing 
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/OctetLength.java
Removing 
ql/src/java/org/apache/hadoop/hive/ql/udf/GenericUDFCharacterLength.java
Removing ql/src/java/org/apache/hadoop/hive/ql/udf/GenericUDFOctetLength.java
Removing ql/src/test/queries/clientpositive/udf_character_length.q
Removing ql/src/test/queries/clientpositive/udf_octet_length.q
Removing ql/src/test/queries/clientpositive/vector_udf_character_length.q
Removing ql/src/test/queries/clientpositive/vector_udf_octet_length.q
Removing 
ql/src/test/results/clientpositive/llap/dynpart_sort_optimization_acid.q.out.orig
Removing ql/src/test/results/clientpositive/llap/sqlmerge.q.out.orig
Removing ql/src/test/results/clientpositive/udf_character_length.q.out
Removing ql/src/test/results/clientpositive/udf_octet_length.q.out
Removing ql/src/test/results/clientpositive/vector_udf_character_length.q.out
Removing ql/src/test/results/clientpositive/vector_udf_octet_length.q.out
+ git checkout master
Already on 'master'
Your branch is up-to-date with 'origin/master'.
+ git reset --hard origin/master
HEAD is now at ba8de30 HIVE-14459: TestBeeLineDriver - migration and re-enable 
(Peter Vary via Zoltan Haindrich reviewed by Vihang Karajgaonkar)
+ git merge --ff-only origin/master
Already up-to-date.
+ date '+%Y-%m-%d %T.%3N'
2017-03-02 01:45:27.337
+ patchCommandPath=/data/hiveptest/working/scratch/smart-apply-patch.sh
+ patchFilePath=/data/hiveptest/working/scratch/build.patch
+ [[ -f /data/hiveptest/working/scratch/build.patch ]]
+ chmod +x /data/hiveptest/working/scratch/smart-apply-patch.sh
+ /data/hiveptest/working/scratch/smart-apply-patch.sh 
/data/hiveptest/working/scratch/build.patch
Going to apply patch with: patch -p1
patching file common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
patching file 
llap-tez/src/java/org/apache/hadoop/hive/llap/tezplugins/LlapTaskCommunicator.java
+ [[ maven == \m\a\v\e\n ]]
+ rm -rf /data/hiveptest/working/maven/org/apache/hive
+ mvn -B clean install -DskipTests -T 4 -q 
-Dmaven.repo.local=/data/hiveptest/working/maven
ANTLR Parser Generator  Version 3.5.2
Output file 
/data/hiveptest/working/apache-github-source-source/metastore/target/generated-sources/antlr3/org/apache/hadoop/hive/metastore/parser/FilterParser.java
 does not exist: must build 
/data/hiveptest/working/apache-github-source-source/metastore/src/java/org/apache/hadoop/hive/metastore/parser/Filter.g
org/apache/hadoop/hive/metastore/parser/Filter.g
DataNucleus Enhancer (version 4.1.6) for API "JDO"
DataNucleus Enhancer : Classpath
>>  /usr/share/maven/boot/plexus-classworlds-2.x.jar
ENHANCED (Persistable) : org.apache.hadoop.hive.metasto

[jira] [Commented] (HIVE-16082) Allow user to change number of listener thread in LlapTaskCommunicator

2017-03-01 Thread Rajesh Balamohan (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-16082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15891525#comment-15891525
 ] 

Rajesh Balamohan commented on HIVE-16082:
-

lgtm. +1 pending tests.

> Allow user to change number of listener thread in LlapTaskCommunicator
> --
>
> Key: HIVE-16082
> URL: https://issues.apache.org/jira/browse/HIVE-16082
> Project: Hive
>  Issue Type: Bug
>Reporter: Zhiyuan Yang
>Assignee: Zhiyuan Yang
> Attachments: HIVE-16082.1.patch
>
>
> Now LlapTaskCommunicator always has same number of RPC listener thread with 
> TezTaskCommunicatorImpl. There are scenarios when we want them different: for 
> example, in Llap only mode, we want less TezTaskCommunicatorImpl's listener 
> thread to reduce off-heap memory usage.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HIVE-16082) Allow user to change number of listener thread in LlapTaskCommunicator

2017-03-02 Thread Hive QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-16082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15893700#comment-15893700
 ] 

Hive QA commented on HIVE-16082:




Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12855670/HIVE-16082.2.patch

{color:red}ERROR:{color} -1 due to no test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 10 failed/errored test(s), 10285 tests 
executed
*Failed tests:*
{noformat}
TestCommandProcessorFactory - did not produce a TEST-*.xml file (likely timed 
out) (batchId=272)
TestDbTxnManager - did not produce a TEST-*.xml file (likely timed out) 
(batchId=272)
TestDummyTxnManager - did not produce a TEST-*.xml file (likely timed out) 
(batchId=272)
TestHiveInputSplitComparator - did not produce a TEST-*.xml file (likely timed 
out) (batchId=272)
TestIndexType - did not produce a TEST-*.xml file (likely timed out) 
(batchId=272)
TestSplitFilter - did not produce a TEST-*.xml file (likely timed out) 
(batchId=272)
org.apache.hadoop.hive.cli.TestBeeLineDriver.testCliDriver[escape_comments] 
(batchId=229)
org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[schema_evol_text_vec_table]
 (batchId=147)
org.apache.hadoop.hive.cli.TestPerfCliDriver.testCliDriver[query14] 
(batchId=224)
org.apache.hadoop.hive.cli.TestSparkCliDriver.testCliDriver[vector_between_in] 
(batchId=119)
{noformat}

Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/3902/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/3902/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-3902/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 10 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12855670 - PreCommit-HIVE-Build

> Allow user to change number of listener thread in LlapTaskCommunicator
> --
>
> Key: HIVE-16082
> URL: https://issues.apache.org/jira/browse/HIVE-16082
> Project: Hive
>  Issue Type: Bug
>Reporter: Zhiyuan Yang
>Assignee: Zhiyuan Yang
> Attachments: HIVE-16082.1.patch, HIVE-16082.2.patch
>
>
> Now LlapTaskCommunicator always has same number of RPC listener thread with 
> TezTaskCommunicatorImpl. There are scenarios when we want them different: for 
> example, in Llap only mode, we want less TezTaskCommunicatorImpl's listener 
> thread to reduce off-heap memory usage.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HIVE-16082) Allow user to change number of listener thread in LlapTaskCommunicator

2017-03-03 Thread Siddharth Seth (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-16082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15895094#comment-15895094
 ] 

Siddharth Seth commented on HIVE-16082:
---

Test failures are unrelated. Committing.

> Allow user to change number of listener thread in LlapTaskCommunicator
> --
>
> Key: HIVE-16082
> URL: https://issues.apache.org/jira/browse/HIVE-16082
> Project: Hive
>  Issue Type: Bug
>Reporter: Zhiyuan Yang
>Assignee: Zhiyuan Yang
> Attachments: HIVE-16082.1.patch, HIVE-16082.2.patch
>
>
> Now LlapTaskCommunicator always has same number of RPC listener thread with 
> TezTaskCommunicatorImpl. There are scenarios when we want them different: for 
> example, in Llap only mode, we want less TezTaskCommunicatorImpl's listener 
> thread to reduce off-heap memory usage.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (HIVE-16082) Allow user to change number of listener thread in LlapTaskCommunicator

2017-03-05 Thread Lefty Leverenz (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-16082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15896624#comment-15896624
 ] 

Lefty Leverenz commented on HIVE-16082:
---

Doc note:  This adds *hive.llap.task.communicator.listener.thread-count* to 
HiveConf.java, so it needs to be documented in the LLAP section of 
Configuration Properties.

* [Configuration Properties -- LLAP | 
https://cwiki.apache.org/confluence/display/Hive/Configuration+Properties#ConfigurationProperties-LLAP]

Added a TODOC2.2 label.

> Allow user to change number of listener thread in LlapTaskCommunicator
> --
>
> Key: HIVE-16082
> URL: https://issues.apache.org/jira/browse/HIVE-16082
> Project: Hive
>  Issue Type: Bug
>Reporter: Zhiyuan Yang
>Assignee: Zhiyuan Yang
>  Labels: TODOC2.2
> Fix For: 2.2.0
>
> Attachments: HIVE-16082.1.patch, HIVE-16082.2.patch
>
>
> Now LlapTaskCommunicator always has same number of RPC listener thread with 
> TezTaskCommunicatorImpl. There are scenarios when we want them different: for 
> example, in Llap only mode, we want less TezTaskCommunicatorImpl's listener 
> thread to reduce off-heap memory usage.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)