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

Database encoding is set to SQL_ASCII #13

Closed
treyhyde opened this issue Jul 30, 2014 · 1 comment · Fixed by #14
Closed

Database encoding is set to SQL_ASCII #13

treyhyde opened this issue Jul 30, 2014 · 1 comment · Fixed by #14

Comments

@treyhyde
Copy link

The dockerfile lists initdb params of UTF-8 but the databases that ship by default with this image are all SQL_ASCII.

\l
List of databases
Name | Owner | Encoding | Collate | Ctype | Access privileges
-----------+----------+-----------+---------+-------+-----------------------
docker | docker | SQL_ASCII | C | C |
postgres | postgres | SQL_ASCII | C | C |
template0 | postgres | SQL_ASCII | C | C | =c/postgres +
| | | | | postgres=CTc/postgres
template1 | postgres | SQL_ASCII | C | C | =c/postgres +
| | | | | postgres=CTc/postgres
(4 rows)

@aanand
Copy link
Member

aanand commented Jul 30, 2014

I can reproduce this on master, but not on the 9.1 branch:

$ git checkout -b 9.1 origin/9.1
$ docker build -t postgresql .
$ docker run -d --name db postgresql
$ docker run --link db:db -i -t postgresql psql -h db -U docker -c '\l'
Password for user docker:
                                  List of databases
   Name    |  Owner   | Encoding |   Collate   |    Ctype    |   Access privileges
-----------+----------+----------+-------------+-------------+-----------------------
 docker    | docker   | UTF8     | en_US.UTF-8 | en_US.UTF-8 |
 postgres  | postgres | UTF8     | en_US.UTF-8 | en_US.UTF-8 |
 template0 | postgres | UTF8     | en_US.UTF-8 | en_US.UTF-8 | =c/postgres          +
           |          |          |             |             | postgres=CTc/postgres
 template1 | postgres | UTF8     | en_US.UTF-8 | en_US.UTF-8 | =c/postgres          +
           |          |          |             |             | postgres=CTc/postgres
(4 rows)

Looks like this regression was caused by #8 - could be an Ubuntu 14.04 thing. Investigating.

aanand added a commit that referenced this issue Jul 31, 2014
Regressed in bd66030

Closes #13.
aanand added a commit that referenced this issue Aug 1, 2014
Regressed in bd66030

Closes #13.
@bfirsh bfirsh closed this as completed in #14 Aug 1, 2014
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 a pull request may close this issue.

2 participants