[GitHub] couchdb-couch pull request #209: remove compact files on failure

2016-10-27 Thread jaydoane
Github user jaydoane commented on a diff in the pull request:

https://github.com/apache/couchdb-couch/pull/209#discussion_r85454009
  
--- Diff: src/couch_db_updater.erl ---
@@ -1454,3 +1450,13 @@ default_security_object(_DbName) ->
 "everyone" ->
 []
 end.
+
+delete_compact_files(RootDir, FullFilepath) ->
+lists:foreach(fun(Ext) ->
+couch_file:delete(RootDir, FullFilepath ++ Ext)
+end, [".compact", ".compact.data", ".compact.meta"]).
+
+purge_compact_files(FullFilepath) ->
+lists:foreach(fun(Ext) ->
+file:delete(FullFilepath ++ Ext)
+end, [".compact", ".compact.data", ".compact.meta"]).
--- End diff --

Since you repeat this list, any reason to not create a 
`define(COMPACT_SUFFIXES, [".compact", ".compact.data", ".compact.meta"]).`
?


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Commented] (COUCHDB-3216) Corrupted compact.meta files can lead to cyclic crashes with no recovery

2016-10-27 Thread ILYA (JIRA)

[ 
https://issues.apache.org/jira/browse/COUCHDB-3216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15613726#comment-15613726
 ] 

ILYA commented on COUCHDB-3216:
---

This PR https://github.com/apache/couchdb-couch/pull/209 should fix it.

> Corrupted compact.meta files can lead to cyclic crashes with no recovery
> 
>
> Key: COUCHDB-3216
> URL: https://issues.apache.org/jira/browse/COUCHDB-3216
> Project: CouchDB
>  Issue Type: Bug
>Reporter: ILYA
>
> In the case when compact.meta file is corrupted the compactor keeps crashing 
> over and over with no recovery.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (COUCHDB-3216) Corrupted compact.meta files can lead to cyclic crashes with no recovery

2016-10-27 Thread ILYA (JIRA)
ILYA created COUCHDB-3216:
-

 Summary: Corrupted compact.meta files can lead to cyclic crashes 
with no recovery
 Key: COUCHDB-3216
 URL: https://issues.apache.org/jira/browse/COUCHDB-3216
 Project: CouchDB
  Issue Type: Bug
Reporter: ILYA


In the case when compact.meta file is corrupted the compactor keeps crashing 
over and over with no recovery.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] couchdb-couch pull request #209: remove compact files on failure

2016-10-27 Thread iilyak
GitHub user iilyak opened a pull request:

https://github.com/apache/couchdb-couch/pull/209

remove compact files on failure

In the case when `compact.meta` file is corrupted the compaction fails over 
and over. 
This fixes the cyclic failure aspect by use of a `.compact.crash` flag file.
The logic is:

```python
if there is an exception and  `.compact.crash` exists:
 remove(.compact.{meta|data})
```

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloudant/couchdb-couch 
76900-remove-compact-files-on-failure

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/couchdb-couch/pull/209.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #209


commit 0444072d8c10416800f05fb5c51bf67b25fda569
Author: ILYA Khlopotov 
Date:   2016-10-24T20:50:32Z

Extract {delete|purge}_compact_files

commit 7e7498369e52c58f52423d58b1971bac97440765
Author: ILYA Khlopotov 
Date:   2016-10-24T21:02:31Z

Use couch_db_updater:delete_compact_files/2

commit 72761387b02f73c9c5fa1960c9967fa7d918d2e9
Author: ILYA Khlopotov 
Date:   2016-10-27T22:33:26Z

Use .compact.crash file to recover cyclic failures

Remove compact.{meta|data} files if compaction crashes

BugzID: 76900




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] couchdb-couch-replicator pull request #51: Format gen_server state to remove...

2016-10-27 Thread iilyak
Github user iilyak commented on a diff in the pull request:


https://github.com/apache/couchdb-couch-replicator/pull/51#discussion_r85435146
  
