-
Notifications
You must be signed in to change notification settings - Fork 12
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
Merge Release #3615
Open
github-actions
wants to merge
42
commits into
master
Choose a base branch
from
release
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Merge Release #3615
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
* restructures ResultType and decouples printing into ResultPrinters: - ResultType is now very simple mix of primitive enum and LIST. (We hope to remove even more logic from it) - Result printing was always weird with Mappings embedded in StringT. We've moved that responsibility into ResultInfo via SelectResultInfo for mapped Selects. - This PR opens the possibility to embed anonymization into the ResultInfo. - PrintSettings now caches the C10n calls as they were before that sometimes invoked per cell, which should make rendering faster. * Reworks Printers: - We inject the result specific Printers using the printerFactory. This untangles the burden of formatting. - Now a Select claiming to produce a specific type only needs to guarantee they are in a format capable of being handled by the printers it elects to use: This allows us later to make a printer that alters the result before printing it, without having to inject into all renderers. * reworks Money to no longer be "cent" based and instead just use it as storage primitive. Is now BigDecimal - pull default for decimalShift from CurrencyConfig
…hierarchy-2 Fix search for multi struct node hierarchy
Update non-critical minor versions for non-tooling dependencies
…etting-manually Fix defaultParams resetting also when clicking reset in history navigation
Update vite, storybook, apache-arrow dependencies, update stories
#3586) * test if query is initialized before building full-status to avoid issues * Cleanup query overview in admin-ui * Shorten toString of MultiSelect FilterValue
#3580) * submit ForwardToNamespace messages to the job manager of the namespace * unify usage of NetworkSession in NetworkMessageContext --------- Co-authored-by: awildturtok <[email protected]>
remove CentralRegistry
improve validation message for form
Fixes faulty calculation of width in TableExportQueryConverter by using unified method from TableExportQuery#calculateWidth --------- Co-authored-by: Jonas Arnhold <[email protected]>
Try and fix logo url
Try and fix search again for multi-nested struct nodes
Rework calculateSpecificChildrenPaths: split into mode with and without column, simplify both along the distinction. This fixes a bug where values are faultily resolved to the root.
fix resolution of MoneyRange, move it into FilterValue --------- Co-authored-by: MT <[email protected]>
fix render error on admin ui imports
Reintegrate Main
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.