Skip to content

Commit b759f96

Browse files
author
RolefH
committed
avoid tracing "null" when default app config is used
1 parent a589b0c commit b759f96

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

com.ibm.streamsx.messagehub/impl/java/src/com/ibm/streamsx/messagehub/operators/MessageHubConsumerOperator.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,13 +88,14 @@ protected void loadProperties() throws Exception {
8888

8989
@Override
9090
protected void loadFromAppConfig() throws Exception {
91-
final KafkaOperatorProperties appCfgProps = MessageHubOperatorUtil.loadMessageHubCredsFromAppConfig(getOperatorContext(), appConfigName);
91+
final String appCfgName = appConfigName == null? MessageHubOperatorUtil.DEFAULT_MESSAGE_HUB_APP_CONFIG_NAME: appConfigName;
92+
final KafkaOperatorProperties appCfgProps = MessageHubOperatorUtil.loadMessageHubCredsFromAppConfig(getOperatorContext(), appCfgName);
9293
if (appConfigRequired && (appCfgProps == null || appCfgProps.isEmpty())) {
9394
final String msg = "Message Hub credentials not found in properties file nor in an Application Configuration";
9495
logger.error(msg);
9596
throw new RuntimeException(msg);
9697
}
97-
logger.info ("kafka properties derived from App Config " + appConfigName + " ("
98+
logger.info ("kafka properties derived from App Config " + appCfgName + " ("
9899
+ MessageHubOperatorUtil.DEFAULT_MESSAGE_HUB_CREDS_PROPERTY_NAME + "): "
99100
+ appCfgProps.keySet());
100101

com.ibm.streamsx.messagehub/impl/java/src/com/ibm/streamsx/messagehub/operators/MessageHubProducerOperator.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,13 +59,14 @@ protected void loadProperties() throws Exception {
5959

6060
@Override
6161
protected void loadFromAppConfig() throws Exception {
62-
final KafkaOperatorProperties appCfgProps = MessageHubOperatorUtil.loadMessageHubCredsFromAppConfig(getOperatorContext(), appConfigName);
62+
final String appCfgName = appConfigName == null? MessageHubOperatorUtil.DEFAULT_MESSAGE_HUB_APP_CONFIG_NAME: appConfigName;
63+
final KafkaOperatorProperties appCfgProps = MessageHubOperatorUtil.loadMessageHubCredsFromAppConfig(getOperatorContext(), appCfgName);
6364
if (appConfigRequired && (appCfgProps == null || appCfgProps.isEmpty())) {
6465
final String msg = "Message Hub credentials not found in properties file nor in an Application Configuration";
6566
logger.error(msg);
6667
throw new RuntimeException(msg);
6768
}
68-
logger.info ("kafka properties derived from App Config " + appConfigName + " ("
69+
logger.info ("kafka properties derived from App Config " + appCfgName + " ("
6970
+ MessageHubOperatorUtil.DEFAULT_MESSAGE_HUB_CREDS_PROPERTY_NAME + "): "
7071
+ appCfgProps.keySet());
7172
// When we are here, we might have read the properties derived from a JSON credentials file. In this case 'appConfigRequired' is false.

0 commit comments

Comments
 (0)