*** jeromatron has joined #magnetodb | 00:16 | |
*** jeromatron has quit IRC | 00:29 | |
*** jeromatron has joined #magnetodb | 00:38 | |
*** jeromatron has quit IRC | 00:40 | |
*** bogeyon18 has joined #magnetodb | 01:17 | |
*** charlesw has joined #magnetodb | 01:37 | |
*** jeromatron has joined #magnetodb | 02:24 | |
*** jeromatron has quit IRC | 02:27 | |
*** rushiagr_away is now known as rushiagr | 03:07 | |
*** rushiagr is now known as rushiagr_away | 03:28 | |
*** charlesw has quit IRC | 03:46 | |
*** rushiagr_away is now known as rushiagr | 04:12 | |
*** bogeyon18 has quit IRC | 04:26 | |
*** jeromatron has joined #magnetodb | 04:52 | |
*** jeromatron has quit IRC | 04:54 | |
*** jeromatron has joined #magnetodb | 05:56 | |
*** ajayaa has joined #magnetodb | 06:03 | |
*** jeromatron has quit IRC | 06:29 | |
*** k4n0 has joined #magnetodb | 06:45 | |
*** ajayaa has quit IRC | 06:57 | |
*** ajayaa has joined #magnetodb | 07:24 | |
rushiagr | I'm trying conditional put, but I'm getting a malformed request | 08:21 |
---|---|---|
rushiagr | here is the JSON I'm passing | 08:21 |
rushiagr | http://paste.openstack.org/show/110550/ | 08:21 |
rushiagr | I've tried EQ instead of equal, but still same error | 08:22 |
rushiagr | can someone tell if I'm using wrong syntax or something? | 08:22 |
rushiagr | oh, ok, wait. I catched it | 08:22 |
rushiagr | nope, I think I've written the JSON correctly | 08:23 |
rushiagr | can anyone help? | 08:23 |
rushiagr | there is already an item with the same attributes | 08:24 |
ikhudoshyn_ | hi rushiagr | 08:45 |
rushiagr | ikhudoshyn_: hi | 08:47 |
ikhudoshyn_ | "conditional_operator" part in ur request is excess part | 08:47 |
ikhudoshyn_ | it's always 'AND' implicitly | 08:47 |
ikhudoshyn_ | 'comparison_operator' for put_item is redundant as well | 08:48 |
ikhudoshyn_ | i's always 'EQUAL' unless you use 'if not exists' semantics | 08:49 |
ikhudoshyn_ | for 'expected' part pls use 'value', not 'attribute_value_list' | 08:51 |
ikhudoshyn_ | oops,.. I assume you use openstack API, not AWS onw | 08:53 |
*** rushiagr is now known as rushiagr_away | 09:01 | |
*** rushiagr_away is now known as rushiagr | 09:13 | |
rushiagr | ikhudoshyn_: hey, thanks | 09:15 |
rushiagr | I'll try them now and let you know | 09:15 |
ikhudoshyn_ | tnx | 09:16 |
rushiagr | ikhudoshyn_: it now says HTTP 400 error | 09:21 |
rushiagr | message": "Wrong 'value' type. Object is expected, but [{\"S\": \"si\"}] found", | 09:21 |
rushiagr | here also API is changed? | 09:21 |
openstackgerrit | Dmitriy Ukhlov proposed a change to stackforge/magnetodb: Use unlogged batches instead of atomic batches https://review.openstack.org/120801 | 09:21 |
*** ikhudoshyn_ has quit IRC | 09:23 | |
*** ikhudoshyn has joined #magnetodb | 09:25 | |
ikhudoshyn | rushiagr, could you pls share current request | 09:26 |
ikhudoshyn | ? | 09:26 |
rushiagr | ikhudoshyn: sure | 09:26 |
rushiagr | ikhudoshyn: http://paste.openstack.org/show/110566/ | 09:27 |
ikhudoshyn | pls try removing brackets [] after 'value' | 09:29 |
ikhudoshyn | it should be in form "value" : {"S": "strval"} | 09:29 |
rushiagr | ikhudoshyn: thanks | 09:33 |
rushiagr | it passed now | 09:33 |
rushiagr | ikhudoshyn: thanks :) | 09:33 |
ikhudoshyn | u r welcome | 09:34 |
rushiagr | ikhudoshyn: what all operators are supported other than EQUAL ? | 09:34 |
ikhudoshyn | for 'put_item' only two options: 'EQUALS' and 'exists' : true/falce | 09:37 |
ikhudoshyn | false | 09:37 |
ikhudoshyn | https://wiki.openstack.org/wiki/MagnetoDB/specs/data-api#Putitem | 09:37 |
rushiagr | ikhudoshyn: ohkay | 09:37 |
rushiagr | ikhudoshyn: is this wiki page up-to-date? | 09:38 |
ikhudoshyn | more or less, we're trying to keep it so | 09:38 |
rushiagr | ikhudoshyn: if yes, I'll stop my hit-and-miss business :) | 09:38 |
rushiagr | ikhudoshyn: okay. Thanks | 09:39 |
ikhudoshyn | basically we hope we stabilized the API, but... | 09:39 |
rushiagr | ikhudoshyn: okay | 09:40 |
isviridov | rushiagr, ikhudoshyn it is a spec and won't be updated after implementation. The documentation we are releasing is here http://magnetodb.readthedocs.org/en/latest/put_item.html | 09:40 |
rushiagr | isviridov: cool! | 09:41 |
rushiagr | isviridov: thanks for that | 09:42 |
rushiagr | isviridov: very helpful | 09:42 |
rushiagr | isviridov: how is the unit and functional test coverage for these APIs? I'm sorry, I'm finding it difficult to judge from the code, so taking the easier route -- asking you :) | 09:42 |
isviridov | rushiagr, there is no regular test coverage checking process right now. We are following the rule 'anything implemented must be covered by unit and ffunctional (tempest) test' | 09:45 |
rushiagr | isviridov: okay, thanks. That answers my question | 09:45 |
isviridov | rushiagr, ATM there is no dedicated QA effort | 09:46 |
rushiagr | isviridov: okay | 09:46 |
rushiagr | rushiagr: should I update the wiki API reference to say it is now superseded by docs at readthedocs? | 09:49 |
isviridov | rushiagr, yes sure | 09:51 |
rushiagr | done | 09:53 |
rushiagr | :) | 09:53 |
isviridov | gr8! | 10:04 |
rushiagr | I am assuming only EQ is supported comparison operator for 'query' API | 10:05 |
rushiagr | I think we should start making a list somewhere as to what operators are supported, and what are not | 10:05 |
rushiagr | or atleast add it to the docs, so that it is easily referencible | 10:06 |
isviridov | rushiagr, I agree. | 10:16 |
rushiagr | I see that 'ConditionalOperator' equivalent is not present for 'scan' API in the docs | 10:18 |
rushiagr | does that mean that currently only "AND" conditional operator is supported? | 10:18 |
rushiagr | or does that mean that the doc is outdated :) | 10:19 |
ikhudoshyn | conditional operator you write, AND, is the only supported boolean operationthat joins conditionals on separate attributes | 10:25 |
rushiagr | ikhudoshyn: okay, thanks | 10:26 |
ikhudoshyn | a list of conditionals for separate attibutes is supported | 10:26 |
ikhudoshyn | like attr1 equals "a" AND attr2 begins_with "b" AND attr2 LESS 42 | 10:26 |
rushiagr | ikhudoshyn: okay | 10:27 |
ikhudoshyn | for query list of conditionals is rather bounded due to the nature of 'query' | 10:27 |
ikhudoshyn | for scan that list of conditionals is much richer, but scan itself could be much slower | 10:28 |
rushiagr | ikhudoshyn: I see | 10:28 |
*** rushiagr is now known as rushiagr_away | 10:56 | |
openstackgerrit | A change was merged to stackforge/magnetodb: Use unlogged batches instead of atomic batches https://review.openstack.org/120801 | 11:37 |
*** rushiagr_away is now known as rushiagr | 11:58 | |
*** rushiagr is now known as rushiagr_away | 12:05 | |
*** rushiagr_away is now known as rushiagr | 12:12 | |
openstackgerrit | Andrei V. Ostapenko proposed a change to stackforge/magnetodb: Fixes bug with scan on hash key attribute https://review.openstack.org/120654 | 12:30 |
isviridov | ajayaa, please rebase you patches, there is a fix of devstack gate. Should work now. | 12:32 |
ajayaa | isviridov, rebase or recheck? | 12:34 |
isviridov | ajayaa, if you add this change to your code https://review.openstack.org/#/c/120801/ the gate probably pass | 12:35 |
isviridov | ajayaa, rebase | 12:35 |
ajayaa | isviridov, okay. Thanks | 12:35 |
*** aostapenko_away is now known as aostapenko | 12:37 | |
openstackgerrit | Ajaya Agrawal proposed a change to stackforge/magnetodb: Correct error message on creating duplicate table https://review.openstack.org/120744 | 12:38 |
openstackgerrit | Ajaya Agrawal proposed a change to stackforge/magnetodb: Added debug env to tox https://review.openstack.org/120873 | 12:47 |
*** SpyRay has joined #magnetodb | 12:52 | |
SpyRay | Hi all! | 12:52 |
isviridov | SpyRay, wellcome back | 12:55 |
isviridov | Any news about devstack behind the proxy problem? | 12:55 |
SpyRay | I finish writing the documentation for the solution of this issue | 12:57 |
isviridov | SpyRay, what was the problem? | 12:57 |
SpyRay | https://bugs.launchpad.net/magnetodb/+bug/1366788 | 12:59 |
isviridov | SpyRay, :) I know the bug. Have have you fixed it? | 13:02 |
isviridov | *How | 13:02 |
SpyRay | this is no bug, it is necessary to correctly identify some of the variables | 13:10 |
isviridov | SpyRay, do you mean export proxy settings? | 13:11 |
SpyRay | Sure | 13:11 |
SpyRay | and not only | 13:12 |
isviridov | What else? | 13:12 |
SpyRay | for git and curl also | 13:12 |
isviridov | SpyRay, could you please describe it in our dev guide? | 13:13 |
SpyRay | and for ANT if necessary | 13:13 |
SpyRay | yes | 13:14 |
isviridov | Looking forward for your patch | 13:14 |
ajayaa | isviridov, suddenly in every request to mdb, I am seeing this 500 error. | 13:18 |
ajayaa | http://paste.openstack.org/show/110643/ | 13:19 |
*** keith_newstadt has joined #magnetodb | 13:23 | |
isviridov | ajayaa, can't check right now, have a meeting soon | 13:23 |
ajayaa | isviridov, np. I am trying! :) | 13:24 |
*** charlesw has joined #magnetodb | 13:24 | |
dukhlov | ajayaa, it could be when you created table using old magnetodb code and now using new magnetodb code | 13:26 |
dukhlov | try to clean cassandra keyspaces manually | 13:27 |
dukhlov | and then create table | 13:27 |
openstackgerrit | Dmitriy Ukhlov proposed a change to stackforge/magnetodb: Update cassandra version to 2.0.10 https://review.openstack.org/119411 | 13:29 |
ajayaa | dukhlov, works. | 13:37 |
ajayaa | Thanks | 13:37 |
*** keith_newstadt has quit IRC | 14:17 | |
aostapenko | https://wiki.openstack.org/wiki/MagnetoDB/specs/all_old_for_put_item | 14:18 |
rushiagr | SpyRay: isviridov: the "MDB behind proxy" guide will be very helpful | 14:22 |
isviridov | aostapenko, could you please leave all the original structure and it is described in template | 14:26 |
isviridov | ? | 14:26 |
openstackgerrit | Alexei Vinogradov proposed a change to stackforge/magnetodb: Added documentation for installation devstack and magnetodb behind a proxy https://review.openstack.org/121136 | 14:39 |
aostapenko | isviridov: sure | 14:40 |
openstackgerrit | Alexei Vinogradov proposed a change to stackforge/magnetodb: Added documentation for installation devstack and magnetodb behind a proxy https://review.openstack.org/121136 | 14:41 |
*** bogeyon18 has joined #magnetodb | 14:48 | |
ajayaa | Should the creation_time of table be inserted into magnetodb.table_info? | 15:01 |
ajayaa | This is an attribute we return when a table is either created or deleted. | 15:01 |
*** bogeyon18 has quit IRC | 15:15 | |
openstackgerrit | Alexei Vinogradov proposed a change to stackforge/magnetodb: Added documentation for installation magnetodb behind a proxy https://review.openstack.org/121136 | 15:15 |
*** bogeyon18 has joined #magnetodb | 15:32 | |
aostapenko | isviridov: could you approve bp please? | 15:33 |
isviridov | aostapenko, let me check | 15:39 |
*** ajayaa has quit IRC | 15:51 | |
*** rushiagr is now known as rushiagr_away | 15:52 | |
isviridov | dukhlov, ikhudoshyn please look at https://wiki.openstack.org/wiki/MagnetoDB/specs/all_old_for_put_item | 15:54 |
dukhlov | isviridov: LGTM | 15:59 |
isviridov | aostapenko, looks clear for me | 15:59 |
isviridov | dukhlov, great! Approved | 15:59 |
*** bogeyon18 has quit IRC | 15:59 | |
*** rushiagr_away is now known as rushiagr | 16:12 | |
openstackgerrit | Dmitriy Ukhlov proposed a change to stackforge/magnetodb: Improve json schema validation https://review.openstack.org/121162 | 16:17 |
*** k4n0 has quit IRC | 16:40 | |
*** jeromatron has joined #magnetodb | 16:57 | |
*** jeromatron has quit IRC | 17:06 | |
*** bogeyon18 has joined #magnetodb | 17:28 | |
*** rushiagr is now known as rushiagr_away | 17:45 | |
*** bogeyon18 has quit IRC | 18:56 | |
*** bogeyon18 has joined #magnetodb | 18:56 | |
*** charlesw has quit IRC | 18:57 | |
*** jeromatron has joined #magnetodb | 20:15 | |
*** charlesw has joined #magnetodb | 20:17 | |
*** jeromatron has quit IRC | 21:32 | |
*** aostapenko has quit IRC | 21:45 | |
*** aostapenko has joined #magnetodb | 21:46 | |
*** jeromatron has joined #magnetodb | 21:47 | |
*** igormarnat has quit IRC | 22:18 | |
*** boris-42_ has joined #magnetodb | 22:22 | |
*** openstack has joined #magnetodb | 23:37 | |
*** sendak.freenode.net sets mode: +ns | 23:37 | |
*** sendak.freenode.net sets mode: -o openstack | 23:42 | |
-sendak.freenode.net- *** Notice -- TS for #magnetodb changed from 1410565072 to 1385052681 | 23:42 | |
*** sendak.freenode.net sets mode: +ct-s | 23:42 | |
*** jeromatron has joined #magnetodb | 23:42 | |
*** igormarnat has joined #magnetodb | 23:42 | |
*** openstackgerrit has joined #magnetodb | 23:42 | |
*** ominakov- has joined #magnetodb | 23:42 | |
*** idegtiarov_ has joined #magnetodb | 23:42 | |
*** ikhudoshyn_ has joined #magnetodb | 23:42 | |
*** achuprin_ has joined #magnetodb | 23:42 | |
*** boris-42 has joined #magnetodb | 23:42 | |
*** aostapenko has joined #magnetodb | 23:42 | |
*** rushiagr_away has joined #magnetodb | 23:42 | |
*** dukhlov has joined #magnetodb | 23:42 | |
*** isviridov has joined #magnetodb | 23:42 | |
*** [o__o] has joined #magnetodb | 23:42 | |
*** anteaya has joined #magnetodb | 23:42 | |
*** achudnovets has joined #magnetodb | 23:42 | |
*** ChanServ has joined #magnetodb | 23:42 | |
*** sendak.freenode.net sets mode: +o ChanServ | 23:42 | |
*** sendak.freenode.net changes topic to "MagnetoDB - key-value database service for OpenStack (https://wiki.openstack.org/wiki/MagnetoDB, logs @ https://botbot.me/freenode/magnetodb/ and http://eavesdrop.openstack.org/irclogs/%23magnetodb/) | More info in our screencast http://goo.gl/21HRCf" | 23:42 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!