[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127225#comment-16127225 ] Peter Vary commented on HIVE-17291: --- Ok. So based on this discussion we should leave the logic which

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126744#comment-16126744 ] Xuefu Zhang commented on HIVE-17291: [~lirui], that makes sense. > Set the number of executors based

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126738#comment-16126738 ] Rui Li commented on HIVE-17291: --- [~pvary], bq. Logic suggests, that in this case we will request new

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126284#comment-16126284 ] Xuefu Zhang commented on HIVE-17291: Did you confirm that the configuration is updated and new session

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126257#comment-16126257 ] Peter Vary commented on HIVE-17291: --- For testing this is exactly what I did in HIVE-17292.3.patch. I am

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126242#comment-16126242 ] Xuefu Zhang commented on HIVE-17291: Getting a value greater than 0 doesn't necessarily mean that all

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126224#comment-16126224 ] Peter Vary commented on HIVE-17291: --- To clarify: I see the flakiness appear only when running the HoS

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126125#comment-16126125 ] Xuefu Zhang commented on HIVE-17291: For test result stability, I'm fine with whatever solution. It

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16125842#comment-16125842 ] Peter Vary commented on HIVE-17291: --- Thanks [~lirui], helpful as always! What I do not get, why we do

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16125508#comment-16125508 ] Rui Li commented on HIVE-17291: --- [~pvary], thanks so much for working in the middle of the night - I'll make

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16125439#comment-16125439 ] Peter Vary commented on HIVE-17291: --- [~lirui]: You might be using black magic, or similar :) In the

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16125137#comment-16125137 ] Rui Li commented on HIVE-17291: --- Is this just to avoid unstable test output? If so, it seems we're doing

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123835#comment-16123835 ] Xuefu Zhang commented on HIVE-17291: You're right. Thanks for the explanation. +1 to the patch. >

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123826#comment-16123826 ] Peter Vary commented on HIVE-17291: --- Hi [~xuefuz], I might not see the whole picture here, but my

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123749#comment-16123749 ] Xuefu Zhang commented on HIVE-17291: Thanks for working on this, [~pvary]. The patch looks good.

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123370#comment-16123370 ] Peter Vary commented on HIVE-17291: --- Failures are not related. [~lirui], [~xuefuz], could you please

[jira] [Commented] (HIVE-17291) Set the number of executors based on config if client does not provide information

[ https://issues.apache.org/jira/browse/HIVE-17291?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122583#comment-16122583 ] Hive QA commented on HIVE-17291: Here are the results of testing the latest attachment: