| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
@lovasoa could you pay attention to this change please? This is not just optimization. initialDbSize allows us to merge several tables in a acceptable amount of time. The union is too slow without it. Thank you! |
Sorry, something went wrong.
|
The structure of this repo makes it very difficult to review changes. I am currently not merging pull requests by lack of time. One thing to do that would make it much easier to make changes is to handle compilation to JS by a continuous integration tool, and maybe use a more popular language such as JS of TypeScript. |
Sorry, something went wrong.
|
What if I removed the compiled files from pull request and you compiled them yourself or left the compilation to prepublish command? |
Sorry, something went wrong.
|
@lovasoa OK, I totally agree. Internet is dangerous place. To be sure there are no malware in minified JS: May I ask you to take diff from src/api.coffee only and make dist/* by yourself? |
Sorry, something went wrong.
The fact is we also are tremble because it's hard to understand that dist/* from latest version currently does not contains any suspicious code. |
Sorry, something went wrong.
|
What would both reduce the maintenance burden (which is the biggest problem here) and improve security is having a continuous integration service bundling the files. |
Sorry, something went wrong.
|
@kripken : maybe you can activate GitHub actions for this repo? https://github.com/features/actions |
Sorry, something went wrong.
|
@lovasoa all I see is a way to request being added to the beta - is there something else? I did that now. |
Sorry, something went wrong.
|
No, nothing else. Once you are added to the beta, we can add a yaml file in .github/workflows with the scripts we want to run. |
Sorry, something went wrong.
|
I got an email that I am in the beta, so maybe it can work now... |
Sorry, something went wrong.
|
Thank you Alon. I created a github workflow that compiles the code, tests it, and uploads the resulting artifacts: https://github.com/kripken/sql.js/actions |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
When we have to work with big data, the ability to predefine the size of a new database can significantly speed up script execution time, because multiple calls to MEMF.expandFileStorage is way less efficient than one "bigger" call to it.
JavaScript profile of merging 7 databases (each of size ~15 MB) into one, without expanding the final db file first

The same operation but with expanding the final database to the size of (sum of 7 databases' sizes)
