cartodb4/cartodb-postgresql
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
373 Commits 1 Branch 92 Tags 1.5 MiB
4fe85a6a76
Commit Graph

373 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Rafa de la Torre
4fe85a6a76 Merge pull request #129 from CartoDB/120-fix-extension-upgrade 
Do not remove old function #120
 2015-08-31 14:05:55 +02:00
Rafa de la Torre
2269dc0cb5 Add explanation in NEWS file #120 
as suggested in PR.
 2015-08-31 12:56:16 +02:00
Rafa de la Torre
0ba57f436a Do not remove old function #120 
The `DROP FUNCTION IF EXISTS` was added as transient code and not needed
anymore. See the ticket #120 for more information on this.
 2015-08-31 12:01:04 +02:00
Rafa de la Torre
a5ccbdddcf Update version to 0.9.4 in Makefile #123 2015-08-27 16:47:10 +02:00
Rafa de la Torre
45383d7c8a Merge pull request #127 from CartoDB/123-fix-indices 
Fix for index generation when renaming table #123
 2015-08-27 16:44:19 +02:00
Rafa de la Torre
0057e2ddec Fix for index generation when renaming table #123 2015-08-27 16:33:46 +02:00
Rafa de la Torre
79cacb8ef4 Modify sampling of table quota trigger #126 2015-08-27 12:52:30 +02:00
Rafa de la Torre
7f63688a2f New version 0.9.2 2015-08-24 15:20:50 +02:00
Rafa de la Torre
7ed3c29f82 Merge pull request #121 from CartoDB/text_geom_columns 
Handle text 'the_geom' columns as desired in #117
 2015-08-24 14:47:27 +02:00
Paul Ramsey
8c2252a9cb Handle text 'the_geom' columns as desired in #117 2015-08-21 13:10:05 -07:00
Rafa de la Torre
95c97a6b1c Update version and news 2015-08-19 19:42:29 +02:00
Rafa de la Torre
5dd497bf20 Fix transformation to webmercator in corner cases #116 2015-08-19 19:37:50 +02:00
Rafa de la Torre
74e6807c2f Merge pull request #78 from CartoDB/new_cartodbfy 
[wip] New cartodbfy function (overwrites CDB_CartodbfyTable)
 2015-08-19 15:11:55 +02:00
Rafa de la Torre
f71a2ac52e Version updated to 0.9.0 plus release notes 2015-08-19 15:08:07 +02:00
Rafa de la Torre
805af3babf Review of format strings and escaping of id's 
Just found cartodbfy failed for schema-names-with-dashes. This should
fix it.
 2015-08-18 17:05:32 +02:00
Rafa de la Torre
78bf202b17 Fix for schema-with-dashes 2015-08-18 15:41:11 +02:00
Rafa de la Torre
ed97d87a23 Go back to version 0.8.2 
In order to be able to test and rollback, should be needed

See
https://github.com/CartoDB/cartodb-postgresql/blob/master/CONTRIBUTING.md#testing-changes-live
 2015-08-18 15:15:56 +02:00
Rafa de la Torre
b798130d23 Update CONTRIBUTING.md 2015-08-18 15:08:35 +02:00
Rafa de la Torre
7800e4e5a8 Update CONTRIBUTING.md 2015-08-18 14:56:37 +02:00
Rafa de la Torre
520024ecb0 Update CONTRIBUTING.md 2015-08-18 14:56:23 +02:00
Raul Ochoa
5d22464036 Merge pull request #113 from CartoDB/issue-108 
Improve cartodb._CDB_total_relation_size performance
 2015-08-18 11:17:42 +02:00
Rafa de la Torre
53754236e3 Fix for quota test after merge with master 
Now cartodbfyied tables take less space because of the timestamp
columns.
 2015-08-17 15:56:48 +02:00
Rafa de la Torre
3f588df6f6 Drop function in order to change return value 2015-08-17 15:28:37 +02:00
Rafa de la Torre
2b48f90374 Merge remote-tracking branch 'origin/master' into new_cartodbfy 
Conflicts:
	test/CDB_QuotaTest.sql
 2015-08-17 15:27:33 +02:00
Raul Ochoa
50169e58d5 Raise better notice on _CDB_total_relation_size errors 2015-08-17 10:25:31 +02:00
Raul Ochoa
6bc91c7125 Using exception with pg_total_relation_size for better performance 
IF EXISTS is too slow, one order of magnitude, than using exception
handling. In combination with thousands of tables to check total
relation size that's a problem.

Exception handles specifically undefined_table as it was the original
issue but also any other exception to guarantee a size is always
returned and no error is raised.
 2015-08-17 10:17:07 +02:00
Paul Ramsey
a61a92a8f7 Merge pull request #109 from CartoDB/new_cartodbfy_bw_compat_signature 
Replace CDB_CartodbfyTable by new CartodbfyTable2
 2015-08-14 10:34:54 -07:00
