Skip to content

Conversation

@GauravJayaraj
Copy link

Minor:

Cycle field expression allows path as well to access nested feilds( in 7.6.0)
A new error is added in 7.62, to disallow any other expression, for eg: 1+1(an arithmetic expression), in 7.6.0 we just ignore such examples:

 WITH RECURSIVE cyc AS ( SELECT 0 as _from, 1 as _to, 0 as lvl UNION SELECT n._from, n._to, c.lvl+1 as lvl FROM `shellTest`._default.cycleTest n JOIN cyc c ON c._to = n._from) CYCLE _from, _to, 1+1 RESTRICT SELECT cyc.* FROM cyc;
{
    "requestID": "5aa50b56-8fe4-41ad-a395-0a984e181c79",
    "errors": [
        {
            "code": 3307,
            "msg": "Cycle fields validation failed for with term: cyc - cause: invalid cycle field expression term: (1 + 1) only identifier/path expressions are allowed"
        }
    ],
    "status": "fatal",
    "metrics": {
        "elapsedTime": "7.668208ms",
        "executionTime": "7.335208ms",
        "resultCount": 0,
        "resultSize": 0,
        "serviceLoad": 2,
        "errorCount": 1
    }
}

@julia-cb
Copy link
Contributor

@GauravJayaraj hey Gaurav, feel free to close this PR since I already made the changes to the page :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants