Skip to content

cherrypick-2.0: sql: clarify the error message upon access to vtable with no db#24809

Merged
craig[bot] merged 1 commit intocockroachdb:release-2.0from
knz:20180415-cherrypick-24772
Apr 17, 2018
Merged

cherrypick-2.0: sql: clarify the error message upon access to vtable with no db#24809
craig[bot] merged 1 commit intocockroachdb:release-2.0from
knz:20180415-cherrypick-24772

Conversation

@knz
Copy link
Copy Markdown
Contributor

@knz knz commented Apr 15, 2018

Picks #24772.

cc @cockroachdb/release

@knz knz requested review from a team and jordanlewis April 15, 2018 16:56
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@knz knz requested a review from justinj April 16, 2018 15:11
Prior to this patch:

```
root@:26257/> select * from pg_catalog.pg_class;
pq: no database specified
```

After:

```
root@:26257/> select * from pg_catalog.pg_class;
pq: cannot access virtual schema in anonymous database
HINT: verify that the current database is set
```

Release note (sql change): the error message produced upon accessing a
virtual schema with no database prefix (e.g. when `database` is not
set) is now clarified.
@knz knz force-pushed the 20180415-cherrypick-24772 branch from 06b5dbf to 0e69155 Compare April 16, 2018 21:35
@knz
Copy link
Copy Markdown
Contributor Author

knz commented Apr 16, 2018

rebased to resolve conflict, rfal

@knz
Copy link
Copy Markdown
Contributor Author

knz commented Apr 17, 2018

Thanks!

@knz
Copy link
Copy Markdown
Contributor Author

knz commented Apr 17, 2018

bors r+

craig bot pushed a commit that referenced this pull request Apr 17, 2018
24809: cherrypick-2.0: sql: clarify the error message upon access to vtable with no db r=knz a=knz

Picks #24772.

cc @cockroachdb/release 

Co-authored-by: Raphael 'kena' Poss <knz@cockroachlabs.com>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 17, 2018

Build succeeded

@craig craig bot merged commit 0e69155 into cockroachdb:release-2.0 Apr 17, 2018
@knz knz deleted the 20180415-cherrypick-24772 branch April 17, 2018 12:49
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.

3 participants