Rafa de la Torre
47d8429277 Use return value from cartodbfy 2015-08-14 17:41:55 +02:00
Rafa de la Torre
565edcb50d Make cartodbfy return destoid 2015-08-14 16:53:43 +02:00
Rafa de la Torre
b7b5be1f3f Add minor piece of doc 2015-08-14 16:10:38 +02:00
Rafa de la Torre
900531f0c1 Disable a couple of tests 
Comment out tests that check cartodb_id text columns. These are no
longer taken into consideration as candidate primary ID (candidate
columns should be numeric).
 2015-08-14 13:46:01 +02:00
Rafa de la Torre
010dd13e4d Simple fix for type cheking in test 2015-08-14 13:40:10 +02:00
Rafa de la Torre
72ebc398f8 Recover _CDB_check_prerequisites (sorry, my fault) 2015-08-14 12:30:01 +02:00
Rafa de la Torre
3d89d8231f Fix deletion of cartodb_postgresql_unpriv_user 
only used in tests
 2015-08-14 09:48:34 +00:00
Rafa de la Torre
f211669e9e Tweak expected output of test_ddl_triggers 
Just touch expected output to adapt to NOTICEs and other stuff that
don't affect functionality.
 2015-08-14 10:57:11 +02:00
Paul Ramsey
7f55a0263b Fix regclass mismatch on column alter/drop 
This logic SHOULD BE MOVED TO Cartodbfy internals.
 2015-08-13 16:10:23 -07:00
Paul Ramsey
d268cd07cb Fix many tests and handle ownership issues involved 
with Cartodbfy being invoked by schema triggers. Some
issues with regclass interpretation in tests still remain.
Some issues with slightly different behavior to old version
remain. Some issues with error messages / notification messages
changing a little still remain.
 2015-08-13 15:59:45 -07:00
Raul Ochoa
219d876973 Merge pull request #110 from CartoDB/issue-108 
Fixes CDB_UserDataSize failing due `ERROR: relation "*" does not exist.`
 2015-08-13 19:01:05 +02:00
Raul Ochoa
60cc218664 Merge pull request #112 from CartoDB/issue-98 
Review test to validate permissions in public tables
 2015-08-13 19:00:09 +02:00
Raul Ochoa
e959bba335 Validates it's enought to grant publicuser to user. 
Reference to PR where it will be actually fixed for now.
 2015-08-13 18:53:41 +02:00
Raul Ochoa
714ba9d0dc Keep all test running :D 2015-08-13 13:27:27 +02:00
Raul Ochoa
4e31d3a37e Fixes CDB_UserDataSize failing due ERROR: relation "*" does not exist. 
Adds new _CDB_total_relation_size function that handles nonexistent
tables and does fallback to size=0.

That function could be used to cache total relation size or query another
table view with a cached total relation size.

Fixes #108
 2015-08-13 13:23:35 +02:00
Rafa de la Torre
a5321ec7a5 Replace CDB_CartodbfyTable by new CartodbfyTable2 
- Delete old CDB_CartodbfyTable code
- Delete auxiliary functions no longer used
- Modify the new CDB_CartodbfyTable signature to be backwards
  compatible.
 2015-08-12 18:39:07 +02:00
Rafa de la Torre
c00d607ee2 Merge pull request #107 from CartoDB/new_cartodbfy_rtorre 
Do not create timestamp columns/triggers on cartodbfy
 2015-08-12 18:34:33 +02:00
Rafa de la Torre
8a031f56f5 Recover test for cartodb_id not-null constraint 2015-08-12 15:58:30 +00:00
Rafa de la Torre
8c41203db6 Fix for the_geom does not exist 
When creating triggers, expectation is to have the columns the_geom and
the_geom_webmercator even if the source table does not have any geometry
columns. Populate it in the rewrite with NULL values and right types.
 2015-08-12 17:34:47 +02:00
Rafa de la Torre
6b9ab3d956 Fix quota test 
Now the cartodbfied table is a bit smaller because it does not have the
timestamp columns.
 2015-08-12 10:26:37 +02:00
Raul Ochoa
14213c5d6a Test scenario to show how public tables can be accessed by other roles 
This will require changes in cartodb/cartodb rails app because user/role
creation happens over there. So it's not possible to fix all the problems
within the extension right now.
 2015-08-12 10:10:21 +02:00
Rafa de la Torre
c11d1bbf50 Fix CDB_CartodbfyTableTest 
by removing references to created_at and updated_at columns
 2015-08-11 19:52:37 +02:00
Rafa de la Torre
67f8a8cd69 Fix test_ddl_triggers.sql 
by removing references to created_at & updated_at columns
 2015-08-11 19:18:10 +02:00