Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread keith
On March 28, 2014, 6:53 p.m., kturner wrote: docs/src/main/resources/design/ACCUMULO-378-design.mdtext, line 139 https://reviews.apache.org/r/19790/diff/1/?file=539855#file539855line139 A walog or bulk imported file could be referenced by multiple tablets. I am wondering if it

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread keith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19790/#review39049 --- Josh, after I converted your doc to markdown I should have sent it

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread keith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19790/#review39051 --- docs/src/main/resources/design/ACCUMULO-378-design.mdtext

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread Josh Elser
On March 31, 2014, 4:13 p.m., kturner wrote: Josh, after I converted your doc to markdown I should have sent it to you and let you post it to RB. I was so excited about experimenting w/ reviewing design docs on RB that I did not really think that part through. Josh, what do you

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread keith
On March 31, 2014, 4:21 p.m., kturner wrote: docs/src/main/resources/design/ACCUMULO-378-design.mdtext, line 34 https://reviews.apache.org/r/19790/diff/1/?file=539855#file539855line34 Can a table be replicated to multiple clusters? More specifically, can a table on one cluster be

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread keith
On March 31, 2014, 4:13 p.m., kturner wrote: Josh, after I converted your doc to markdown I should have sent it to you and let you post it to RB. I was so excited about experimenting w/ reviewing design docs on RB that I did not really think that part through. Josh, what do you

Re: Review Request 19804: ACCUMULO-2519 Aborts upgrade if there are Fate transactions from an old version.

2014-03-31 Thread Bill Havanki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19804/#review39056 --- server/src/main/java/org/apache/accumulo/server/Accumulo.java

Re: Review Request 19749: ACCUMULO-2566 Hadoop reflection for TeraSortIngest

2014-03-31 Thread Bill Havanki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19749/#review39059 --- Ship it! Ship It! - Bill Havanki On March 27, 2014, 4:25 p.m.,

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread Josh Elser
On March 31, 2014, 4:21 p.m., kturner wrote: docs/src/main/resources/design/ACCUMULO-378-design.mdtext, line 34 https://reviews.apache.org/r/19790/diff/1/?file=539855#file539855line34 Can a table be replicated to multiple clusters? kturner wrote: More specifically, can a

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread Bill Havanki
On March 28, 2014, 2:53 p.m., kturner wrote: docs/src/main/resources/design/ACCUMULO-378-design.mdtext, line 157 https://reviews.apache.org/r/19790/diff/1/?file=539855#file539855line157 Are you thinking a FATE operation per file? FATE uses zookeeper, and zookeeper keeps

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread Bill Havanki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19790/#review39061 --- docs/src/main/resources/design/ACCUMULO-378-design.mdtext

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread keith
On March 31, 2014, 4:21 p.m., kturner wrote: docs/src/main/resources/design/ACCUMULO-378-design.mdtext, line 80 https://reviews.apache.org/r/19790/diff/1/?file=539855#file539855line80 Whats the rational for replicating WAL as opposed to replicating minor compacted rfiles? What

Re: Review Request 19790: ACCUMULO-378 Design document

2014-03-31 Thread keith
On March 28, 2014, 6:53 p.m., kturner wrote: docs/src/main/resources/design/ACCUMULO-378-design.mdtext, line 119 https://reviews.apache.org/r/19790/diff/1/?file=539855#file539855line119 Seems like accumulo should have a public API for querying what needs to be replicated,

Re: Review Request 19804: ACCUMULO-2519 Aborts upgrade if there are Fate transactions from an old version.

2014-03-31 Thread keith
On March 29, 2014, 12:26 a.m., kturner wrote: server/src/main/java/org/apache/accumulo/server/master/Master.java, line 313 https://reviews.apache.org/r/19804/diff/1/?file=539926#file539926line313 I think this check can cause problems. Master.run() starts StatusThread,

Re: Review Request 19804: ACCUMULO-2519 Aborts upgrade if there are Fate transactions from an old version.

2014-03-31 Thread Sean Busbey
On March 29, 2014, 12:26 a.m., kturner wrote: server/src/main/java/org/apache/accumulo/server/master/Master.java, line 313 https://reviews.apache.org/r/19804/diff/1/?file=539926#file539926line313 I think this check can cause problems. Master.run() starts StatusThread,

Re: Question about scanid in ActiveScan - is it valid?

2014-03-31 Thread Keith Turner
I looked at the 1.4 and 1.6 branches and its not set in either. This certainly needs a ticket. On Sun, Mar 30, 2014 at 7:34 PM, Ed Coleman d...@etcoleman.com wrote: I was looking to profile some long running scans and tried to use the scan id to correlate long running scans between

Re: Review Request 19732: ACCUMULO-2558 - unit tests for server/gc

2014-03-31 Thread Bill Havanki
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19732/ --- (Updated March 31, 2014, 7:36 p.m.) Review request for accumulo. Changes

Review Request 19862: Design document for review on cross-cluster replication

2014-03-31 Thread Josh Elser
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19862/ --- Review request for accumulo. Bugs: ACCUMULO-378