[
https://issues.apache.org/jira/browse/HBASE-17728?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15901854#comment-15901854
]
Enis Soztutar commented on HBASE-17728:
---------------------------------------
bq. It seems the declaration of string constants in the code base isn't
consistent.
Indeed. We have started using Google style guides after there was already some
code in, and we did not have the chance to clean up existing usage yet.
bq. On initial survey, I didn't find the clear advantage of using constexpr. If
you can explain some more, that would be nice.
Fair enough. We are following Google's style guidelines and cpplint scripts in
the code base. For example, naming the constants are like this:
https://google.github.io/styleguide/cppguide.html#Constant_Names. The global
static declarations are following this:
https://google.github.io/styleguide/cppguide.html#Static_and_Global_Variables.
I should have been more explicit about the naming as well in my initial
comment. We should name the static expr as {{kKerberos}}. Please make the
change and commit afterwards.
Also, capitalize {{isSecurityEnabled}} to {{IsSecurityEnabled}}
> Create separate build target for Configuration classes
> ------------------------------------------------------
>
> Key: HBASE-17728
> URL: https://issues.apache.org/jira/browse/HBASE-17728
> Project: HBase
> Issue Type: Sub-task
> Reporter: Ted Yu
> Assignee: Ted Yu
> Attachments: 17728.v1.txt, 17728.v2.txt, 17728.v3.txt, 17728.v4.txt,
> 17728.v5.txt, 17728.v6.txt, 17728.v7.txt, 17728.v8.txt, 17728.v9.txt
>
>
> User is in security module.
> When User::isSecurityEnabled() is added, we need to query Configuration for
> security setting.
> However, this introduces a circular build dependency:
> BUILD FAILED: Cycle found: //connection:connection -> //security:security ->
> //core:core -> //connection:connection
> This issue is to create separate build target for Configuration which is
> depended upon by both core and security modules.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
