Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace disk approach with TCP/IP for data exchange. #48

Closed
FellippeHeitor opened this issue May 18, 2018 · 5 comments
Closed

Replace disk approach with TCP/IP for data exchange. #48

FellippeHeitor opened this issue May 18, 2018 · 5 comments
Assignees
Milestone

Comments

@FellippeHeitor
Copy link
Owner

No description provided.

@callowaysutton
Copy link

Wait wait wait, What is this?...

@FellippeHeitor
Copy link
Owner Author

FellippeHeitor commented May 19, 2018

InForm's editor and the Preview form are two separate applications running and exchanging data via a disk file. The Preview is constantly sending the form data back to the editor and the editor sends edits over to the Preview by writing to disk. I intend to replace that paradigm with a local TCP/IP connection between both. Under tests locally already ;-)

@callowaysutton
Copy link

Woahhhh that's really smart, what do you think the benefits would be?

@FellippeHeitor
Copy link
Owner Author

FellippeHeitor commented May 22, 2018

Just switching to a more modern approach is already a benefit, but I envision you could load more than a form at once, for one thing (which would make sense considering the planned enhancement described in #20).

I also imagine that the burden on the disk is going to be magnitudes smaller this way, of course - especially considering the many reads/writes per minute that the editor currently has to perform. The first versions of InForm had issues with flushing to disk on Linux machines, which I had to circumvent by opening and closing the exchange files multiple times a second.

Now I will consider this switch a success if the end-user ends up not noticing that I've changed it at all. The idea is to make it as seamless as possible.

@FellippeHeitor FellippeHeitor added this to the Beta 8 milestone May 28, 2018
@FellippeHeitor
Copy link
Owner Author

Effective for Beta 8 starting with 420c0f6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants