You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@allura.apache.org by Dave Brondsema <da...@brondsema.net> on 2016/09/22 19:28:15 UTC

[allura:tickets] #8131 refresh repo task uses wrong query



---

** [tickets:#8131] refresh repo task uses wrong query**

**Status:** in-progress
**Milestone:** unreleased
**Created:** Thu Sep 22, 2016 07:28 PM UTC by Dave Brondsema
**Last Updated:** Thu Sep 22, 2016 07:28 PM UTC
**Owner:** Dave Brondsema


`refreshrepo.py` queries `LastCommitDoc` by `commit_ids` but it should be `commit_id`.  Using the wrong field name means it runs extremely slowly (unindexed) and never finds any records.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #8131 refresh repo task uses wrong query

Posted by Dave Brondsema <da...@brondsema.net>.
Fix on db/8131

Unfortunately we don't have tests for this, but you can run the script/task using instructions here: https://forge-allura.apache.org/docs/getting_started/administration.html#refreshrepo-py  Use `--clean` for the affected code to run.


---

** [tickets:#8131] refresh repo task uses wrong query**

**Status:** in-progress
**Milestone:** unreleased
**Created:** Thu Sep 22, 2016 07:28 PM UTC by Dave Brondsema
**Last Updated:** Thu Sep 22, 2016 07:28 PM UTC
**Owner:** Dave Brondsema


`refreshrepo.py` queries `LastCommitDoc` by `commit_ids` but it should be `commit_id`.  Using the wrong field name means it runs extremely slowly (unindexed) and never finds any records.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #8131 refresh repo task uses wrong query

Posted by Dave Brondsema <da...@brondsema.net>.
- **status**: in-progress --> review



---

** [tickets:#8131] refresh repo task uses wrong query**

**Status:** review
**Milestone:** unreleased
**Created:** Thu Sep 22, 2016 07:28 PM UTC by Dave Brondsema
**Last Updated:** Thu Sep 22, 2016 07:30 PM UTC
**Owner:** Dave Brondsema


`refreshrepo.py` queries `LastCommitDoc` by `commit_ids` but it should be `commit_id`.  Using the wrong field name means it runs extremely slowly (unindexed) and never finds any records.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #8131 refresh repo task uses wrong query

Posted by Dave Brondsema <da...@brondsema.net>.
- **status**: review --> closed



---

** [tickets:#8131] refresh repo task uses wrong query**

**Status:** closed
**Milestone:** unreleased
**Created:** Thu Sep 22, 2016 07:28 PM UTC by Dave Brondsema
**Last Updated:** Mon Sep 26, 2016 02:50 PM UTC
**Owner:** Dave Brondsema


`refreshrepo.py` queries `LastCommitDoc` by `commit_ids` but it should be `commit_id`.  Using the wrong field name means it runs extremely slowly (unindexed) and never finds any records.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.

[allura:tickets] #8131 refresh repo task uses wrong query

Posted by Kenton Taylor <kt...@slashdotmedia.com>.
Change looks good; clear to merge.


---

** [tickets:#8131] refresh repo task uses wrong query**

**Status:** review
**Milestone:** unreleased
**Created:** Thu Sep 22, 2016 07:28 PM UTC by Dave Brondsema
**Last Updated:** Thu Sep 22, 2016 07:30 PM UTC
**Owner:** Dave Brondsema


`refreshrepo.py` queries `LastCommitDoc` by `commit_ids` but it should be `commit_id`.  Using the wrong field name means it runs extremely slowly (unindexed) and never finds any records.


---

Sent from forge-allura.apache.org because dev@allura.apache.org is subscribed to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is a mailing list, you can unsubscribe from the mailing list.