Added more logging for submissions, to get timing information.
- Legacy-Id: 18635
This commit is contained in:
parent
72c7018073
commit
45a808db95
|
@ -7,6 +7,7 @@ import io
|
||||||
import os
|
import os
|
||||||
import pathlib
|
import pathlib
|
||||||
import re
|
import re
|
||||||
|
import time
|
||||||
|
|
||||||
from typing import Callable, Optional # pyflakes:ignore
|
from typing import Callable, Optional # pyflakes:ignore
|
||||||
|
|
||||||
|
@ -671,6 +672,7 @@ def save_files(form):
|
||||||
with io.open(name, 'wb+') as destination:
|
with io.open(name, 'wb+') as destination:
|
||||||
for chunk in f.chunks():
|
for chunk in f.chunks():
|
||||||
destination.write(chunk)
|
destination.write(chunk)
|
||||||
|
log("saved file %s" % name)
|
||||||
return file_name
|
return file_name
|
||||||
|
|
||||||
def get_draft_meta(form, saved_files):
|
def get_draft_meta(form, saved_files):
|
||||||
|
@ -786,6 +788,7 @@ def apply_checkers(submission, file_name):
|
||||||
symbol=checker.symbol)
|
symbol=checker.symbol)
|
||||||
check.save()
|
check.save()
|
||||||
|
|
||||||
|
mark = time.time()
|
||||||
for checker_path in settings.IDSUBMIT_CHECKER_CLASSES:
|
for checker_path in settings.IDSUBMIT_CHECKER_CLASSES:
|
||||||
checker_class = import_string(checker_path)
|
checker_class = import_string(checker_path)
|
||||||
checker = checker_class()
|
checker = checker_class()
|
||||||
|
@ -795,6 +798,8 @@ def apply_checkers(submission, file_name):
|
||||||
if hasattr(checker, method) and ext in file_name:
|
if hasattr(checker, method) and ext in file_name:
|
||||||
apply_check(submission, checker, method, file_name[ext])
|
apply_check(submission, checker, method, file_name[ext])
|
||||||
break
|
break
|
||||||
|
tau = time.time() - mark
|
||||||
|
log("ran submission checks (%.3fs) for %s" % (tau, file_name))
|
||||||
|
|
||||||
def send_confirmation_emails(request, submission, requires_group_approval, requires_prev_authors_approval):
|
def send_confirmation_emails(request, submission, requires_group_approval, requires_prev_authors_approval):
|
||||||
docevent_from_submission(request, submission, desc="Uploaded new revision")
|
docevent_from_submission(request, submission, desc="Uploaded new revision")
|
||||||
|
|
|
@ -50,6 +50,7 @@ def upload_submission(request):
|
||||||
try:
|
try:
|
||||||
form = SubmissionManualUploadForm(request, data=request.POST, files=request.FILES)
|
form = SubmissionManualUploadForm(request, data=request.POST, files=request.FILES)
|
||||||
if form.is_valid():
|
if form.is_valid():
|
||||||
|
log('got valid submission form for %s' % form.filename)
|
||||||
saved_files = save_files(form)
|
saved_files = save_files(form)
|
||||||
authors, abstract, file_name, file_size = get_draft_meta(form, saved_files)
|
authors, abstract, file_name, file_size = get_draft_meta(form, saved_files)
|
||||||
|
|
||||||
|
@ -97,7 +98,6 @@ def upload_submission(request):
|
||||||
form._errors["__all__"] = form.error_class(["There was a failure processing your upload -- please verify that your draft passes idnits. (%s)" % e.message])
|
form._errors["__all__"] = form.error_class(["There was a failure processing your upload -- please verify that your draft passes idnits. (%s)" % e.message])
|
||||||
if debug.debug:
|
if debug.debug:
|
||||||
raise
|
raise
|
||||||
|
|
||||||
else:
|
else:
|
||||||
form = SubmissionManualUploadForm(request=request)
|
form = SubmissionManualUploadForm(request=request)
|
||||||
|
|
||||||
|
@ -119,6 +119,7 @@ def api_submit(request):
|
||||||
try:
|
try:
|
||||||
form = SubmissionAutoUploadForm(request, data=request.POST, files=request.FILES)
|
form = SubmissionAutoUploadForm(request, data=request.POST, files=request.FILES)
|
||||||
if form.is_valid():
|
if form.is_valid():
|
||||||
|
log('got valid submission form for %s' % form.filename)
|
||||||
username = form.cleaned_data['user']
|
username = form.cleaned_data['user']
|
||||||
user = User.objects.filter(username=username)
|
user = User.objects.filter(username=username)
|
||||||
if user.count() == 0:
|
if user.count() == 0:
|
||||||
|
|
Loading…
Reference in a new issue