Commit 1f99e28f by noraiz-anwar

remove logs added under EDUCATOR-1652

parent 33335f7a
...@@ -468,9 +468,7 @@ def _accessible_courses_list_from_groups(request): ...@@ -468,9 +468,7 @@ def _accessible_courses_list_from_groups(request):
course_keys = course_keys.values() course_keys = course_keys.values()
if course_keys: if course_keys:
start_time = time.time()
courses_list = modulestore().get_course_summaries(course_keys=course_keys) courses_list = modulestore().get_course_summaries(course_keys=course_keys)
log.info('[%d] Course summaries fetched in [%f]', len(courses_list), (time.time() - start_time))
return courses_list, [] return courses_list, []
...@@ -497,26 +495,14 @@ def course_listing(request): ...@@ -497,26 +495,14 @@ def course_listing(request):
""" """
List all courses available to the logged in user List all courses available to the logged in user
""" """
def _execute_method_and_log_time(func, *args):
"""
Call func passed in method with logging the time it took to complete.
Logging is temporary, we will remove this once we get required information.
"""
start_time = time.time()
output = func(*args)
log.info('[%s] completed in [%f]', func.__name__, (time.time() - start_time))
return output
optimization_enabled = GlobalStaff().has_user(request.user) and \ optimization_enabled = GlobalStaff().has_user(request.user) and \
WaffleSwitchNamespace(name=WAFFLE_NAMESPACE).is_enabled(u'enable_global_staff_optimization') WaffleSwitchNamespace(name=WAFFLE_NAMESPACE).is_enabled(u'enable_global_staff_optimization')
org = request.GET.get('org', '') if optimization_enabled else None org = request.GET.get('org', '') if optimization_enabled else None
courses_iter, in_process_course_actions = _execute_method_and_log_time(get_courses_accessible_to_user, request, org) courses_iter, in_process_course_actions = get_courses_accessible_to_user(request, org)
user = request.user user = request.user
libraries = _accessible_libraries_iter(request.user, org) if LIBRARIES_ENABLED else []
libraries = []
if LIBRARIES_ENABLED:
libraries = _execute_method_and_log_time(_accessible_libraries_iter, request.user, org)
def format_in_process_course_view(uca): def format_in_process_course_view(uca):
""" """
...@@ -553,33 +539,24 @@ def course_listing(request): ...@@ -553,33 +539,24 @@ def course_listing(request):
} }
split_archived = settings.FEATURES.get(u'ENABLE_SEPARATE_ARCHIVED_COURSES', False) split_archived = settings.FEATURES.get(u'ENABLE_SEPARATE_ARCHIVED_COURSES', False)
active_courses, archived_courses = _execute_method_and_log_time( active_courses, archived_courses = _process_courses_list(courses_iter, in_process_course_actions, split_archived)
_process_courses_list,
courses_iter,
in_process_course_actions,
split_archived
)
in_process_course_actions = [format_in_process_course_view(uca) for uca in in_process_course_actions] in_process_course_actions = [format_in_process_course_view(uca) for uca in in_process_course_actions]
response = _execute_method_and_log_time( return render_to_response(u'index.html', {
render_to_response, u'courses': active_courses,
u'index.html', u'archived_courses': archived_courses,
{ u'in_process_course_actions': in_process_course_actions,
u'courses': active_courses, u'libraries_enabled': LIBRARIES_ENABLED,
u'archived_courses': archived_courses, u'libraries': [format_library_for_view(lib) for lib in libraries],
u'in_process_course_actions': in_process_course_actions, u'show_new_library_button': get_library_creator_status(user),
u'libraries_enabled': LIBRARIES_ENABLED, u'user': user,
u'libraries': [format_library_for_view(lib) for lib in libraries], u'request_course_creator_url': reverse('request_course_creator'),
u'show_new_library_button': get_library_creator_status(user), u'course_creator_status': _get_course_creator_status(user),
u'user': user, u'rerun_creator_status': GlobalStaff().has_user(user),
u'request_course_creator_url': reverse('request_course_creator'), u'allow_unicode_course_id': settings.FEATURES.get(u'ALLOW_UNICODE_COURSE_ID', False),
u'course_creator_status': _get_course_creator_status(user), u'allow_course_reruns': settings.FEATURES.get(u'ALLOW_COURSE_RERUNS', True),
u'rerun_creator_status': GlobalStaff().has_user(user), u'optimization_enabled': optimization_enabled
u'allow_unicode_course_id': settings.FEATURES.get(u'ALLOW_UNICODE_COURSE_ID', False), })
u'allow_course_reruns': settings.FEATURES.get(u'ALLOW_COURSE_RERUNS', True),
u'optimization_enabled': optimization_enabled
}
)
return response return response
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment