annotate src/lib-dict/dict-client.c @ 9464:939edf3ed09b HEAD

maildir: Having a lot of keywords assert-crashed with "stack frame changed".
author Timo Sirainen <tss@iki.fi>
date Mon, 02 Nov 2009 17:54:17 -0500
parents a1b92a251bb9
children 00cd9aacd03c
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
8590
b9faf4db2a9f Updated copyright notices to include year 2009.
Timo Sirainen <tss@iki.fi>
parents: 8105
diff changeset
1 /* Copyright (c) 2005-2009 Dovecot authors, see the included COPYING file */
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
2
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
3 #include "lib.h"
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
4 #include "llist.h"
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
5 #include "str.h"
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
6 #include "network.h"
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
7 #include "istream.h"
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
8 #include "ostream.h"
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
9 #include "dict-private.h"
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
10 #include "dict-client.h"
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
11
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
12 #include <stdlib.h>
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
13 #include <unistd.h>
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
14 #include <fcntl.h>
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
15
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
16 struct client_dict {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
17 struct dict dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
18
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
19 pool_t pool;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
20 int fd;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
21 const char *uri;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
22 const char *username;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
23 const char *path;
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
24 enum dict_data_type value_type;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
25
7008
6950bb5e7921 Don't try to reconnect more often than once/sec.
Timo Sirainen <tss@iki.fi>
parents: 7007
diff changeset
26 time_t last_connect_try;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
27 struct istream *input;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
28 struct ostream *output;
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
29 struct io *io;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
30
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
31 struct client_dict_transaction_context *transactions;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
32
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
33 unsigned int connect_counter;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
34 unsigned int transaction_id_counter;
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
35 unsigned int async_commits;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
36
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
37 unsigned int in_iteration:1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
38 unsigned int handshaked:1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
39 };
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
40
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
41 struct client_dict_iterate_context {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
42 struct dict_iterate_context ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
43
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
44 pool_t pool;
3863
55df57c028d4 Added "bool" type and changed all ints that were used as booleans to bool.
Timo Sirainen <tss@iki.fi>
parents: 3858
diff changeset
45 bool failed;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
46 };
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
47
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
48 struct client_dict_transaction_context {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
49 struct dict_transaction_context ctx;
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
50 struct client_dict_transaction_context *prev, *next;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
51
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
52 /* for async commits */
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
53 dict_transaction_commit_callback_t *callback;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
54 void *context;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
55
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
56 unsigned int id;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
57 unsigned int connect_counter;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
58
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
59 unsigned int failed:1;
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
60 unsigned int sent_begin:1;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
61 };
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
62
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
63 static int client_dict_connect(struct client_dict *dict);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
64 static void client_dict_disconnect(struct client_dict *dict);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
65
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
66 const char *dict_client_escape(const char *src)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
67 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
68 const char *p;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
69 string_t *dest;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
70
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
71 /* first do a quick lookup to see if there's anything to escape.
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
72 probably not. */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
73 for (p = src; *p != '\0'; p++) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
74 if (*p == '\t' || *p == '\n' || *p == '\001')
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
75 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
76 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
77
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
78 if (*p == '\0')
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
79 return src;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
80
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
81 dest = t_str_new(256);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
82 str_append_n(dest, src, p - src);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
83
3858
2e5c34fe9634 Small (compiling) fixes
Timo Sirainen <tss@iki.fi>
parents: 3852
diff changeset
84 for (; *p != '\0'; p++) {
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
85 switch (*p) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
86 case '\t':
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
87 str_append_c(dest, '\001');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
88 str_append_c(dest, 't');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
89 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
90 case '\n':
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
91 str_append_c(dest, '\001');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
92 str_append_c(dest, 'n');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
93 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
94 case '\001':
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
95 str_append_c(dest, '\001');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
96 str_append_c(dest, '1');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
97 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
98 default:
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
99 str_append_c(dest, *p);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
100 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
101 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
102 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
103 return str_c(dest);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
104 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
105
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
106 const char *dict_client_unescape(const char *src)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
107 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
108 const char *p;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
109 string_t *dest;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
110
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
111 /* first do a quick lookup to see if there's anything to unescape.
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
112 probably not. */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
113 for (p = src; *p != '\0'; p++) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
114 if (*p == '\001')
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
115 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
116 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
117
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
118 if (*p == '\0')
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
119 return src;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
120
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
121 dest = t_str_new(256);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
122 str_append_n(dest, src, p - src);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
123 for (; *p != '\0'; p++) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
124 if (*p != '\001')
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
125 str_append_c(dest, *p);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
126 else if (p[1] != '\0') {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
127 p++;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
128 switch (*p) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
129 case '1':
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
130 str_append_c(dest, '\001');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
131 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
132 case 't':
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
133 str_append_c(dest, '\t');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
134 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
135 case 'n':
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
136 str_append_c(dest, '\n');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
137 break;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
138 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
139 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
140 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
141 return str_c(dest);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
142 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
143
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
144 static int client_dict_send_query(struct client_dict *dict, const char *query)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
145 {
4368
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
146 if (dict->output == NULL) {
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
147 /* not connected currently */
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
148 if (client_dict_connect(dict) < 0)
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
149 return -1;
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
150 }
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
151
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
152 if (o_stream_send_str(dict->output, query) < 0 ||
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
153 o_stream_flush(dict->output) < 0) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
154 /* Send failed */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
155 if (!dict->handshaked) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
156 /* we're trying to send hello, don't try to reconnect */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
157 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
158 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
159
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
160 /* Reconnect and try again. */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
161 client_dict_disconnect(dict);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
162 if (client_dict_connect(dict) < 0)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
163 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
164
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
165 if (o_stream_send_str(dict->output, query) < 0 ||
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
166 o_stream_flush(dict->output) < 0) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
167 i_error("write(%s) failed: %m", dict->path);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
168 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
169 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
170 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
171 return 0;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
172 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
173
4368
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
174 static int
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
175 client_dict_transaction_send_begin(struct client_dict_transaction_context *ctx)
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
176 {
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
177 struct client_dict *dict = (struct client_dict *)ctx->ctx.dict;
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
178
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
179 if (ctx->failed)
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
180 return -1;
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
181
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
182 T_BEGIN {
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
183 const char *query;
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
184
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
185 query = t_strdup_printf("%c%u\n", DICT_PROTOCOL_CMD_BEGIN,
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
186 ctx->id);
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
187 if (client_dict_send_query(dict, query) < 0)
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
188 ctx->failed = TRUE;
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
189 else
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
190 ctx->connect_counter = dict->connect_counter;
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
191 } T_END;
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
192
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
193 return ctx->failed ? -1 : 0;
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
194 }
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
195
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
196 static int
4368
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
197 client_dict_send_transaction_query(struct client_dict_transaction_context *ctx,
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
198 const char *query)
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
199 {
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
200 struct client_dict *dict = (struct client_dict *)ctx->ctx.dict;
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
201
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
202 if (!ctx->sent_begin) {
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
203 if (client_dict_transaction_send_begin(ctx) < 0)
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
204 return -1;
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
205 ctx->sent_begin = TRUE;
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
206 }
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
207
4385
Timo Sirainen <tss@iki.fi>
parents: 4368
diff changeset
208 if (ctx->connect_counter != dict->connect_counter || ctx->failed)
Timo Sirainen <tss@iki.fi>
parents: 4368
diff changeset
209 return -1;
Timo Sirainen <tss@iki.fi>
parents: 4368
diff changeset
210
Timo Sirainen <tss@iki.fi>
parents: 4368
diff changeset
211 if (dict->output == NULL) {
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
212 /* not connected, this'll fail */
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
213 return -1;
4385
Timo Sirainen <tss@iki.fi>
parents: 4368
diff changeset
214 }
Timo Sirainen <tss@iki.fi>
parents: 4368
diff changeset
215
4368
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
216 if (o_stream_send_str(dict->output, query) < 0 ||
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
217 o_stream_flush(dict->output) < 0) {
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
218 /* Send failed. Our transactions have died, so don't even try
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
219 to re-send the command */
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
220 ctx->failed = TRUE;
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
221 client_dict_disconnect(dict);
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
222 return -1;
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
223 }
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
224 return 0;
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
225 }
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
226
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
227 static struct client_dict_transaction_context *
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
228 client_dict_transaction_find(struct client_dict *dict, unsigned int id)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
229 {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
230 struct client_dict_transaction_context *ctx;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
231
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
232 for (ctx = dict->transactions; ctx != NULL; ctx = ctx->next) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
233 if (ctx->id == id)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
234 return ctx;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
235 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
236 return NULL;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
237 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
238
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
239 static void
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
240 client_dict_finish_transaction(struct client_dict *dict,
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
241 unsigned int id, int ret)
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
242 {
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
243 struct client_dict_transaction_context *ctx;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
244
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
245 ctx = client_dict_transaction_find(dict, id);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
246 if (ctx == NULL) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
247 i_error("dict-client: Unknown transaction id %u", id);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
248 return;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
249 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
250 if (ctx->callback != NULL)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
251 ctx->callback(ret, ctx->context);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
252
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
253 DLLIST_REMOVE(&dict->transactions, ctx);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
254 i_free(ctx);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
255
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
256 i_assert(dict->async_commits > 0);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
257 if (--dict->async_commits == 0)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
258 io_remove(&dict->io);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
259 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
260
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
261 static int client_dict_read_one_line(struct client_dict *dict, char **line_r)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
262 {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
263 unsigned int id;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
264 char *line;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
265 int ret;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
266
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
267 *line_r = NULL;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
268 while ((line = i_stream_next_line(dict->input)) == NULL) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
269 ret = i_stream_read(dict->input);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
270 switch (ret) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
271 case -1:
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
272 if (dict->input->stream_errno != 0)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
273 i_error("read(%s) failed: %m", dict->path);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
274 else {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
275 i_error("read(%s) failed: Remote disconnected",
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
276 dict->path);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
277 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
278 return -1;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
279 case -2:
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
280 i_error("read(%s) returned too much data", dict->path);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
281 return -1;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
282 default:
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
283 i_assert(ret > 0);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
284 break;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
285 }
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
286 }
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
287 if (*line == DICT_PROTOCOL_REPLY_ASYNC_COMMIT) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
288 switch (line[1]) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
289 case DICT_PROTOCOL_REPLY_OK:
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
290 ret = 1;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
291 break;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
292 case DICT_PROTOCOL_REPLY_NOTFOUND:
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
293 ret = 0;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
294 break;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
295 case DICT_PROTOCOL_REPLY_FAIL:
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
296 ret = -1;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
297 break;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
298 default:
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
299 i_error("dict-client: Invalid async commit line: %s",
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
300 line);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
301 return 0;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
302 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
303 id = strtoul(line+2, NULL, 10);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
304 client_dict_finish_transaction(dict, id, ret);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
305 return 0;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
306 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
307 *line_r = line;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
308 return 1;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
309 }
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
310
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
311 static char *client_dict_read_line(struct client_dict *dict)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
312 {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
313 char *line;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
314
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
315 while (client_dict_read_one_line(dict, &line) == 0)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
316 ;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
317 return line;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
318 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
319
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
320 static int client_dict_connect(struct client_dict *dict)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
321 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
322 const char *query;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
323
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
324 i_assert(dict->fd == -1);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
325
7008
6950bb5e7921 Don't try to reconnect more often than once/sec.
Timo Sirainen <tss@iki.fi>
parents: 7007
diff changeset
326 if (dict->last_connect_try == ioloop_time) {
6950bb5e7921 Don't try to reconnect more often than once/sec.
Timo Sirainen <tss@iki.fi>
parents: 7007
diff changeset
327 /* Try again later */
6950bb5e7921 Don't try to reconnect more often than once/sec.
Timo Sirainen <tss@iki.fi>
parents: 7007
diff changeset
328 return -1;
6950bb5e7921 Don't try to reconnect more often than once/sec.
Timo Sirainen <tss@iki.fi>
parents: 7007
diff changeset
329 }
6950bb5e7921 Don't try to reconnect more often than once/sec.
Timo Sirainen <tss@iki.fi>
parents: 7007
diff changeset
330 dict->last_connect_try = ioloop_time;
6950bb5e7921 Don't try to reconnect more often than once/sec.
Timo Sirainen <tss@iki.fi>
parents: 7007
diff changeset
331
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
332 dict->fd = net_connect_unix(dict->path);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
333 if (dict->fd == -1) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
334 i_error("net_connect_unix(%s) failed: %m", dict->path);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
335 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
336 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
337
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
338 /* Dictionary lookups are blocking */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
339 net_set_nonblock(dict->fd, FALSE);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
340
6162
896cc473c1f0 Renamed i_stream_create_file() to i_stream_create_fd().
Timo Sirainen <tss@iki.fi>
parents: 6161
diff changeset
341 dict->input = i_stream_create_fd(dict->fd, (size_t)-1, FALSE);
7969
2cd775b45b0c Set dict proxy istream nonblocking so EINTR (^C) won't crash.
Timo Sirainen <tss@iki.fi>
parents: 7418
diff changeset
342 dict->input->blocking = TRUE;
6161
c62f7ee79446 Split o_stream_create_file() to _create_fd() and _create_fd_file().
Timo Sirainen <tss@iki.fi>
parents: 6142
diff changeset
343 dict->output = o_stream_create_fd(dict->fd, 4096, FALSE);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
344 dict->transaction_id_counter = 0;
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
345 dict->async_commits = 0;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
346
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
347 query = t_strdup_printf("%c%u\t%u\t%d\t%s\t%s\n",
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
348 DICT_PROTOCOL_CMD_HELLO,
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
349 DICT_CLIENT_PROTOCOL_MAJOR_VERSION,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
350 DICT_CLIENT_PROTOCOL_MINOR_VERSION,
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
351 dict->value_type, dict->username, dict->uri);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
352 if (client_dict_send_query(dict, query) < 0) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
353 client_dict_disconnect(dict);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
354 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
355 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
356
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
357 dict->handshaked = TRUE;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
358 return 0;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
359 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
360
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
361 static void client_dict_disconnect(struct client_dict *dict)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
362 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
363 dict->connect_counter++;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
364 dict->handshaked = FALSE;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
365
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
366 if (dict->io != NULL)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
367 io_remove(&dict->io);
3879
928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to
Timo Sirainen <tss@iki.fi>
parents: 3863
diff changeset
368 if (dict->input != NULL)
4070
71b8faa84ec6 Added i_stream_destroy() and o_stream_destroy() and used them instead of
Timo Sirainen <tss@iki.fi>
parents: 3990
diff changeset
369 i_stream_destroy(&dict->input);
3879
928229f8b3e6 deinit, unref, destroy, close, free, etc. functions now take a pointer to
Timo Sirainen <tss@iki.fi>
parents: 3863
diff changeset
370 if (dict->output != NULL)
4070
71b8faa84ec6 Added i_stream_destroy() and o_stream_destroy() and used them instead of
Timo Sirainen <tss@iki.fi>
parents: 3990
diff changeset
371 o_stream_destroy(&dict->output);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
372
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
373 if (dict->fd != -1) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
374 if (close(dict->fd) < 0)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
375 i_error("close(%s) failed: %m", dict->path);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
376 dict->fd = -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
377 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
378 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
379
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
380 static struct dict *
4517
e661182eab75 Berkeley DB dict support is now enabled only when using --with-db configure option.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4516
diff changeset
381 client_dict_init(struct dict *driver, const char *uri,
9174
eed86bcc33aa dict proxy: Use base_dir as the default dict-server location.
Timo Sirainen <tss@iki.fi>
parents: 8683
diff changeset
382 enum dict_data_type value_type, const char *username,
eed86bcc33aa dict proxy: Use base_dir as the default dict-server location.
Timo Sirainen <tss@iki.fi>
parents: 8683
diff changeset
383 const char *base_dir)
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
384 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
385 struct client_dict *dict;
3967
6fabe878c46d Dictionary takes now a username parameter, which is used for private
Timo Sirainen <tss@iki.fi>
parents: 3879
diff changeset
386 const char *dest_uri;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
387 pool_t pool;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
388
3967
6fabe878c46d Dictionary takes now a username parameter, which is used for private
Timo Sirainen <tss@iki.fi>
parents: 3879
diff changeset
389 /* uri = [<path>] ":" <uri> */
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
390 dest_uri = strchr(uri, ':');
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
391 if (dest_uri == NULL) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
392 i_error("dict-client: Invalid URI: %s", uri);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
393 return NULL;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
394 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
395
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
396 pool = pool_alloconly_create("client dict", 1024);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
397 dict = p_new(pool, struct client_dict, 1);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
398 dict->pool = pool;
4517
e661182eab75 Berkeley DB dict support is now enabled only when using --with-db configure option.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4516
diff changeset
399 dict->dict = *driver;
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
400 dict->value_type = value_type;
3967
6fabe878c46d Dictionary takes now a username parameter, which is used for private
Timo Sirainen <tss@iki.fi>
parents: 3879
diff changeset
401 dict->username = p_strdup(pool, username);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
402
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
403 dict->fd = -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
404
3967
6fabe878c46d Dictionary takes now a username parameter, which is used for private
Timo Sirainen <tss@iki.fi>
parents: 3879
diff changeset
405 if (*uri != ':') {
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
406 /* path given */
3967
6fabe878c46d Dictionary takes now a username parameter, which is used for private
Timo Sirainen <tss@iki.fi>
parents: 3879
diff changeset
407 dict->path = p_strdup_until(pool, uri, dest_uri);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
408 } else {
9174
eed86bcc33aa dict proxy: Use base_dir as the default dict-server location.
Timo Sirainen <tss@iki.fi>
parents: 8683
diff changeset
409 dict->path = p_strconcat(pool, base_dir,
eed86bcc33aa dict proxy: Use base_dir as the default dict-server location.
Timo Sirainen <tss@iki.fi>
parents: 8683
diff changeset
410 "/"DEFAULT_DICT_SERVER_SOCKET_FNAME, NULL);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
411 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
412 dict->uri = p_strdup(pool, dest_uri + 1);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
413 return &dict->dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
414 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
415
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
416 static void client_dict_deinit(struct dict *_dict)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
417 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
418 struct client_dict *dict = (struct client_dict *)_dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
419
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
420 client_dict_disconnect(dict);
6428
7cad076906eb pool_unref() now takes ** pointer.
Timo Sirainen <tss@iki.fi>
parents: 6162
diff changeset
421 pool_unref(&dict->pool);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
422 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
423
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
424 static int client_dict_wait(struct dict *_dict)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
425 {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
426 struct client_dict *dict = (struct client_dict *)_dict;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
427 char *line;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
428 int ret = 0;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
429
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
430 while (dict->async_commits > 0) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
431 if (client_dict_read_one_line(dict, &line) < 0) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
432 ret = -1;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
433 break;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
434 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
435 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
436 return ret;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
437 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
438
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
439 static int client_dict_lookup(struct dict *_dict, pool_t pool,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
440 const char *key, const char **value_r)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
441 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
442 struct client_dict *dict = (struct client_dict *)_dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
443 const char *line;
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
444 int ret;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
445
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
446 T_BEGIN {
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
447 const char *query;
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
448
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
449 query = t_strdup_printf("%c%s\n", DICT_PROTOCOL_CMD_LOOKUP,
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
450 dict_client_escape(key));
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
451 ret = client_dict_send_query(dict, query);
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
452 } T_END;
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
453 if (ret < 0)
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
454 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
455
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
456 /* read reply */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
457 line = client_dict_read_line(dict);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
458 if (line == NULL)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
459 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
460
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
461 if (*line == DICT_PROTOCOL_REPLY_OK) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
462 *value_r = p_strdup(pool, dict_client_unescape(line + 1));
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
463 return 1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
464 } else {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
465 *value_r = NULL;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
466 return *line == DICT_PROTOCOL_REPLY_NOTFOUND ? 0 : -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
467 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
468 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
469
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
470 static struct dict_iterate_context *
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
471 client_dict_iterate_init(struct dict *_dict, const char *path,
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
472 enum dict_iterate_flags flags)
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
473 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
474 struct client_dict *dict = (struct client_dict *)_dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
475 struct client_dict_iterate_context *ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
476
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
477 if (dict->in_iteration)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
478 i_panic("dict-client: Only one iteration supported");
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
479 dict->in_iteration = TRUE;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
480
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
481 ctx = i_new(struct client_dict_iterate_context, 1);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
482 ctx->ctx.dict = _dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
483 ctx->pool = pool_alloconly_create("client dict iteration", 512);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
484
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
485 T_BEGIN {
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
486 const char *query;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
487
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
488 query = t_strdup_printf("%c%d\t%s\n", DICT_PROTOCOL_CMD_ITERATE,
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
489 flags, dict_client_escape(path));
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
490 if (client_dict_send_query(dict, query) < 0)
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
491 ctx->failed = TRUE;
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
492 } T_END;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
493 return &ctx->ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
494 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
495
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
496 static int client_dict_iterate(struct dict_iterate_context *_ctx,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
497 const char **key_r, const char **value_r)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
498 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
499 struct client_dict_iterate_context *ctx =
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
500 (struct client_dict_iterate_context *)_ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
501 struct client_dict *dict = (struct client_dict *)_ctx->dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
502 char *line, *value;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
503
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
504 if (ctx->failed)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
505 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
506
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
507 /* read next reply */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
508 line = client_dict_read_line(dict);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
509 if (line == NULL)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
510 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
511
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
512 if (*line == '\0') {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
513 /* end of iteration */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
514 return 0;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
515 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
516
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
517 /* line contains key \t value */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
518 p_clear(ctx->pool);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
519
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
520 if (*line != DICT_PROTOCOL_REPLY_OK)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
521 value = NULL;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
522 else
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
523 value = strchr(++line, '\t');
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
524 if (value == NULL) {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
525 /* broken protocol */
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
526 i_error("dict client (%s) sent broken reply", dict->path);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
527 return -1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
528 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
529 *value++ = '\0';
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
530
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
531 *key_r = p_strdup(ctx->pool, dict_client_unescape(line));
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
532 *value_r = p_strdup(ctx->pool, dict_client_unescape(value));
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
533 return 1;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
534 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
535
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
536 static void client_dict_iterate_deinit(struct dict_iterate_context *_ctx)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
537 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
538 struct client_dict *dict = (struct client_dict *)_ctx->dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
539 struct client_dict_iterate_context *ctx =
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
540 (struct client_dict_iterate_context *)_ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
541
6428
7cad076906eb pool_unref() now takes ** pointer.
Timo Sirainen <tss@iki.fi>
parents: 6162
diff changeset
542 pool_unref(&ctx->pool);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
543 i_free(ctx);
8105
ad9ad222729c Calling dict_iterate_init() multiple times for a dict assert-crashed.
Timo Sirainen <tss@iki.fi>
parents: 7988
diff changeset
544 dict->in_iteration = FALSE;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
545 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
546
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
547 static struct dict_transaction_context *
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
548 client_dict_transaction_init(struct dict *_dict)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
549 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
550 struct client_dict *dict = (struct client_dict *)_dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
551 struct client_dict_transaction_context *ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
552
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
553 ctx = i_new(struct client_dict_transaction_context, 1);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
554 ctx->ctx.dict = _dict;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
555 ctx->id = ++dict->transaction_id_counter;
4368
51a1d0fbbc94 Fixes to dict proxy
Timo Sirainen <tss@iki.fi>
parents: 4367
diff changeset
556
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
557 DLLIST_PREPEND(&dict->transactions, ctx);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
558 return &ctx->ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
559 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
560
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
561 static void dict_async_input(struct client_dict *dict)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
562 {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
563 char *line;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
564 size_t size;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
565 int ret;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
566
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
567 i_assert(!dict->in_iteration);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
568
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
569 do {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
570 ret = client_dict_read_one_line(dict, &line);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
571 (void)i_stream_get_data(dict->input, &size);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
572 } while (ret == 0 && size > 0);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
573
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
574 if (ret < 0)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
575 io_remove(&dict->io);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
576 }
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
577
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
578 static int
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
579 client_dict_transaction_commit(struct dict_transaction_context *_ctx,
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
580 bool async,
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
581 dict_transaction_commit_callback_t *callback,
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
582 void *context)
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
583 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
584 struct client_dict_transaction_context *ctx =
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
585 (struct client_dict_transaction_context *)_ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
586 struct client_dict *dict = (struct client_dict *)_ctx->dict;
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
587 int ret = ctx->failed ? -1 : 1;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
588
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
589 if (ctx->sent_begin && !ctx->failed) T_BEGIN {
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
590 const char *query, *line;
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
591
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
592 query = t_strdup_printf("%c%u\n", !async ?
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
593 DICT_PROTOCOL_CMD_COMMIT :
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
594 DICT_PROTOCOL_CMD_COMMIT_ASYNC,
8664
446775a31754 dict proxy: Handle async commits better.
Timo Sirainen <tss@iki.fi>
parents: 8660
diff changeset
595 ctx->id);
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
596 if (client_dict_send_transaction_query(ctx, query) < 0)
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
597 ret = -1;
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
598 else if (async) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
599 ctx->callback = callback;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
600 ctx->context = context;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
601 if (dict->async_commits++ == 0) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
602 dict->io = io_add(dict->fd, IO_READ,
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
603 dict_async_input, dict);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
604 }
8660
d8a56ea9f408 Added dict_transaction_commit_async().
Timo Sirainen <tss@iki.fi>
parents: 8590
diff changeset
605 } else {
8683
a68ed51b681d dict proxy client: Don't hang when doing an async commit.
Timo Sirainen <tss@iki.fi>
parents: 8664
diff changeset
606 /* sync commit, read reply */
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
607 line = client_dict_read_line(dict);
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
608 if (line == NULL)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
609 ret = -1;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
610 else if (*line == DICT_PROTOCOL_REPLY_OK)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
611 ret = 1;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
612 else if (*line == DICT_PROTOCOL_REPLY_NOTFOUND)
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
613 ret = 0;
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
614 else
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
615 ret = -1;
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
616 }
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
617 } T_END;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
618
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
619 if (ret < 0 || !async) {
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
620 DLLIST_REMOVE(&dict->transactions, ctx);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
621 i_free(ctx);
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
622 }
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
623 return ret;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
624 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
625
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
626 static void
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
627 client_dict_transaction_rollback(struct dict_transaction_context *_ctx)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
628 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
629 struct client_dict_transaction_context *ctx =
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
630 (struct client_dict_transaction_context *)_ctx;
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
631 struct client_dict *dict = (struct client_dict *)_ctx->dict;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
632
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
633 if (ctx->sent_begin) T_BEGIN {
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
634 const char *query;
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
635
4512
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
636 query = t_strdup_printf("%c%u\n", DICT_PROTOCOL_CMD_ROLLBACK,
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
637 ctx->id);
b5d4c1e9a492 Delay sending the transaction begin command to server until the first
Timo Sirainen <tss@iki.fi>
parents: 4385
diff changeset
638 (void)client_dict_send_transaction_query(ctx, query);
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
639 } T_END;
4385
Timo Sirainen <tss@iki.fi>
parents: 4368
diff changeset
640
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
641 DLLIST_REMOVE(&dict->transactions, ctx);
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
642 i_free(ctx);
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
643 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
644
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
645 static void client_dict_set(struct dict_transaction_context *_ctx,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
646 const char *key, const char *value)
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
647 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
648 struct client_dict_transaction_context *ctx =
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
649 (struct client_dict_transaction_context *)_ctx;
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
650
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
651 T_BEGIN {
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
652 const char *query;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
653
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
654 query = t_strdup_printf("%c%u\t%s\t%s\n",
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
655 DICT_PROTOCOL_CMD_SET, ctx->id,
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
656 dict_client_escape(key),
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
657 dict_client_escape(value));
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
658 (void)client_dict_send_transaction_query(ctx, query);
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
659 } T_END;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
660 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
661
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
662 static void client_dict_unset(struct dict_transaction_context *_ctx,
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
663 const char *key)
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
664 {
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
665 struct client_dict_transaction_context *ctx =
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
666 (struct client_dict_transaction_context *)_ctx;
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
667
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
668 T_BEGIN {
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
669 const char *query;
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
670
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
671 query = t_strdup_printf("%c%u\t%s\n",
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
672 DICT_PROTOCOL_CMD_UNSET, ctx->id,
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
673 dict_client_escape(key));
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
674 (void)client_dict_send_transaction_query(ctx, query);
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
675 } T_END;
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
676 }
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
677
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
678 static void client_dict_atomic_inc(struct dict_transaction_context *_ctx,
3990
e2e6919c6c4d LF wasn't sent at the end of all commands.
Timo Sirainen <tss@iki.fi>
parents: 3967
diff changeset
679 const char *key, long long diff)
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
680 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
681 struct client_dict_transaction_context *ctx =
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
682 (struct client_dict_transaction_context *)_ctx;
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
683
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
684 T_BEGIN {
6940
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
685 const char *query;
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
686 query = t_strdup_printf("%c%u\t%s\t%lld\n",
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
687 DICT_PROTOCOL_CMD_ATOMIC_INC,
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
688 ctx->id, dict_client_escape(key), diff);
414c9d631a81 Replaced t_push/t_pop calls with T_FRAME*() macros.
Timo Sirainen <tss@iki.fi>
parents: 6525
diff changeset
689 (void)client_dict_send_transaction_query(ctx, query);
7226
e6693a0ec8e1 Renamed T_FRAME_BEGIN/END to T_BEGIN/END. Removed T_FRAME() macro and
Timo Sirainen <tss@iki.fi>
parents: 7086
diff changeset
690 } T_END;
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
691 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
692
4517
e661182eab75 Berkeley DB dict support is now enabled only when using --with-db configure option.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4516
diff changeset
693 struct dict dict_driver_client = {
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
694 MEMBER(name) "proxy",
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
695
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
696 {
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
697 client_dict_init,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
698 client_dict_deinit,
9361
a1b92a251bb9 dict: Added support for async commits. Changed dict_atomic_inc() behavior.
Timo Sirainen <tss@iki.fi>
parents: 9174
diff changeset
699 client_dict_wait,
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
700 client_dict_lookup,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
701 client_dict_iterate_init,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
702 client_dict_iterate,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
703 client_dict_iterate_deinit,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
704 client_dict_transaction_init,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
705 client_dict_transaction_commit,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
706 client_dict_transaction_rollback,
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
707 client_dict_set,
4516
aa2f73a4df26 Dictionary changes: Added support for defining value's type. Key is still always a string. Added support for sorting the iteration replies. Added dict_unset(). Added Berkeley DB support. Most of the code written by Tianyan Liu.
Timo Sirainen <timo.sirainen@movial.fi>
parents: 4512
diff changeset
708 client_dict_unset,
3793
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
709 client_dict_atomic_inc
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
710 }
0b3f3323e460 Forgot to add
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
711 };