prosody
6216743c188c
plugins/mod_storage_sql2.lua

Find changesets by keywords by author, files, the commit message, revision number or hash, or revset expression.

Software / code / prosody

Annotate

plugins/mod_storage_sql2.lua @ 6739:6216743c188c

mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
author Matthew Wild <mwild1@gmail.com>
date Wed, 24 Jun 2015 23:24:32 +0100
parent 6738:845bc5ba306d
child 6740:99b3f29c3c71
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
1
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
2 local json = require "util.json";
6739
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
3 local sql = require "util.sql";
5735
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
4 local xml_parse = require "util.xml".parse;
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
5 local uuid = require "util.uuid";
6165
6a184b16b717 core.certmanager, core.moduleapi, mod_storage_sql, mod_storage_sql2: Import from util.paths
Kim Alvefur <zash@zash.se>
parents: 6060
diff changeset
6 local resolve_relative_path = require "util.paths".resolve_relative_path;
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
7
5735
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
8 local stanza_mt = require"util.stanza".stanza_mt;
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
9 local getmetatable = getmetatable;
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
10 local t_concat = table.concat;
5735
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
11 local function is_stanza(x) return getmetatable(x) == stanza_mt; end
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
12
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
13 local noop = function() end
5733
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
14 local unpack = unpack
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
15 local function iterator(result)
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
16 return function(result)
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
17 local row = result();
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
18 if row ~= nil then
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
19 return unpack(row);
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
20 end
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
21 end, result, nil;
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
22 end
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
23
6739
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
24 local default_params = { driver = "SQLite3" };
5883
39b187e7e892 mod_storage_sql2, util.sql: Move code for setting encoding to util.sql
Kim Alvefur <zash@zash.se>
parents: 5882
diff changeset
25
6739
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
26 local engine;
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
27
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
28 local function serialize(value)
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
29 local t = type(value);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
30 if t == "string" or t == "boolean" or t == "number" then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
31 return t, tostring(value);
5735
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
32 elseif is_stanza(value) then
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
33 return "xml", tostring(value);
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
34 elseif t == "table" then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
35 local value,err = json.encode(value);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
36 if value then return "json", value; end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
37 return nil, err;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
38 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
39 return nil, "Unhandled value type: "..t;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
40 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
41 local function deserialize(t, value)
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
42 if t == "string" then return value;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
43 elseif t == "boolean" then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
44 if value == "true" then return true;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
45 elseif value == "false" then return false; end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
46 elseif t == "number" then return tonumber(value);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
47 elseif t == "json" then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
48 return json.decode(value);
5735
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
49 elseif t == "xml" then
24f4e73645fe mod_storage_sql2: Support XML serialization for stanzas.
Kim Alvefur <zash@zash.se>
parents: 5734
diff changeset
50 return xml_parse(value);
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
51 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
52 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
53
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
54 local host = module.host;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
55 local user, store;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
56
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
57 local function keyval_store_get()
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
58 local haveany;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
59 local result = {};
5732
4aa1d6f5083a mod_storage_sql2: Make sure the user field is not NULL
Kim Alvefur <zash@zash.se>
parents: 5711
diff changeset
60 for row in engine:select("SELECT `key`,`type`,`value` FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=?", host, user or "", store) do
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
61 haveany = true;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
62 local k = row[1];
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
63 local v = deserialize(row[2], row[3]);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
64 if k and v then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
65 if k ~= "" then result[k] = v; elseif type(v) == "table" then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
66 for a,b in pairs(v) do
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
67 result[a] = b;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
68 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
69 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
70 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
71 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
72 if haveany then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
73 return result;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
74 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
75 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
76 local function keyval_store_set(data)
5732
4aa1d6f5083a mod_storage_sql2: Make sure the user field is not NULL
Kim Alvefur <zash@zash.se>
parents: 5711
diff changeset
77 engine:delete("DELETE FROM `prosody` WHERE `host`=? AND `user`=? AND `store`=?", host, user or "", store);
5776
bd0ff8ae98a8 Remove all trailing whitespace
Florian Zeitz <florob@babelmonkeys.de>
parents: 5765
diff changeset
78
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
79 if data and next(data) ~= nil then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
80 local extradata = {};
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
81 for key, value in pairs(data) do
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
82 if type(key) == "string" and key ~= "" then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
83 local t, value = serialize(value);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
84 assert(t, value);
5732
4aa1d6f5083a mod_storage_sql2: Make sure the user field is not NULL
Kim Alvefur <zash@zash.se>
parents: 5711
diff changeset
85 engine:insert("INSERT INTO `prosody` (`host`,`user`,`store`,`key`,`type`,`value`) VALUES (?,?,?,?,?,?)", host, user or "", store, key, t, value);
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
86 else
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
87 extradata[key] = value;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
88 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
89 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
90 if next(extradata) ~= nil then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
91 local t, extradata = serialize(extradata);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
92 assert(t, extradata);
5732
4aa1d6f5083a mod_storage_sql2: Make sure the user field is not NULL
Kim Alvefur <zash@zash.se>
parents: 5711
diff changeset
93 engine:insert("INSERT INTO `prosody` (`host`,`user`,`store`,`key`,`type`,`value`) VALUES (?,?,?,?,?,?)", host, user or "", store, "", t, extradata);
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
94 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
95 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
96 return true;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
97 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
98
6736
4aee55c0cc5c mod_storage_sql2: Add some comments
Matthew Wild <mwild1@gmail.com>
parents: 6532
diff changeset
99 --- Key/value store API (default store type)
4aee55c0cc5c mod_storage_sql2: Add some comments
Matthew Wild <mwild1@gmail.com>
parents: 6532
diff changeset
100
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
101 local keyval_store = {};
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
102 keyval_store.__index = keyval_store;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
103 function keyval_store:get(username)
6737
9f932a31eeba mod_storage_sql2: Some reformatting and variable name improvements
Matthew Wild <mwild1@gmail.com>
parents: 6736
diff changeset
104 user, store = username, self.store;
6282
bce801e40484 mod_storage_sql2: Don't ignore failure in keyval_store:get() (thanks daurnimator)
Kim Alvefur <zash@zash.se>
parents: 6165
diff changeset
105 local ok, result = engine:transaction(keyval_store_get);
6738
845bc5ba306d mod_storage_sql2: Improve logging when database read fails
Matthew Wild <mwild1@gmail.com>
parents: 6737
diff changeset
106 if not ok then
845bc5ba306d mod_storage_sql2: Improve logging when database read fails
Matthew Wild <mwild1@gmail.com>
parents: 6737
diff changeset
107 module:log("error", "Unable to read from database %s store for %s: %s", store, username or "<host>", result);
845bc5ba306d mod_storage_sql2: Improve logging when database read fails
Matthew Wild <mwild1@gmail.com>
parents: 6737
diff changeset
108 return nil, result;
845bc5ba306d mod_storage_sql2: Improve logging when database read fails
Matthew Wild <mwild1@gmail.com>
parents: 6737
diff changeset
109 end
845bc5ba306d mod_storage_sql2: Improve logging when database read fails
Matthew Wild <mwild1@gmail.com>
parents: 6737
diff changeset
110 return result;
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
111 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
112 function keyval_store:set(username, data)
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
113 user,store = username,self.store;
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
114 return engine:transaction(function()
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
115 return keyval_store_set(data);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
116 end);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
117 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
118 function keyval_store:users()
5733
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
119 local ok, result = engine:transaction(function()
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
120 return engine:select("SELECT DISTINCT `user` FROM `prosody` WHERE `host`=? AND `store`=?", host, self.store);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
121 end);
5733
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
122 if not ok then return ok, result end
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
123 return iterator(result);
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
124 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
125
6736
4aee55c0cc5c mod_storage_sql2: Add some comments
Matthew Wild <mwild1@gmail.com>
parents: 6532
diff changeset
126 --- Archive store API
4aee55c0cc5c mod_storage_sql2: Add some comments
Matthew Wild <mwild1@gmail.com>
parents: 6532
diff changeset
127
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
128 local archive_store = {}
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
129 archive_store.__index = archive_store
5969
0a3b8003ac9c mod_storage_sql2: Expose the unique key argument, allowing arbitrary ids. Conflicting items are removed.
Kim Alvefur <zash@zash.se>
parents: 5968
diff changeset
130 function archive_store:append(username, key, when, with, value)
0a3b8003ac9c mod_storage_sql2: Expose the unique key argument, allowing arbitrary ids. Conflicting items are removed.
Kim Alvefur <zash@zash.se>
parents: 5968
diff changeset
131 if value == nil then -- COMPAT early versions
0a3b8003ac9c mod_storage_sql2: Expose the unique key argument, allowing arbitrary ids. Conflicting items are removed.
Kim Alvefur <zash@zash.se>
parents: 5968
diff changeset
132 when, with, value, key = key, when, with, value
0a3b8003ac9c mod_storage_sql2: Expose the unique key argument, allowing arbitrary ids. Conflicting items are removed.
Kim Alvefur <zash@zash.se>
parents: 5968
diff changeset
133 end
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
134 local user,store = username,self.store;
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
135 return engine:transaction(function()
5996
e7efa9703a3f mod_storage_sql2: Only attempt to delete conflicting items if an ID/key is given
Kim Alvefur <zash@zash.se>
parents: 5995
diff changeset
136 if key then
e7efa9703a3f mod_storage_sql2: Only attempt to delete conflicting items if an ID/key is given
Kim Alvefur <zash@zash.se>
parents: 5995
diff changeset
137 engine:delete("DELETE FROM `prosodyarchive` WHERE `host`=? AND `user`=? AND `store`=? AND `key`=?", host, user or "", store, key);
e7efa9703a3f mod_storage_sql2: Only attempt to delete conflicting items if an ID/key is given
Kim Alvefur <zash@zash.se>
parents: 5995
diff changeset
138 else
e7efa9703a3f mod_storage_sql2: Only attempt to delete conflicting items if an ID/key is given
Kim Alvefur <zash@zash.se>
parents: 5995
diff changeset
139 key = uuid.generate();
e7efa9703a3f mod_storage_sql2: Only attempt to delete conflicting items if an ID/key is given
Kim Alvefur <zash@zash.se>
parents: 5995
diff changeset
140 end
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
141 local t, value = serialize(value);
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
142 engine:insert("INSERT INTO `prosodyarchive` (`host`, `user`, `store`, `when`, `with`, `key`, `type`, `value`) VALUES (?,?,?,?,?,?,?,?)", host, user or "", store, when, with, key, t, value);
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
143 return key;
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
144 end);
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
145 end
5847
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
146
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
147 -- Helpers for building the WHERE clause
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
148 local function archive_where(query, args, where)
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
149 -- Time range, inclusive
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
150 if query.start then
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
151 args[#args+1] = query.start
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
152 where[#where+1] = "`when` >= ?"
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
153 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
154
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
155 if query["end"] then
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
156 args[#args+1] = query["end"];
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
157 if query.start then
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
158 where[#where] = "`when` BETWEEN ? AND ?" -- is this inclusive?
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
159 else
5903
0e0aab930e10 mod_storage_sql2: Fix backwards comparison of timestamp
Kim Alvefur <zash@zash.se>
parents: 5894
diff changeset
160 where[#where+1] = "`when` <= ?"
5847
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
161 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
162 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
163
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
164 -- Related name
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
165 if query.with then
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
166 where[#where+1] = "`with` = ?";
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
167 args[#args+1] = query.with
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
168 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
169
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
170 -- Unique id
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
171 if query.key then
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
172 where[#where+1] = "`key` = ?";
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
173 args[#args+1] = query.key
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
174 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
175 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
176 local function archive_where_id_range(query, args, where)
5968
e1e9f1411442 mod_storage_sql2: Include user, host and store in id lookup
Kim Alvefur <zash@zash.se>
parents: 5913
diff changeset
177 local args_len = #args
5847
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
178 -- Before or after specific item, exclusive
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
179 if query.after then -- keys better be unique!
5985
98ed9fe368ac mod_storage_sql2: Fix SQL syntax
Kim Alvefur <zash@zash.se>
parents: 5979
diff changeset
180 where[#where+1] = "`sort_id` > (SELECT `sort_id` FROM `prosodyarchive` WHERE `key` = ? AND `host` = ? AND `user` = ? AND `store` = ? LIMIT 1)"
5968
e1e9f1411442 mod_storage_sql2: Include user, host and store in id lookup
Kim Alvefur <zash@zash.se>
parents: 5913
diff changeset
181 args[args_len+1], args[args_len+2], args[args_len+3], args[args_len+4] = query.after, args[1], args[2], args[3];
e1e9f1411442 mod_storage_sql2: Include user, host and store in id lookup
Kim Alvefur <zash@zash.se>
parents: 5913
diff changeset
182 args_len = args_len + 4
5847
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
183 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
184 if query.before then
5985
98ed9fe368ac mod_storage_sql2: Fix SQL syntax
Kim Alvefur <zash@zash.se>
parents: 5979
diff changeset
185 where[#where+1] = "`sort_id` < (SELECT `sort_id` FROM `prosodyarchive` WHERE `key` = ? AND `host` = ? AND `user` = ? AND `store` = ? LIMIT 1)"
5968
e1e9f1411442 mod_storage_sql2: Include user, host and store in id lookup
Kim Alvefur <zash@zash.se>
parents: 5913
diff changeset
186 args[args_len+1], args[args_len+2], args[args_len+3], args[args_len+4] = query.before, args[1], args[2], args[3];
5847
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
187 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
188 end
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
189
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
190 function archive_store:find(username, query)
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
191 query = query or {};
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
192 local user,store = username,self.store;
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
193 local total;
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
194 local ok, result = engine:transaction(function()
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
195 local sql_query = "SELECT `key`, `type`, `value`, `when` FROM `prosodyarchive` WHERE %s ORDER BY `sort_id` %s%s;";
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
196 local args = { host, user or "", store, };
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
197 local where = { "`host` = ?", "`user` = ?", "`store` = ?", };
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
198
5847
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
199 archive_where(query, args, where);
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
200
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
201 -- Total matching
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
202 if query.total then
6060
de4c83feb064 mod_storage_sql2: Build counter query without ORDER BY clause
Kim Alvefur <zash@zash.se>
parents: 6013
diff changeset
203 local stats = engine:select("SELECT COUNT(*) FROM `prosodyarchive` WHERE " .. t_concat(where, " AND "), unpack(args));
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
204 if stats then
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
205 local _total = stats()
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
206 total = _total and _total[1];
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
207 end
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
208 if query.limit == 0 then -- Skip the real query
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
209 return noop, total;
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
210 end
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
211 end
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
212
5847
6aaa7ad4463c mod_storage_sql2: Split out code for building WHERE clauses into separate functions
Kim Alvefur <zash@zash.se>
parents: 5776
diff changeset
213 archive_where_id_range(query, args, where);
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
214
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
215 if query.limit then
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
216 args[#args+1] = query.limit;
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
217 end
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
218
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
219 sql_query = sql_query:format(t_concat(where, " AND "), query.reverse and "DESC" or "ASC", query.limit and " LIMIT ?" or "");
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
220 module:log("debug", sql_query);
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
221 return engine:select(sql_query, unpack(args));
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
222 end);
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
223 if not ok then return ok, result end
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
224 return function()
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
225 local row = result();
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
226 if row ~= nil then
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
227 return row[1], deserialize(row[2], row[3]), row[4];
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
228 end
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
229 end, total;
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
230 end
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
231
5848
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
232 function archive_store:delete(username, query)
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
233 query = query or {};
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
234 local user,store = username,self.store;
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
235 return engine:transaction(function()
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
236 local sql_query = "DELETE FROM `prosodyarchive` WHERE %s;";
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
237 local args = { host, user or "", store, };
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
238 local where = { "`host` = ?", "`user` = ?", "`store` = ?", };
6013
918ab89cb68d mod_storage_sql2: archive:delete() with username = true deletes for all users
Kim Alvefur <zash@zash.se>
parents: 5996
diff changeset
239 if user == true then
918ab89cb68d mod_storage_sql2: archive:delete() with username = true deletes for all users
Kim Alvefur <zash@zash.se>
parents: 5996
diff changeset
240 table.remove(args, 2);
918ab89cb68d mod_storage_sql2: archive:delete() with username = true deletes for all users
Kim Alvefur <zash@zash.se>
parents: 5996
diff changeset
241 table.remove(where, 2);
918ab89cb68d mod_storage_sql2: archive:delete() with username = true deletes for all users
Kim Alvefur <zash@zash.se>
parents: 5996
diff changeset
242 end
5848
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
243 archive_where(query, args, where);
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
244 archive_where_id_range(query, args, where);
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
245 sql_query = sql_query:format(t_concat(where, " AND "));
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
246 module:log("debug", sql_query);
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
247 return engine:delete(sql_query, unpack(args));
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
248 end);
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
249 end
06156bfd4eaf mod_storage_sql2: Add method for deleting items from archives with same syntax as :find()
Kim Alvefur <zash@zash.se>
parents: 5847
diff changeset
250
5734
49f1fed6e25e mod_storage_sql2: Keep available store types in a table
Kim Alvefur <zash@zash.se>
parents: 5733
diff changeset
251 local stores = {
49f1fed6e25e mod_storage_sql2: Keep available store types in a table
Kim Alvefur <zash@zash.se>
parents: 5733
diff changeset
252 keyval = keyval_store;
5741
c7a664e496b3 mod_storage_sql2: Add archive store with append and find methods
Kim Alvefur <zash@zash.se>
parents: 5740
diff changeset
253 archive = archive_store;
5734
49f1fed6e25e mod_storage_sql2: Keep available store types in a table
Kim Alvefur <zash@zash.se>
parents: 5733
diff changeset
254 };
49f1fed6e25e mod_storage_sql2: Keep available store types in a table
Kim Alvefur <zash@zash.se>
parents: 5733
diff changeset
255
6736
4aee55c0cc5c mod_storage_sql2: Add some comments
Matthew Wild <mwild1@gmail.com>
parents: 6532
diff changeset
256 --- Implement storage driver API
4aee55c0cc5c mod_storage_sql2: Add some comments
Matthew Wild <mwild1@gmail.com>
parents: 6532
diff changeset
257
4aee55c0cc5c mod_storage_sql2: Add some comments
Matthew Wild <mwild1@gmail.com>
parents: 6532
diff changeset
258 -- FIXME: Some of these operations need to operate on the archive store(s) too
4aee55c0cc5c mod_storage_sql2: Add some comments
Matthew Wild <mwild1@gmail.com>
parents: 6532
diff changeset
259
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
260 local driver = {};
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
261
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
262 function driver:open(store, typ)
5734
49f1fed6e25e mod_storage_sql2: Keep available store types in a table
Kim Alvefur <zash@zash.se>
parents: 5733
diff changeset
263 local store_mt = stores[typ or "keyval"];
49f1fed6e25e mod_storage_sql2: Keep available store types in a table
Kim Alvefur <zash@zash.se>
parents: 5733
diff changeset
264 if store_mt then
49f1fed6e25e mod_storage_sql2: Keep available store types in a table
Kim Alvefur <zash@zash.se>
parents: 5733
diff changeset
265 return setmetatable({ store = store }, store_mt);
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
266 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
267 return nil, "unsupported-store";
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
268 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
269
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
270 function driver:stores(username)
6737
9f932a31eeba mod_storage_sql2: Some reformatting and variable name improvements
Matthew Wild <mwild1@gmail.com>
parents: 6736
diff changeset
271 local query = "SELECT DISTINCT `store` FROM `prosody` WHERE `host`=? AND `user`" ..
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
272 (username == true and "!=?" or "=?");
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
273 if username == true or not username then
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
274 username = "";
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
275 end
5733
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
276 local ok, result = engine:transaction(function()
6737
9f932a31eeba mod_storage_sql2: Some reformatting and variable name improvements
Matthew Wild <mwild1@gmail.com>
parents: 6736
diff changeset
277 return engine:select(query, host, username);
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
278 end);
5733
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
279 if not ok then return ok, result end
aeeced7b0149 mod_storage_sql2: Fix iteration over users and stores
Kim Alvefur <zash@zash.se>
parents: 5732
diff changeset
280 return iterator(result);
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
281 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
282
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
283 function driver:purge(username)
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
284 return engine:transaction(function()
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
285 local stmt,err = engine:delete("DELETE FROM `prosody` WHERE `host`=? AND `user`=?", host, username);
6737
9f932a31eeba mod_storage_sql2: Some reformatting and variable name improvements
Matthew Wild <mwild1@gmail.com>
parents: 6736
diff changeset
286 return true, err;
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
287 end);
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
288 end
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
289
6739
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
290 --- Initialization
5494
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
291
9916f0a2d178 mod_storage_sql2 (temporary name), sql.lib, util.sql: New SQL API supporting cross-module connection sharing, transactions and Things - a work in progress
Matthew Wild <mwild1@gmail.com>
parents:
diff changeset
292
6739
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
293 local function create_table(name)
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
294 local Table, Column, Index = sql.Table, sql.Column, sql.Index;
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
295
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
296 local ProsodyTable = Table {
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
297 name= name or "prosody";
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
298 Column { name="host", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
299 Column { name="user", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
300 Column { name="store", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
301 Column { name="key", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
302 Column { name="type", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
303 Column { name="value", type="MEDIUMTEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
304 Index { name="prosody_index", "host", "user", "store", "key" };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
305 };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
306 engine:transaction(function()
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
307 ProsodyTable:create(engine);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
308 end);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
309
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
310 local ProsodyArchiveTable = Table {
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
311 name="prosodyarchive";
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
312 Column { name="sort_id", type="INTEGER", primary_key=true, auto_increment=true };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
313 Column { name="host", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
314 Column { name="user", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
315 Column { name="store", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
316 Column { name="key", type="TEXT", nullable=false }; -- item id
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
317 Column { name="when", type="INTEGER", nullable=false }; -- timestamp
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
318 Column { name="with", type="TEXT", nullable=false }; -- related id
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
319 Column { name="type", type="TEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
320 Column { name="value", type="MEDIUMTEXT", nullable=false };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
321 Index { name="prosodyarchive_index", unique = true, "host", "user", "store", "key" };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
322 };
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
323 engine:transaction(function()
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
324 ProsodyArchiveTable:create(engine);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
325 end);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
326 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
327
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
328 local function upgrade_table(params, apply_changes)
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
329 local changes = false;
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
330 if params.driver == "MySQL" then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
331 local success,err = engine:transaction(function()
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
332 local result = engine:execute("SHOW COLUMNS FROM prosody WHERE Field='value' and Type='text'");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
333 if result:rowcount() > 0 then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
334 changes = true;
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
335 if apply_changes then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
336 module:log("info", "Upgrading database schema...");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
337 engine:execute("ALTER TABLE prosody MODIFY COLUMN `value` MEDIUMTEXT");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
338 module:log("info", "Database table automatically upgraded");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
339 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
340 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
341 return true;
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
342 end);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
343 if not success then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
344 module:log("error", "Failed to check/upgrade database schema (%s), please see "
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
345 .."http://prosody.im/doc/mysql for help",
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
346 err or "unknown error");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
347 return false;
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
348 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
349
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
350 -- COMPAT w/pre-0.10: Upgrade table to UTF-8 if not already
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
351 local check_encoding_query = "SELECT `COLUMN_NAME`,`COLUMN_TYPE` FROM `information_schema`.`columns` WHERE `TABLE_NAME`='prosody' AND ( `CHARACTER_SET_NAME`!='utf8' OR `COLLATION_NAME`!='utf8_bin' );";
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
352 success,err = engine:transaction(function()
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
353 local result = engine:execute(check_encoding_query);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
354 local n_bad_columns = result:rowcount();
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
355 if n_bad_columns > 0 then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
356 changes = true;
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
357 if apply_changes then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
358 module:log("warn", "Found %d columns in prosody table requiring encoding change, updating now...", n_bad_columns);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
359 local fix_column_query1 = "ALTER TABLE `prosody` CHANGE `%s` `%s` BLOB;";
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
360 local fix_column_query2 = "ALTER TABLE `prosody` CHANGE `%s` `%s` %s CHARACTER SET 'utf8' COLLATE 'utf8_bin';";
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
361 for row in result:rows() do
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
362 local column_name, column_type = unpack(row);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
363 engine:execute(fix_column_query1:format(column_name, column_name));
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
364 engine:execute(fix_column_query2:format(column_name, column_name, column_type));
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
365 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
366 module:log("info", "Database encoding upgrade complete!");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
367 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
368 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
369 end);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
370 success,err = engine:transaction(function() return engine:execute(check_encoding_query); end);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
371 if not success then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
372 module:log("error", "Failed to check/upgrade database encoding: %s", err or "unknown error");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
373 return false;
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
374 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
375 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
376 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
377
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
378 local function normalize_params(params)
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
379 assert(params.driver and params.database, "Configuration error: Both the SQL driver and the database need to be specified");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
380 if params.driver == "SQLite3" then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
381 params.database = resolve_relative_path(prosody.paths.data or ".", params.database or "prosody.sqlite");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
382 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
383 return params;
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
384 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
385
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
386 function module.load()
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
387 if prosody.prosodyctl then return; end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
388 local params = normalize_params(module:get_option("sql", default_params));
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
389 engine = sql:create_engine(params, function (engine)
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
390 if module:get_option("sql_manage_tables", true) then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
391 -- Automatically create table, ignore failure (table probably already exists)
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
392 -- FIXME: we should check in information_schema, etc.
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
393 create_table();
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
394 -- Check whether the table needs upgrading
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
395 if not upgrade_table(params, true) then
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
396 module:log("error", "Old database format detected, and upgrade failed");
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
397 return false, "database upgrade needed";
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
398 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
399 end
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
400 end);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
401
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
402 module:provides("storage", driver);
6216743c188c mod_storage_sql2: Break up monolithic code into functions, theoretically no functionality changes.
Matthew Wild <mwild1@gmail.com>
parents: 6738
diff changeset
403 end