[GitHub] couchdb-fauxton pull request: Test for error handling in Active Ta...

2015-04-21 Thread michellephung
Github user michellephung commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/388#issuecomment-94946079 merged 431e164b497ba5b21536b85c2654d6279d77ed02 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] couchdb-fauxton pull request: Test for error handling in Active Ta...

2015-04-21 Thread michellephung
Github user michellephung closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/388 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] couchdb-fauxton pull request: Updating Auth module to use React

2015-04-21 Thread benkeen
Github user benkeen commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/390#discussion_r28822513 --- Diff: app/addons/auth/actions.js --- @@ -0,0 +1,116 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may not +/

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread michellephung
Github user michellephung commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28821282 --- Diff: app/addons/databases/actions.js --- @@ -0,0 +1,105 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may

[jira] [Commented] (COUCHDB-2657) if _metadata is there, creating a db does not work

2015-04-21 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/COUCHDB-2657?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14505576#comment-14505576 ] ASF subversion and git services commented on COUCHDB-2657: -- Com

[GitHub] couchdb-couch pull request: Fix error handling in couch_util:with_...

2015-04-21 Thread chewbranca
Github user chewbranca commented on the pull request: https://github.com/apache/couchdb-couch/pull/45#issuecomment-94896828 Manually merged. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have th

[GitHub] couchdb-couch pull request: Fix error handling in couch_util:with_...

2015-04-21 Thread chewbranca
Github user chewbranca closed the pull request at: https://github.com/apache/couchdb-couch/pull/45 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featu