--- Diff: src/couch_replicator.erl ---
@@ -600,8 +562,23 @@ terminate_cleanup(State) ->
 
 
 format_status(_Opt, [_PDict, State]) ->
-[{data, [{"State", state_strip_creds(State)}]}].
-
+Rep = 
couch_replicator_utils:format_rep_record(State#rep_state.rep_details),
+[{data, [
+{"State", ?from_record(rep_state, State, [
+start_seq,
+committed_seq,
+current_through_seq,
+%%seqs_in_progress = [],
--- End diff --

not sure how useful seqs_in_progress


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] couchdb-couch pull request #208: 71267 format process state

2016-10-27 Thread iilyak
GitHub user iilyak opened a pull request:

https://github.com/apache/couchdb-couch/pull/208

71267 format process state

Companion PR for https://github.com/apache/couchdb-couch-replicator/pull/51

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloudant/couchdb-couch 
71267-format_process_state

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/couchdb-couch/pull/208.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #208


commit da8cd161a7c6af74ca3d88ce42d537aa4312c7c4
Author: ILYA Khlopotov 
Date:   2016-10-27T21:27:08Z

Add record_to_keyval and from_record macro

COUCHDB-1606

commit d73697c3d953b27bf458e059eaf8ab9b12afb287
Author: ILYA Khlopotov 
Date:   2016-10-27T21:29:24Z

Move assertEqualLists to couch_eunit.hrl for reuse

COUCHDB-1606

commit 846064b7212980068296ade033e7b21aa0f75372
Author: ILYA Khlopotov 
Date:   2016-10-27T21:30:35Z

Implement couch_db:format_db_record/1

COUCHDB-1606




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] couchdb-couch-replicator pull request #51: Format gen_server state to remove...

2016-10-27 Thread iilyak
GitHub user iilyak opened a pull request:

https://github.com/apache/couchdb-couch-replicator/pull/51

Format gen_server state to remove plain passwords

Add format_status/2 to every gen_server to prune the state before
passing it to SASL logger. There are two goals for this work:
- eliminate plain text passwords in the logs
- reduce the size of the terms we log

The size of a term is a real problem. Since what's currently happening is:
1. we have lot's of data in state (queues for example)
2. the process crashes so the term is send to group leader
3. eventually lagger receives the term
4. couch_log format the term (we have io:format fork 
[couch_log_trunc_io_fmt]

(https://github.com/apache/couchdb-couch-log/blob/master/src/couch_log_trunc_io_fmt.erl))

Having truncate in step `#4` doesn't help much in case we have lot's of 
events to log. Since we bottleneck on group leader mailbox. Therefore we need 
to reduce the term size before we attempt to log it. 

COUCHDB-1606

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloudant/couchdb-couch-replicator 
71267-format_process_state

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/couchdb-couch-replicator/pull/51.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #51


commit 6adced02a672b065fb77432d58779b0ecf91359a
Author: ILYA Khlopotov 
Date:   2016-10-27T21:18:58Z

Format gen_server state to remove plain passwords

Add format_status/2 to every gen_server to prune the state before
passing it to SASL logger. There are two goals for this work:
- eliminate plain text passwords in the logs
- reduce the size of the terms we log

COUCHDB-1606




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Commented] (COUCHDB-1606) Replicator leaves plaintext password in logs

2016-10-27 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/COUCHDB-1606?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15613331#comment-15613331
 ] 

ASF GitHub Bot commented on COUCHDB-1606:
-

GitHub user iilyak opened a pull request:

https://github.com/apache/couchdb-couch-replicator/pull/51

Format gen_server state to remove plain passwords

Add format_status/2 to every gen_server to prune the state before
passing it to SASL logger. There are two goals for this work:
- eliminate plain text passwords in the logs
- reduce the size of the terms we log

The size of a term is a real problem. Since what's currently happening is:
1. we have lot's of data in state (queues for example)
2. the process crashes so the term is send to group leader
3. eventually lagger receives the term
4. couch_log format the term (we have io:format fork 
[couch_log_trunc_io_fmt]

(https://github.com/apache/couchdb-couch-log/blob/master/src/couch_log_trunc_io_fmt.erl))

Having truncate in step `#4` doesn't help much in case we have lot's of 
events to log. Since we bottleneck on group leader mailbox. Therefore we need 
to reduce the term size before we attempt to log it. 

COUCHDB-1606

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cloudant/couchdb-couch-replicator 
71267-format_process_state

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/couchdb-couch-replicator/pull/51.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #51


commit 6adced02a672b065fb77432d58779b0ecf91359a
Author: ILYA Khlopotov 
Date:   2016-10-27T21:18:58Z

Format gen_server state to remove plain passwords

Add format_status/2 to every gen_server to prune the state before
passing it to SASL logger. There are two goals for this work:
- eliminate plain text passwords in the logs
- reduce the size of the terms we log

COUCHDB-1606




> Replicator leaves plaintext password in logs
> 
>
> Key: COUCHDB-1606
> URL: https://issues.apache.org/jira/browse/COUCHDB-1606
> Project: CouchDB
>  Issue Type: Bug
>  Components: Logging, Replication
>Affects Versions: 1.2
>Reporter: Nathan Vander Wilt
>Assignee: Bob Dionne
> Attachments: pwd log.txt
>
>
> While reviewing logs, I noticed that a password had been recorded in the logs 
> as part of a replicator error.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] couchdb-fauxton pull request #794: 76046 - The Documentation Link in the Que...

2016-10-27 Thread millayr
GitHub user millayr opened a pull request:

https://github.com/apache/couchdb-fauxton/pull/794

76046 - The Documentation Link in the Query Options Panel Points to Futon 
Documentation

This is from Cloudant ticket https://cloudant.fogbugz.com/f/cases/76046/.

The documentation link still pointed to Fauxton docs instead of Cloudant 
docs.  The getDefaultProps() call was happening before 
FauxtonAPI.constants.DOC_URLS.GENERAL was tweaked for Cloudant specific values.

The getDefaultProps() function is now removed and the docURL is passed as a 
prop to MainFieldsView.  The docURL is now a required property for the React 
class per advice from Robert Kowalski.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/millayr/couchdb-fauxton 
76046-query-options-documentation-link

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/couchdb-fauxton/pull/794.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #794


commit ab88d189959118ae9dd90547aa0d4a4ffd8a8445
Author: Ryan Millay 
Date:   2016-10-25T14:12:12Z

Passing docURL as prop to allow for custom Cloudant value




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] couchdb issue #442: Adds a Snap package build option with: ./configure && ma...

2016-10-27 Thread mhall119
Github user mhall119 commented on the issue:

https://github.com/apache/couchdb/pull/442
  
Merged by rnewson


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] couchdb pull request #442: Adds a Snap package build option with: ./configur...

2016-10-27 Thread mhall119
Github user mhall119 closed the pull request at:

https://github.com/apache/couchdb/pull/442


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (COUCHDB-3215) Differet size for the same view

2016-10-27 Thread Raphael Augusto da Silva Nunes Soransso (JIRA)
Raphael Augusto da Silva Nunes Soransso created COUCHDB-3215:


 Summary: Differet size for the same view
 Key: COUCHDB-3215
 URL: https://issues.apache.org/jira/browse/COUCHDB-3215
 Project: CouchDB
  Issue Type: Bug
  Components: Database Core, JavaScript View Server
Reporter: Raphael Augusto da Silva Nunes Soransso


Hi, I have created two structure for a set of data to analyse if it would make 
the queries faster, one data was created using a different table, from 
relational database, as the root of the documents:
Example:
One struture is like this:
{
   "c_customer_sk": 65836,
   "c_first_name": "Frank",
   "c_last_name": "White",
   "store_sales": [   {
   "ss_sales_price": 20.24,
   "ss_ext_sales_price": 1012,
   "ss_coupon_amt": 0,
   "date": [   {
   "d_month_seq": 1187,
   "d_year": 1998   }   ],
   "item": [   {
   "i_item_sk": 10454,
   "i_item_id": "GNIC",
   "i_item_desc": "Results highlight as patterns; so right 
years show. Sometimes suitable lips move with the critics. English, old mothers 
ought to lift now perhaps future managers. Active, single ch",
   "i_current_price": 2.88,
   "i_class": "romance",
   "i_category_id": 9,
   "i_category": "Books"   }   ]   },
   {
"ss_sales_price": 225,
   "ss_ext_sales_price": 1023,
   "ss_coupon_amt": 0,... 

The other organization for the same data is:
{
  "i_item_sk": 10454,
  "i_item_id": "GNIC",
  "i_item_desc": "Results highlight as patterns; so right years show. Sometimes 
suitable lips move with the critics. English, old mothers ought to lift now 
perhaps future managers. Active, single ch",
  "i_current_price": 2.88,
  "i_class": "romance",
  "i_category_id": 9,
  "i_category": "Books",
   "store_sales": [
   {
   "ss_sales_price": 20.24,
   "ss_ext_sales_price": 1012,
   "ss_coupon_amt": 0,
   "date": [
   {
   "d_month_seq": 1187,
   "d_year": 1998
   }
   ],
   "customer": [
   { 
   "c_customer_sk": 65836,
   "c_first_name": "Frank",
   "c_last_name": "White",
   }
   ]
   },
   {
"ss_sales_price": 225,
   "ss_ext_sales_price": 1023,
   "ss_coupon_amt": 0,...  

If I create the same view for both data, their data_sizes are different, can 
someone explain me why?
I have posted it on stackoverflow too: 
http://stackoverflow.com/questions/38509074/different-size-for-the-same-view-on-couchdb/38521448#38521448



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] couchdb-fauxton issue #782: Encoding issues: fix for database names and refa...

2016-10-27 Thread garrensmith
Github user garrensmith commented on the issue:

https://github.com/apache/couchdb-fauxton/pull/782
  
If I click on a shared db's database I get `base.js:243 Uncaught TypeError: 
Cannot read property 'server' of undefined(…)`


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] couchdb-fauxton issue #782: Encoding issues: fix for database names and refa...

2016-10-27 Thread garrensmith
Github user garrensmith commented on the issue:

https://github.com/apache/couchdb-fauxton/pull/782
  
If I click on the x icon in shared db's I get `components.react.jsx:213 
Uncaught TypeError: Cannot read property 'get' of undefined(…)`


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Commented] (COUCHDB-3213) Various Fauxton usability issues

2016-10-27 Thread Garren Smith (JIRA)

[ 
https://issues.apache.org/jira/browse/COUCHDB-3213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15611399#comment-15611399
 ] 

Garren Smith commented on COUCHDB-3213:
---

Thanks Nolan, this is helpful. You won't be able to update CORS via Fauxton if 
you are on port 5984 you need to use one of the backend ports for that.

For the setup failing is there anything in the dev tools console that can give 
us a bit more info?

> Various Fauxton usability issues
> 
>
> Key: COUCHDB-3213
> URL: https://issues.apache.org/jira/browse/COUCHDB-3213
> Project: CouchDB
>  Issue Type: Bug
>  Components: Fauxton
>Reporter: Nolan Lawson
>
> I figured it'd be too time-consuming to list all issues I've found with 
> Fauxton, so I just decided to create one master issue. Please feel free to 
> divide into smaller issues or ignore as applicable. :)
> - Can't create a new non-admin user
> - Click "Configuration", click "CORS", click "Main config," nothing happens, 
> have to refresh
> - Click "Setup," click "Configure single node," fill in admin/password, click 
> "Configure Node," nothing happens.
> I'm using Edge 14 on Windows 10 if that helps at all.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)