Update docker - #1
Open
cawallin wants to merge 2061 commits into
Open
Conversation
Refactor the method so that the error code and error message are passed in as argument.
- The arguments to GROUPING() are not evaluated. The function returns a bitmap as an integer where the rightmost column in the GROUPING() argument list is the least significant bit and is set to 1 when that column is not present in the grouping and to 0 otherwise. - Ensure each argument to GROUPING() is a grouping column (i.e. a column in GROUP BY, GROUPING SETS, ROLLUP or CUBE). - Parse GROUPING() to it's own AST node and rewrite it to a FunctionCall expression in QueryPlanner. - Depending on the number of arguments to GROUPING() pick the appropriate implementing function that either returns an integer or a bigint. - Rewrite GROUPING() to a 0 literal when used in a GROUP BY. - Disallow GROUPING() in a WHERE clause, in a JOIN condition, a GROUP BY statement and in a LAMBDA expression.
Add a new unit test for PruneValuesColumns. Extend ValuesMatcher to optionally validate rows, and adjust the PlanMatchPattern.values() functions accordingly. The previous approach was odd, in that there was a ValuesMatcher for each alias to be created; it's more direct to implement Matcher and create the aliases in detailMatches().
Since the flag is turned on by default, it should not be named "experimental".
The feature toggle is plural and so the flag should be.
Recursive computeIfAbsent (that modifies the same collection) may corrupt HashMap, see: http://bugs.java.com/bugdatabase/view_bug.do?bug_id=8172951
Since unreflect calls caused GC issues recently unifying their callsites will help us better track their usage.
Includes a PlanBuilder.tableScan() overload that provides the TableHandle. That overload is lifted from not-yet-merged 9cbe62c, which implemnents a PushDownTableConstraints rule. The change is identical, so later merging or rebasing should go smoothly.
distinctSymbols field allows for verification that at the end of planning DistinctLimitNode is provided with exactly required symbols from child node. Additionally, distinctSymbols field allows for symbols pruning in child nodes.
A cell in a table can be huge. A worker can OOM when reading too many large cells. Export block size to monitor the distribution of loaded blocks.
ConcurrentHashMap is unnecessary as all accessor methods are synchronized
Previously following scenario would fail: CREATE TABLE memory.default.test (a BIGINT); INSERT INTO memory.default.test SELECT nationkey FROM tpch.tiny.nation; with an error message, that table test was not found on a worker. It is fixed by allowing to "initialize" table on writes (not as it was before only on creates) and by moving sanity checks for detecting worker crashes to asserting expected number of rows per worker.
Add support for creating custom schemas and for handling tables from different schemas.
Must be reverted and replaced with the proper solution after JDBC driver gets the support for specifying so.
Use BasicAuthenticationFilter from Airlift instead
Add hidden -P presto-cli parameter that allows to specify password in non-interactive way. The parameter is hidden and supposed to be used for testing purposes only.
Introduce LDAP authentication to TLS secured communication product tests
Ensure that we have all the prorpties set for Hive connector when using Kerberos to access Hive metastore. Verify kerberos properties for Presto server authentication.
Add a section in CLI documentation that describes various options that can be used.
This adds the basic product-tests profile with TLS enabled internal communication. The kerberos related configs will be added in a separate commit.
We're running out of space in containers, which is fixed in subsequent versions of Docker.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
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.