Re: MapReduce Simulator (Fwd: how to use an included document's key?)

2015-04-21 Thread Andy Wenk
yes +1. This is cool. Jan, can you remember the tool where one could try out map reduce? It was really nice and I can't remember why it's gone ... All the best Andy On 21 April 2015 at 10:16, Alexander Shorin wrote: > On Tue, Apr 21, 2015 at 10:42 AM, Jan Lehnardt wrote: > > I just found htt

[GitHub] couchdb-couch pull request: Fix error handling in couch_util:with_...

2015-04-21 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/45#issuecomment-94894811 @chewbranca eh, ok, I had good hope on them (: +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] couchdb-couch pull request: Fix error handling in couch_util:with_...

2015-04-21 Thread iilyak
Github user iilyak commented on the pull request: https://github.com/apache/couchdb-couch/pull/45#issuecomment-94894553 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] couchdb-couch pull request: Fix error handling in couch_util:with_...

2015-04-21 Thread chewbranca
Github user chewbranca commented on the pull request: https://github.com/apache/couchdb-couch/pull/45#issuecomment-94894090 @kxepal yeah slipped by me as well. The errors don't have an error flag when they're the return value of a process exiting, as you can see below. ``

[GitHub] couchdb-couch pull request: Fix error handling in couch_util:with_...

2015-04-21 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/45#issuecomment-94893318 Oh..I overlooked this as well. Btw, aren't Errors are already happens in {error, foo} format? Or there is no reason to be ensured in that? --- If your project is se

Re: [RESULT] Logo vote (was: Re: [VOTE] CouchDB Logo - Round #3)

2015-04-21 Thread Andy Wenk
Yes - thanks Robert for running the vote. All the best Andy On 21 April 2015 at 20:04, Noah Slater wrote: > Thanks Robert! > > On 21 April 2015 at 19:12, Robert Kowalski wrote: > > Hi folks, > > > > thanks for every who participated in the vote. A short update > > regarding our logo vote, the

[GitHub] couchdb-couch pull request: Fix error handling in couch_util:with_...

2015-04-21 Thread chewbranca
GitHub user chewbranca opened a pull request: https://github.com/apache/couchdb-couch/pull/45 Fix error handling in couch_util:with_proc You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/couchdb-couch 2657-fix-cassim-fabric

Re: [RESULT] Logo vote (was: Re: [VOTE] CouchDB Logo - Round #3)

2015-04-21 Thread Noah Slater
Thanks Robert! On 21 April 2015 at 19:12, Robert Kowalski wrote: > Hi folks, > > thanks for every who participated in the vote. A short update > regarding our logo vote, the three logos with the most points are: > > Constantin Angheloiu:15.5 > Paul Davis: 9.5 > O

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread sebastianrothbucher
Github user sebastianrothbucher commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28803298 --- Diff: app/addons/databases/actions.js --- @@ -0,0 +1,105 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); yo

[GitHub] couchdb-fauxton pull request: add loading lines to active tasks

2015-04-21 Thread michellephung
Github user michellephung commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/386#issuecomment-94881151 Hey I got some feedback from the #couchdb-dev channel that I'm gonna incorporate into this branch. 1. Instead of 1 row per active task, 2 rows wher

[GitHub] couchdb-fauxton pull request: Add nightwatch custom function: repl...

2015-04-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/couchdb-fauxton/pull/387 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[RESULT] Logo vote (was: Re: [VOTE] CouchDB Logo - Round #3)

2015-04-21 Thread Robert Kowalski
Hi folks, thanks for every who participated in the vote. A short update regarding our logo vote, the three logos with the most points are: Constantin Angheloiu:15.5 Paul Davis: 9.5 Old CouchDb Logo: 2 We will discuss the new logo now in the PMC and

[GitHub] couchdb-fauxton pull request: Test for error handling in Active Ta...

2015-04-21 Thread michellephung
Github user michellephung commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/388#discussion_r28797519 --- Diff: app/addons/activetasks/tests/activetasks.componentsSpec.react.jsx --- @@ -65,11 +65,21 @@ define([ }); af

[GitHub] couchdb-fauxton pull request: [WIP] Fix view bugs

2015-04-21 Thread robertkowalski
Github user robertkowalski commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/391#issuecomment-94846263 i think it is there: https://github.com/garrensmith/couchdb-fauxton/blob/fix-view-bugs/app/addons/documents/index-editor/components.react.jsx#L154 --- If

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread nolanlawson
Github user nolanlawson commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94822123 :+1: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94816845 @nolanlawson Thanks for the input. Ok, seems things getting simpler now, since I was confused different implementations. Good, I'll stay with Couchbase implementatio

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/370#issuecomment-94816126 @sebastianrothbucher I also found one bug. If I go to page 2 and then click new database, it then takes me back to page 1. I think you might need to add `dat

[GitHub] couchdb-fauxton pull request: [WIP] Doc Workflow improvements

2015-04-21 Thread robertkowalski
Github user robertkowalski commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/382#discussion_r28782823 --- Diff: app/addons/fauxton/components.js --- @@ -163,6 +164,11 @@ function (app, FauxtonAPI, ace, spin, ZeroClipboard) { initialize:

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread nolanlawson
Github user nolanlawson commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94812766 > If I make /_bulk_get compatible with PouchDB As @dholth said, his implementation is a *PouchDB plugin*. PouchDB does not depend on it. > A to

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94810216 @dholth About the errors...how do you feel with https://github.com/couchbase/sync_gateway/wiki/Bulk-GET ? Specifically: ``` If there's an error getti

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread dholth
Github user dholth commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94807050 The principle behind my _bulk_get is that it is *exactly* lots of GET returned in an array, the first implementation was even written as a lua intermediary bet

[GitHub] couchdb-fauxton pull request: Part 2/2 for Mango:

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/362#discussion_r28776697 --- Diff: app/addons/documents/index-results/stores.js --- @@ -96,17 +132,74 @@ function (FauxtonAPI, ActionTypes, HeaderActionTypes, Documents)

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94792422 > However assuming RCouch has users why not do their json. Double check to see what arguments PouchDB _bulk_get actually uses. Well, I'm fine to break compat

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94789582 > I may be the only pouchdb bulk get user. A top level array is a little better for streaming. Agree about streaming. > Does get return json for 4

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread dholth
Github user dholth commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94788972 However assuming RCouch has users why not do their json. Double check to see what arguments PouchDB _bulk_get actually uses. On Tue, Apr 21, 2015, at

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread dholth
Github user dholth commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94787191 I may be the only pouchdb bulk get user. A top level array is a little better for streaming. Does get return json for 404? That is what bulk get should do.

[GitHub] couchdb-couch pull request: rebased _bulk_get patch

2015-04-21 Thread kxepal
Github user kxepal commented on the pull request: https://github.com/apache/couchdb-couch/pull/18#issuecomment-94781101 Hi there, short update. I'd started porting this PR over 2.0. First note that in current state it doesn't works and breaks any document update, so don't try

[GitHub] couchdb-fauxton pull request: Part 2/2 for Mango:

2015-04-21 Thread robertkowalski
Github user robertkowalski commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/362#discussion_r28773683 --- Diff: app/addons/documents/index-results/stores.js --- @@ -96,17 +132,74 @@ function (FauxtonAPI, ActionTypes, HeaderActionTypes, Document

[GitHub] couchdb-fauxton pull request: Updating Auth module to use React

2015-04-21 Thread robertkowalski
Github user robertkowalski commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/390#discussion_r28768673 --- Diff: app/addons/auth/components.react.jsx --- @@ -0,0 +1,299 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); yo

[GitHub] couchdb-fauxton pull request: Updating Auth module to use React

2015-04-21 Thread robertkowalski
Github user robertkowalski commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/390#discussion_r28767153 --- Diff: app/addons/auth/components.react.jsx --- @@ -0,0 +1,299 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); yo

[GitHub] couchdb-fauxton pull request: Updating Auth module to use React

2015-04-21 Thread robertkowalski
Github user robertkowalski commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/390#discussion_r28766966 --- Diff: app/addons/auth/actions.js --- @@ -0,0 +1,116 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may not

[GitHub] couchdb-fauxton pull request: Updating Auth module to use React

2015-04-21 Thread robertkowalski
Github user robertkowalski commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/390#discussion_r28766596 --- Diff: app/addons/auth/actions.js --- @@ -0,0 +1,116 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may not

[GitHub] couchdb-fauxton pull request: Test for error handling in Active Ta...

2015-04-21 Thread garrensmith
Github user garrensmith commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/388#issuecomment-94736886 A small change. Otherwise +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] couchdb-fauxton pull request: Test for error handling in Active Ta...

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/388#discussion_r28765885 --- Diff: app/addons/activetasks/tests/activetasks.componentsSpec.react.jsx --- @@ -65,11 +65,21 @@ define([ }); afte

[GitHub] couchdb-fauxton pull request: [WIP] Fix view bugs

2015-04-21 Thread robertkowalski
Github user robertkowalski commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/391#issuecomment-94735478 there is again an invariant-error, if you open http://localhost:8000/#database/animaldb/new_view looking at the stacktrace it originates from `va

[GitHub] couchdb-fauxton pull request: Add nightwatch custom function: repl...

2015-04-21 Thread garrensmith
Github user garrensmith commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/387#issuecomment-94734239 +1. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featu

[GitHub] couchdb-fauxton pull request: add loading lines to active tasks

2015-04-21 Thread garrensmith
Github user garrensmith commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/386#issuecomment-94734074 @michellephung this is awesome. I think the loading lines might be a little too muted. It took me a while to finally notice where the loading lines were. Onc

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/370#issuecomment-94730048 @sebastianrothbucher wow this is amazing work. Thank you. I've left a fair amount of comments, just somethings to get it more consistent with how we are usin

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28765101 --- Diff: app/addons/fauxton/components.react.jsx --- @@ -83,10 +83,132 @@ function (app, FauxtonAPI, React, ZeroClipboard) { } })

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28764806 --- Diff: app/addons/databases/routes.js --- @@ -46,43 +41,37 @@ function (app, FauxtonAPI, Databases, Views, Components) { this.datab

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28764771 --- Diff: app/addons/databases/routes.js --- @@ -46,43 +41,37 @@ function (app, FauxtonAPI, Databases, Views, Components) { this.datab

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28764497 --- Diff: app/addons/databases/components.react.jsx --- @@ -0,0 +1,303 @@ +// Licensed under the Apache License, Version 2.0 (the "License");

[GitHub] couchdb-fauxton pull request: [WIP] Fix view bugs

2015-04-21 Thread garrensmith
GitHub user garrensmith opened a pull request: https://github.com/apache/couchdb-fauxton/pull/391 [WIP] Fix view bugs This fixes some view bugs and converts the Code Editor component to use only React. You can merge this pull request into a Git repository by running: $ git pul

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28764131 --- Diff: app/addons/databases/actions.js --- @@ -0,0 +1,105 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may n

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28764003 --- Diff: app/addons/databases/actions.js --- @@ -0,0 +1,105 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may n

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28763914 --- Diff: app/addons/databases/actions.js --- @@ -0,0 +1,105 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may n

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28763820 --- Diff: app/addons/databases/actions.js --- @@ -0,0 +1,105 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may n

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28763442 --- Diff: app/addons/databases/actions.js --- @@ -0,0 +1,105 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may n

[GitHub] couchdb-fauxton pull request: Databases view in react

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/370#discussion_r28763361 --- Diff: app/addons/databases/actions.js --- @@ -0,0 +1,105 @@ +// Licensed under the Apache License, Version 2.0 (the "License"); you may n

[GitHub] couchdb-fauxton pull request: Part 2/2 for Mango:

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/362#discussion_r28761880 --- Diff: app/addons/documents/mango/mango.components.react.jsx --- @@ -121,35 +277,39 @@ function (app, FauxtonAPI, React, Stores, Actions, Reac

[GitHub] couchdb-fauxton pull request: Part 2/2 for Mango:

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/362#discussion_r28761814 --- Diff: app/addons/documents/mango/mango.actions.js --- @@ -41,18 +55,54 @@ function (app, FauxtonAPI, Documents, ActionTypes, Stores, IndexRes

[GitHub] couchdb-fauxton pull request: Part 2/2 for Mango:

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/362#discussion_r28761654 --- Diff: app/addons/documents/index-results/stores.js --- @@ -96,17 +132,74 @@ function (FauxtonAPI, ActionTypes, HeaderActionTypes, Documents)

[GitHub] couchdb-fauxton pull request: Part 2/2 for Mango:

2015-04-21 Thread garrensmith
Github user garrensmith commented on a diff in the pull request: https://github.com/apache/couchdb-fauxton/pull/362#discussion_r28761603 --- Diff: app/addons/documents/index-results/stores.js --- @@ -96,17 +132,74 @@ function (FauxtonAPI, ActionTypes, HeaderActionTypes, Documents)

[GitHub] couchdb-fauxton pull request: Part 2/2 for Mango:

2015-04-21 Thread garrensmith
Github user garrensmith commented on the pull request: https://github.com/apache/couchdb-fauxton/pull/362#issuecomment-94713530 @robertkowalski this looks great. One thing I've noticed is if I select __All Mango Indexes__ then the header for databases says Indexes and the cog disappea

Re: MapReduce Simulator (Fwd: how to use an included document's key?)

2015-04-21 Thread Alexander Shorin
On Tue, Apr 21, 2015 at 10:42 AM, Jan Lehnardt wrote: > I just found http://fiatjaf.alhur.es/couchdb-mr-simulator/ on user@. > > Would this make a great addition to Fauxton? +1 nice found -- ,,,^..^,,,

MapReduce Simulator (Fwd: how to use an included document's key?)

2015-04-21 Thread Jan Lehnardt
I just found http://fiatjaf.alhur.es/couchdb-mr-simulator/ on user@. Would this make a great addition to Fauxton? Best Jan -- > Begin forwarded message: > > From: ken tashiro > Subject: Re: how to use an included document's key? > Date: 21 Apr 2015 05:28:57 CEST > To: u...@couchdb.apache.org,