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

Remove IF NOT EXISTS from create database command #5707

Closed
jsternberg opened this issue Feb 16, 2016 · 1 comment
Closed

Remove IF NOT EXISTS from create database command #5707

jsternberg opened this issue Feb 16, 2016 · 1 comment
Assignees
Milestone

Comments

@jsternberg
Copy link
Contributor

The IF NOT EXISTS syntax for CREATE DATABASE is now the default and only behavior. This ticket is to remove the syntax from the parser.

The earliest this should be done is 0.12. The syntax will be ignored in 0.10 and 0.11.

Related to #5563.

@toddboom
Copy link
Contributor

For 'v0.13.0', this should log a warning message telling the user that IF NOT EXISTS queries are now default and will be removed in v1.0.0.

Also, we should make sure that any other queries used to support IF NOT EXISTS have this same warning.

jsternberg added a commit that referenced this issue Apr 13, 2016
The deprecated message is now attached to a new attribute returned with
the results. This message can then be read by clients to warn a user
about upcoming changes to the query engine.

The `influx` client has already been modified to read this message and
print it out for every format except CSV.

The first warning message is a deprecated message about removing `IF NOT
EXISTS` from `CREATE DATABASE`.

Fixes #5707.
jsternberg added a commit that referenced this issue Apr 13, 2016
The deprecated message is now attached to a new attribute returned with
the results. This message can then be read by clients to warn a user
about upcoming changes to the query engine.

The `influx` client has already been modified to read this message and
print it out for every format except CSV.

The first warning message is a deprecated message about removing `IF NOT
EXISTS` from `CREATE DATABASE`.

The message will also be printed to the server log.

Fixes #5707.
jsternberg added a commit that referenced this issue Apr 15, 2016
The deprecated message is now attached to a new attribute returned with
the results. This message can then be read by clients to warn a user
about upcoming changes to the query engine.

The `influx` client has already been modified to read this message and
print it out for every format except CSV.

The first warning message is a deprecated message about removing `IF NOT
EXISTS` from `CREATE DATABASE`.

The message will also be printed to the server log.

Fixes #5707.
emidoots added a commit to sourcegraph/appdash that referenced this issue Apr 27, 2016
…reating DB

`IF NOT EXIST` is the default and only behavior (see [InfluxDB issue here](influxdata/influxdb#5707)),
confirmed working locally.
emidoots added a commit to sourcegraph/appdash that referenced this issue Apr 27, 2016
…reating DB

`IF NOT EXIST` is the default and only behavior (see [InfluxDB issue here](influxdata/influxdb#5707)),
confirmed working locally.
curry684 added a commit to curry684/influxdb-php that referenced this issue Oct 31, 2016
As noted in influxdata/influxdb#5707 InfluxDB itself does not support this syntax anymore and will raise a syntax error if attempted.

Refs influxdata#67
karthick-rn pushed a commit to karthick-rn/fluo that referenced this issue Oct 21, 2020
The `IF NOT EXISTS` syntax has been removed from Influxdb v1.0.0
onwards. Ref: influxdata/influxdb#5707
ctubbsii pushed a commit to apache/fluo that referenced this issue Oct 22, 2020
The `IF NOT EXISTS` syntax has been removed from Influxdb v1.0.0
onwards. Ref: influxdata/influxdb#5707

Co-authored-by: Karthick Narendran <kanarend@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants