[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on the pull request: https://github.com/apache/accumulo/pull/103#issuecomment-81434 > What we needed and implemented was a way to bypass the tserver. I would like to see this supported through the connector as an established mechanism for data retri

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread phrocker
Github user phrocker commented on the pull request: https://github.com/apache/accumulo/pull/103#issuecomment-77916 Josh, aka Josh, sent me a note to take a look. Because Josh, aka Josh, tended to my chickens while I was away, will take a moment away from tending to my flock to com

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on the pull request: https://github.com/apache/accumulo/pull/103#issuecomment-73600 I see you didn't switch over the AccumuloFileOutputFormat to use this new API. I would have considered that exercise good validation of the API that you made here. T

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/103#discussion_r64977567 --- Diff: core/src/main/java/org/apache/accumulo/core/file/FileOperations.java --- @@ -278,9 +288,20 @@ public RateLimiter getRateLimiter() { /**

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/103#discussion_r64976262 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFileWriter.java --- @@ -0,0 +1,188 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/103#discussion_r64976184 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFileScanner.java --- @@ -0,0 +1,330 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/103#discussion_r64976153 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFileScanner.java --- @@ -0,0 +1,330 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/103#discussion_r64976056 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFileScanner.java --- @@ -0,0 +1,330 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/103#discussion_r64975799 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFile.java --- @@ -0,0 +1,271 @@ +/* + * Licensed to the Apache Software Founda

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/103#discussion_r64975664 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFile.java --- @@ -0,0 +1,271 @@ +/* + * Licensed to the Apache Software Founda

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/103#discussion_r64975609 --- Diff: core/src/main/java/org/apache/accumulo/core/client/rfile/RFile.java --- @@ -0,0 +1,271 @@ +/* + * Licensed to the Apache Software Founda

[GitHub] accumulo pull request: ACCUMULO-4165 Added a user facing API for R...

2016-05-27 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/accumulo/pull/103 ACCUMULO-4165 Added a user facing API for RFile You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/accumulo rfile-api Alternat