Skip to content

Commit 978e9a5

Browse files
author
SivaMaplelabs
committed
Address the pylint warnings
1 parent 691addc commit 978e9a5

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

api/src/main/resources/application_detailed_summary.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ def generate(self):
5151

5252
# waiting block for all the application to get completed
5353
while len(apps_to_be_processed) != 0:
54-
for app_name in apps_to_be_processed.keys():
54+
for app_name in apps_to_be_processed:
5555
try:
5656
apps_to_be_processed[app_name].task.get(STATUS_INTERVAL) #
5757
del apps_to_be_processed[app_name]

api/src/main/resources/plugins/oozie.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ def stage_flink_components(self, application_name, component_name, properties, s
9393
self.exec_cmds(mkdir_commands)
9494

9595
os.system("cp -r %s %s"
96-
% (staged_component_path + '/lib/*', component_install_path))
96+
% (staged_component_path + '/lib/*', component_install_path))
9797

9898
copy_commands = []
9999
copy_commands.append('sudo mv %s/%s %s/execute.sh' % (component_install_path, service_script, component_install_path))
@@ -120,7 +120,7 @@ def destroy_component(self, application_name, create_data):
120120
# stop flink job and delete component from local
121121
if "flink_staged_path" in create_data:
122122
destroy_commands = ["python %s/flink-stop.py" % create_data["flink_staged_path"],
123-
"sudo rm -rf %s\n" % create_data["flink_staged_path"]]
123+
"sudo rm -rf %s\n" % create_data["flink_staged_path"]]
124124
self.exec_cmds(destroy_commands)
125125

126126
def start_component(self, application_name, create_data):
@@ -160,7 +160,7 @@ def create_component(self, staged_component_path, application_name, user_name, c
160160
properties['deployment_end'] = end.strftime("%Y-%m-%dT%H:%MZ")
161161

162162
# for flink jobs, code need to be staged locally because both ssh action and flink client requires code to be present in local
163-
if properties.get('component_job_type','') == 'flink':
163+
if properties.get('component_job_type', '') == 'flink':
164164
self.stage_flink_components(application_name, component['component_name'], properties, staged_component_path)
165165

166166
# insert required oozie properties
@@ -199,8 +199,8 @@ def create_component(self, staged_component_path, application_name, user_name, c
199199
ret_data["flink_staged_path"] = properties["component_staged_path"]
200200

201201
ret_data.update({'job_handle': undeploy['id'],
202-
'component_hdfs_root': properties['component_hdfs_root'],
203-
'application_user': properties['application_user']})
202+
'component_hdfs_root': properties['component_hdfs_root'],
203+
'application_user': properties['application_user']})
204204
return ret_data
205205

206206
def _setup_queue_config(self, component, staged_component_path, properties):

0 commit comments

Comments
 (0)