Mailing-List: contact issues-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
Date: Thu, 7 Jul 2016 13:11:10 +0000 (UTC)
From: "Tzu-Li (Gordon) Tai (JIRA)" <jira@apache.org>
To: issues@flink.apache.org
Message-ID: <JIRA.12987473.1467896709000.47852.1467897070955@Atlassian.JIRA>
In-Reply-To: <JIRA.12987473.1467896709000@Atlassian.JIRA>
References: <JIRA.12987473.1467896709000@Atlassian.JIRA> <JIRA.12987473.1467896709202@arcas>
Subject: [jira] [Comment Edited] (FLINK-4170) Remove `CONFIG_` prefix from
 KinesisConfigConstants variables
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Thu, 07 Jul 2016 13:11:12 -0000


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

Tzu-Li (Gordon) Tai edited comment on FLINK-4170 at 7/7/16 1:10 PM:
--------------------------------------------------------------------

[~uce] I had this in mind too, especially the recent additional configs in the consumer are really really long :P
+1 from me :)


was (Author: tzulitai):
I had this in mind too, especially the recent additional configs in the consumer are really really long :P
+1 from me :)

> Remove `CONFIG_` prefix from KinesisConfigConstants variables
> -------------------------------------------------------------
>
>                 Key: FLINK-4170
>                 URL: https://issues.apache.org/jira/browse/FLINK-4170
>             Project: Flink
>          Issue Type: Improvement
>          Components: Streaming Connectors
>            Reporter: Ufuk Celebi
>             Fix For: 1.1.0
>
>
> I find the static variable names verbose. I think it's clear from context that they refer to the Kinesis configuration since they are all gathered in that class.
> Therefore would like to remove the {{CONFIG_}} prefix before the release, so that we have
> {code}
> conf.put(KinesisConfigConstants.AWS_REGION, "")
> {code}
> instead of 
> {code}
> conf.put(KinesisConfigConstants.CONFIG_AWS_REGION, "")
> {code}
> For longer variables it becomes even longer otherwise.
> ---
> Some basic variable names that might be accessed frequently are also very long:
> {code}
> CONFIG_AWS_CREDENTIALS_PROVIDER_BASIC_SECRETKEY
> CONFIG_AWS_CREDENTIALS_PROVIDER_BASIC_ACCESSKEYID
> {code}
> It might suffice to just have:
> {code}
> AWS_SECRET_KEY
> AWS_ACCESS_KEY
> {code}


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)