Next/major #6
Closed
Next/major #6
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This allows aggregator implementations to be swapped and added more easily.
This is a breaking change to the contract of `IQuadDestination`. Related to comunica#1301
This refactoring is a preparation for splitting all functions to separate actors.
…sion-feature-merge-major Chore/busify expression feature merge major
* first part of the extraction script * update script to work with abs (spec tested) and see that it fails * functioning abs extracted operator * abs with updated package.json * package level imports for regular functions * nothing significant? * better regularFunction constructors * better extraction script * sneak peak on regular operators * extract function in script * extract function in script * remove special function indicator * remove special threatment of named functions * cleanup after removing named & special * linter fix * generalize operations to ready script * generalize operations to ready script * remove extarction script * small patch
…essions Merge busification branch with next/major
Merge master into next/major
…essions Merge next/major into feature/busify-expressions
Promise rejections rely on the creation of Error objects, which are expensive to create in large numbers due to stack trace building. Since Comunica tends to invoke a huge number of test methods for an average query exection, this can cause a large performance overhead. This commit refactors test methods to asynchronously return a TestResult, which is a holder object that either contains a passed value or a failure message.
All buses now have an error message template, which will be used when reporting mediator failures. Closes comunica#119
…essions Feature/busify expressions
This slightly improves performance
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.