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

Lefty Leverenz commented on HIVE-16336:
---------------------------------------

Please force new lines with \n in the parameter description.  Also, you don't 
need to indent the description.  And the "+" usually goes at the end of the 
line.

In other words, don't follow the example of hive.spark.use.op.stats in 
HiveConf.java, but look at hive.llap.hs2.coordinator.enabled a few parameters 
higher up in the file.

> Rename hive.spark.use.file.size.for.mapjoin to 
> hive.spark.use.ts.stats.for.mapjoin
> ----------------------------------------------------------------------------------
>
>                 Key: HIVE-16336
>                 URL: https://issues.apache.org/jira/browse/HIVE-16336
>             Project: Hive
>          Issue Type: Bug
>          Components: Configuration
>            Reporter: Chao Sun
>            Assignee: Chao Sun
>         Attachments: HIVE-16336.0.patch
>
>
> The name {{hive.spark.use.file.size.for.mapjoin}} is confusing. It indicates 
> that HoS uses file size for mapjoin but in fact it still uses (in-memory) 
> data size. We should change it.



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

Reply via email to