[
https://issues.apache.org/jira/browse/TAJO-1968?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15292590#comment-15292590
]
ASF GitHub Bot commented on TAJO-1968:
--------------------------------------
Github user jihoonson commented on a diff in the pull request:
https://github.com/apache/tajo/pull/1028#discussion_r63984511
--- Diff: tajo-common/src/main/java/org/apache/tajo/SessionVars.java ---
@@ -145,7 +145,7 @@
// for index
INDEX_ENABLED(ConfVars.$INDEX_ENABLED, "index scan enabled", DEFAULT),
- INDEX_SELECTIVITY_THRESHOLD(ConfVars.$INDEX_SELECTIVITY_THRESHOLD, "the
selectivity threshold for index scan",
+ INDEX_SELECTIVITY_LIMIT(ConfVars.$INDEX_SELECTIVITY_LIMIT, "the
selectivity limit for index scan",
--- End diff --
Why did you change the description as well?
```threshold``` is more appropriate because it has a narrow meaning than
```limit```.
> Change 'threshold' in the name of configuration and session variables to
> 'limit'
> --------------------------------------------------------------------------------
>
> Key: TAJO-1968
> URL: https://issues.apache.org/jira/browse/TAJO-1968
> Project: Tajo
> Issue Type: Task
> Components: conf and scripts
> Reporter: Jihoon Son
> Assignee: Byunghoon Lim
> Priority: Minor
> Labels: newbie
> Fix For: 0.12.0
>
>
> See the discussion at
> https://github.com/apache/tajo/pull/848#discussion_r44101976.
> We need to make the configuration names consistent.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
