puhep / pudb

Purdue CMS FPix Database
0 stars 0 forks source link

Consolidated batch submit page crashes but still uploads #191

Closed apatel1995 closed 7 years ago

apatel1995 commented 8 years ago

When uploading to M-B-4-37 I saw that the page would reset on me and then I would have to reload. This only seems to happen with this one module. Despite the connection reset the file does get uploaded, which is not seen unless you view the module page.

screenshot from 2016-07-11 09 46 06

gneeser commented 8 years ago

Is it possible that you were having network issues at the time? Have you tried an upload on that module since then? Were the uploads closely spaced?

I ask because based on how the code is set up, I do expect that occasionally it will crash after most of the data is successfully uploaded, but the reset connection is not something I've seen before, nor is it an error I would expect, so I'm wondering if this is something unrelated to an actual bug in the code as opposed to a network issue.

apatel1995 commented 8 years ago

I tried uploading on different days and on different machines. The same thing happens. The data is completely uploaded then it waits for a server response. The server probably never responds and then the browser just outputs an error.

It only seems to happen to this module. There is a special study being done on this module, so it is undergoing a lot of tests and all of the results are uploaded after every one. Can it be due to the module having a too much data on its page? If so then server is never able to respond before the browser automatically times out.

On Wed, Jul 13, 2016 at 12:13 PM, gneeser notifications@github.com wrote:

Is it possible that you were having network issues at the time? Have you tried an upload on that module since then? Were the uploads closely spaced?

I ask because based on how the code is set up, I do expect that occasionally it will crash after most of the data is successfully uploaded, but the reset connection is not something I've seen before, nor is it an error I would expect, so I'm wondering if this is something unrelated to an actual bug in the code as opposed to a network issue.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/puhep/pudb/issues/191#issuecomment-232422937, or mute the thread https://github.com/notifications/unsubscribe/AJSAdlz1r4VG8OuyWwphy3qEl0UxbZtPks5qVRyjgaJpZM4JJe2W .

gneeser commented 8 years ago

Yes, that would absolutely do it. My comment earlier was in error, I have seen this before. This has happened in the past and my predecessor specifically warned that the database was never designed to handle that amount of information on one module, and that if it was done again, this would happen. The experts at Fermilab were made aware of this at the time.

As we did last time this happened, we can delete test data, but without a major overhaul of how the database handles information there isn't a way around problems occurring from it.

I seem to recall that other problems resulted from this amount of data, but I will have to check my records.

drberry85 commented 8 years ago

Greg (and Jamie),

I think you can just clear out all the old plots for this module, or we can tell the shifters to stop uploading results to lessweb for this guy. Jamie, do you have thoughts?

Doug

On Wed, Jul 13, 2016 at 4:17 PM, gneeser notifications@github.com wrote:

Yes, that would absolutely do it. My comment earlier was in error, I have seen this before. This has happened in the past and my predecessor specifically warned that the database was never designed to handle that amount of information on one module, and that if it was done again, this would happen. The experts at Fermilab were made aware of this at the time.

As we did last time this happened, we can delete test data, but without a major overhaul of how the database handles information there isn't a way around problems occurring from it.

I seem to recall that other problems resulted from this amount of data, but I will have to check my records.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/puhep/pudb/issues/191#issuecomment-232489602, or mute the thread https://github.com/notifications/unsubscribe/AElNLOm3FRM-K9lhsfrzcyrFw4_c2QFVks5qVVXzgaJpZM4JJe2W .

lantone commented 8 years ago

Yeah, I agree, we can just remember to only upload to MoReWeb and not the database in the future. And we can clear out any plots on the module page, they're all available via MoReWeb anyway.

gneeser commented 8 years ago

I'll clear out the plots tomorrow then. I'll let you know if I run into trouble.