diff options
author | 2019-10-31 23:51:29 +0000 | |
---|---|---|
committer | 2019-11-04 14:02:09 +0000 | |
commit | 68d118176bb53c36b31a7060cfa16ad5acac1765 (patch) | |
tree | c6dfd74445126f24e8d63457e33720fc5e0d38bb /vnfs/DAaaS/microservices/GoApps/src/go-hdfs-writer/pkg/utils/types.go | |
parent | 68042495ef1e1e6dff7fed7fc2691b01cf672fe1 (diff) |
HDFS-WriterApp-Fixed all the code review comments
Fixed all the code review comments by Kiran
1. Implemented a boolean channel instead of empty struct channel for signal, use WaitGroup
to ensure all writers finish cleanup.
2. Introduce JSON tags for configs
4. remove all panic and fatalf code to ensure that the app doesnt crash anytime.
5. remove unneccessary hdfsWriter null checks.
6.remove the 'run' variable used in the infinite loop, replaced with 'return'
Issue-ID: ONAPARC-453
Change-Id: Ic77c59dc75a8898a3cf34999850e6687d40e7faa
Signed-off-by: Rajamohan Raj <rajamohan.raj@intel.com>
Diffstat (limited to 'vnfs/DAaaS/microservices/GoApps/src/go-hdfs-writer/pkg/utils/types.go')
-rw-r--r-- | vnfs/DAaaS/microservices/GoApps/src/go-hdfs-writer/pkg/utils/types.go | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/vnfs/DAaaS/microservices/GoApps/src/go-hdfs-writer/pkg/utils/types.go b/vnfs/DAaaS/microservices/GoApps/src/go-hdfs-writer/pkg/utils/types.go new file mode 100644 index 00000000..3db3e420 --- /dev/null +++ b/vnfs/DAaaS/microservices/GoApps/src/go-hdfs-writer/pkg/utils/types.go @@ -0,0 +1,19 @@ +package utils + +// Pipeline type represents a stucture of a general pipeline +type Pipeline struct{ + KafkaConfiguration KafkaConfig `json:"kafkaConfig"` + HdfsConfiguration HdfsConfig `json:"hdfsConfig"` +} + +// HdfsConfig type represents the config items of HDFS +type HdfsConfig struct { + HdfsURL string `json:"hdfs_url"` +} + +// KafkaConfig type represents the config items of Kafka +type KafkaConfig struct { + Broker string `json:"broker"` + Group string `json:"group"` + Topic string `json:"topic"` +}
\ No newline at end of file |