Skip to content
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

Do not flush schema on failed authentication #143

Closed
wants to merge 32 commits into from

Conversation

bigbes
Copy link
Contributor

@bigbes bigbes commented Dec 2, 2018

#71

tony2001 and others added 30 commits March 10, 2016 17:52
Features:
* Rewritten update/upsert, so no more big memory overhead
* Now have Exception Tree: TarantoolException -> TarantoolIOException,TarantoolParsingException,TarantoolClientError
  - TarantoolIOExceptions for send/recv/connect problems
  - TarantoolParsingException for parsing length/header/body problems
  - TarantoolClientError for errors happened in Tarantool
* Refactoring for SchemaParsing for Update/Upsert field no.extraction (you can now use field name, instead of field number in update/upsert)
Namespaces:
* Tarantool                 -> Tarantool\Connection
* TarantoolException        -> Tarantool\Exception
* TarantoolIOException      -> Tarantool\Exception\IOException
* TarantoolParsingException -> Tarantool\Exception\ParsingException
* TarantoolClientError      -> Tarantool\Exception\ClientError

* Tarantool            -> Tarantool16
* Tarantool\Connection -> Tarantool\Connection16
'rv' variable is undeclared and second 'return' is not needed here.
Missing in 0.3.0 tag
For example:
```
    local space = box.schema.space.create('test', {
        format = {
            { type = compat.unsigned, name = 'field1', add_field = 1 },
            { type = compat.unsigned, name = 's1'                    },
            { type = compat.string,   name = 's2'                    },
        }
    })
```

field `add_field` causes schema parsing to fail with error
`Failed to parse schema (space)'`

Also added support for Tarantool 1.9+ tests
Fixing problem caused by non-server fields in space `:format()`
….1+)

* Update test-run.py and lib/tarantool_server.py to support python 3
* Update tests accordingly to new version of PHPUnit
* Added initialization of standart class properties to 'Tarantool'
@bigbes bigbes self-assigned this Dec 2, 2018
@bigbes bigbes requested a review from Totktonada December 2, 2018 22:06
@bigbes bigbes closed this Dec 2, 2018
@bigbes
Copy link
Contributor Author

bigbes commented Dec 2, 2018

Sorry, wrong branch

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.

6 participants