[GitHub] incubator-fluo pull request #837: fixes #835 escaped binary data in collisio...

2017-05-12 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/837#discussion_r116265085 --- Diff: modules/core/src/main/java/org/apache/fluo/core/log/TracingTransaction.java --- @@ -96,6 +96,11 @@ private String encC(Map<Col

[GitHub] incubator-fluo pull request #837: fixes #835 escaped binary data in collisio...

2017-05-12 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/837#discussion_r116251341 --- Diff: modules/core/src/main/java/org/apache/fluo/core/log/TracingTransaction.java --- @@ -96,6 +96,11 @@ private String encC(Map<Col

[GitHub] incubator-fluo pull request #837: fixes #835 escaped binary data in collisio...

2017-05-12 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/837#discussion_r116249464 --- Diff: modules/integration/src/test/java/org/apache/fluo/integration/TestUtil.java --- @@ -17,18 +17,26 @@ import

[GitHub] incubator-fluo issue #829: Observers implemented as lambdas showing up multi...

2017-05-11 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/829 Things are much better after this change ![image](https://cloud.githubusercontent.com/assets/1268739/25965473/174c9a28-3656-11e7-93f1-1a227841d518.png) --- If your project

[GitHub] incubator-fluo issue #834: Seeing spurious NPE in worker logs

2017-05-11 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo/issues/834 Seeing spurious NPE in worker logs Saw the following exception in worker logs while testing 1.1.0-SNAP. I was using commit 4055cdfbe13eb34f8830db3ba2201c6428d26fd4 I think

[GitHub] incubator-fluo-website issue #55: Add mjwall as contributor

2017-05-10 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo-website/pull/55 +1 The change is live on the website. --- 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] incubator-fluo issue #827: Add copyTo() method to bytes

2017-05-09 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/827 Not yet, but hopefully soon [INFRA-14121](https://issues.apache.org/jira/browse/INFRA-14121) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-fluo pull request #832: closes #809 - cleanup buffered data

2017-05-09 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/832#discussion_r115614639 --- Diff: modules/core/src/main/java/org/apache/fluo/core/impl/TransactionImpl.java --- @@ -596,9 +596,17 @@ public CommitData createCommitData

[GitHub] incubator-fluo pull request #832: closes #809 - cleanup buffered data

2017-05-09 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/832#discussion_r115613754 --- Diff: modules/core/src/main/java/org/apache/fluo/core/impl/TransactionImpl.java --- @@ -596,9 +596,17 @@ public CommitData createCommitData

[GitHub] incubator-fluo pull request #832: closes #809 - cleanup buffered data

2017-05-09 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/832#discussion_r115549403 --- Diff: modules/core/src/main/java/org/apache/fluo/core/impl/TransactionImpl.java --- @@ -596,9 +597,14 @@ public CommitData createCommitData

[GitHub] incubator-fluo-recipes pull request #131: Updated for new way of registering...

2017-05-09 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-recipes/pull/131 Updated for new way of registering observers You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo-recipes

[GitHub] incubator-fluo pull request #832: closes #809 - cleanup buffered data

2017-05-09 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/832#discussion_r115501042 --- Diff: modules/core/src/main/java/org/apache/fluo/core/impl/TransactionImpl.java --- @@ -27,12 +30,6 @@ import java.util.Objects

[GitHub] incubator-fluo pull request #832: closes #809 - cleanup buffered data

2017-05-09 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/832#discussion_r115500227 --- Diff: modules/core/src/main/java/org/apache/fluo/core/impl/TransactionImpl.java --- @@ -596,9 +597,14 @@ public CommitData createCommitData

[GitHub] incubator-fluo pull request #831: fixes #829 Changed API to allow giving obs...

2017-05-08 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo/pull/831 fixes #829 Changed API to allow giving observers an optional ID You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo

[GitHub] incubator-fluo issue #830: Optimize ByteArrayUtil.concat()

2017-05-08 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo/issues/830 Optimize ByteArrayUtil.concat() `static byte[] ByteArrayUtil.concat(Bytes ... listOfBytes)` has the following performance problems * It uses ByteArrayOutputStream() which has

[GitHub] incubator-fluo issue #829: Observers implemented as lambdas showing up multi...

2017-05-05 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/829 I did some research to see if these numbers generated for lambdas are stable w.r.t to a compiled class files. From what I read online, it seems they are not. The first number

[GitHub] incubator-fluo-recipes pull request #130: Updated ExportQ and CFM to use new...

2017-05-03 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-recipes/pull/130#discussion_r114658533 --- Diff: docs/combine-queue.md --- @@ -0,0 +1,224 @@ + +# Combine Queue Recipe + +## Background + +When many

[GitHub] incubator-fluo-recipes pull request #130: Updated ExportQ and CFM to use new...

2017-05-03 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-recipes/pull/130#discussion_r114650888 --- Diff: docs/combine-queue.md --- @@ -0,0 +1,224 @@ + +# Combine Queue Recipe + +## Background + +When many

[GitHub] incubator-fluo-recipes pull request #130: Updated ExportQ and CFM to use new...

2017-05-03 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-recipes/pull/130#discussion_r114646561 --- Diff: docs/combine-queue.md --- @@ -0,0 +1,224 @@ + +# Combine Queue Recipe + +## Background + +When many

[GitHub] incubator-fluo-recipes pull request #130: Updated ExportQ and CFM to use new...

2017-05-02 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-recipes/pull/130 Updated ExportQ and CFM to use new ObserverProvider API Create a new CombineQueue API that replaces the CollisionFreeMap. The new combineQ uses the new Fluo Observer APIs

[GitHub] incubator-fluo-recipes issue #128: Updated ExportQ and CFM to use new Observ...

2017-04-28 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo-recipes/pull/128 I am going to resubmit this PR with the CollisionFreeMap deprecated and renamed to CombineQueue. The CombineQueue will only support the new Observer API. --- If your project

[GitHub] incubator-fluo-recipes pull request #129: Reduced creation of intermediate S...

2017-04-28 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-recipes/pull/129#discussion_r113979089 --- Diff: modules/core/src/main/java/org/apache/fluo/recipes/core/map/CollisionFreeMap.java --- @@ -194,12 +199,11 @@ void process

[GitHub] incubator-fluo-recipes pull request #128: Updated ExportQ and CFM to use new...

2017-04-25 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-recipes/pull/128#discussion_r113245202 --- Diff: docs/accumulo-export-queue.md --- @@ -19,8 +19,8 @@ limitations under the License. ## Background The [Export

[GitHub] incubator-fluo-recipes pull request #128: Updated ExportQ and CFM to use new...

2017-04-25 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-recipes/pull/128#discussion_r113224356 --- Diff: docs/accumulo-export-queue.md --- @@ -19,8 +19,8 @@ limitations under the License. ## Background The [Export

[GitHub] incubator-fluo-recipes pull request #128: Updated ExportQ and CFM to use new...

2017-04-25 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-recipes/pull/128#discussion_r113202528 --- Diff: docs/accumulo-export-queue.md --- @@ -19,8 +19,8 @@ limitations under the License. ## Background The [Export

[GitHub] incubator-fluo pull request #825: fixes #823

2017-04-24 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/825#discussion_r113102707 --- Diff: modules/api/src/main/java/org/apache/fluo/api/data/Bytes.java --- @@ -356,29 +357,44 @@ public static final Bytes of(String s, Charset

[GitHub] incubator-fluo-recipes issue #128: Updated ExportQ and CFM to use new Observ...

2017-04-24 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo-recipes/pull/128 This depends on apache/incubator-fluo-recipes#128 --- 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] incubator-fluo issue #826: Avoid using bytesAt() internally in Bytes

2017-04-24 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/826 Can also avoid continually adding the offsets by doing it once outside the loops. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-fluo pull request #824: fixed some warnings

2017-04-21 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo/pull/824 fixed some warnings You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo f1x_warnings Alternatively you can review

[GitHub] incubator-fluo-recipes issue #127: Rename CollisionFreeMap

2017-04-18 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-recipes/issues/127 Rename CollisionFreeMap CollisionFreeMap is a terrible name. I think something like ContinuousCombiner would be much better. I am thinking it would be nice to deprecate

[GitHub] incubator-fluo issue #823: Add endsWith and startsWith methods to Bytes

2017-04-18 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo/issues/823 Add endsWith and startsWith methods to Bytes Multiple times I have run into situations where I wanted to know if Bytes started with a certain prefix. It would be useful to have methods

[GitHub] incubator-fluo issue #817: fixes #816 introduced better way to setup observe...

2017-03-29 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/pull/817 Squash merged in commit d6af386bfacd795350f0bdb79da751748c09cfdc --- 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

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-29 Thread keith-turner
Github user keith-turner closed the pull request at: https://github.com/apache/incubator-fluo/pull/817 --- 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

[GitHub] incubator-fluo issue #555: Saw NotificationGcIT fail on travis build

2017-03-23 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/555 Saw this fail again on travis build of #817 ``` Running org.apache.fluo.integration.impl.NotificationGcIT 2017-03-23 17:27:57,321 [impl.NotificationGcIT] ERROR: r1

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-22 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/817#discussion_r107479688 --- Diff: modules/api/src/main/java/org/apache/fluo/api/config/FluoConfiguration.java --- @@ -86,8 +88,19 @@ /** The properties

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-22 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/817#discussion_r107480219 --- Diff: modules/integration/src/test/java/org/apache/fluo/integration/impl/CollisionIT.java --- @@ -81,27 +77,22 @@ public void load

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-21 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/817#discussion_r107207572 --- Diff: modules/api/src/main/java/org/apache/fluo/api/observer/ObserversFactory.java --- @@ -0,0 +1,119 @@ +/* + * Licensed

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-21 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/817#discussion_r107207929 --- Diff: modules/api/src/main/java/org/apache/fluo/api/observer/ObserverFactory.java --- @@ -0,0 +1,119 @@ +/* + * Licensed

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-20 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/817#discussion_r107015681 --- Diff: modules/api/src/main/java/org/apache/fluo/api/observer/ObserversFactory.java --- @@ -0,0 +1,119 @@ +/* + * Licensed

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-20 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/817#discussion_r107016744 --- Diff: modules/api/src/main/java/org/apache/fluo/api/observer/StringObserver.java --- @@ -0,0 +1,34 @@ +/* + * Licensed

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-20 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/817#discussion_r107015060 --- Diff: modules/api/src/main/java/org/apache/fluo/api/observer/ObserversFactory.java --- @@ -0,0 +1,119 @@ +/* + * Licensed

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-20 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo/pull/817#discussion_r107011906 --- Diff: pom.xml --- @@ -83,6 +83,11 @@ 1.32 +com.google.code.gson +gson

[GitHub] incubator-fluo pull request #817: fixes #816 introduced better way to setup ...

2017-03-17 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo/pull/817 fixes #816 introduced better way to setup observers and deprecated old You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith

[GitHub] incubator-fluo issue #345: fluo yarn start fails with UnknownHostException

2017-02-16 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/345 @dlnufox can you provide more details about the error you saw? --- 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

[GitHub] incubator-fluo-website pull request #53: Blog post about Accumulo Summit 201...

2017-01-10 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/53 Blog post about Accumulo Summit 2016 talks You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/incubator-fluo

[GitHub] incubator-fluo-recipes pull request #126: fixes #122 improved Spark document...

2016-12-28 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-recipes/pull/126#discussion_r94056163 --- Diff: docs/spark.md --- @@ -0,0 +1,29 @@ + +# Apache Spark helper code + +Fluo Recipes has some helper code

[GitHub] incubator-fluo issue #290: Investigate Mesos as an option for running Fluo

2016-12-27 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/290 http://aurora.apache.org/ --- 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] incubator-fluo-website pull request #47: add comment about empty

2016-11-29 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/47 add comment about empty You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/incubator-fluo-website cfm-comment

[GitHub] incubator-fluo-website pull request #46: fix imports

2016-11-29 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/46 fix imports You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/incubator-fluo-website imports Alternatively you

[GitHub] incubator-fluo-website pull request #45: Update (most) old fluo-io links

2016-11-29 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/45#discussion_r90150621 --- Diff: docs/fluo/1.0.0-alpha-1/stress.md --- @@ -34,22 +34,22 @@ Next, build the module: This will create two jars

[GitHub] incubator-fluo-website pull request #45: Update (most) old fluo-io links

2016-11-29 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/45#discussion_r90150295 --- Diff: _posts/blog/2014-12-30-stress-test-long-run.md --- @@ -22,19 +22,19 @@ everything finished, the test was a success. The number

[GitHub] incubator-fluo-website pull request #45: Update (most) old fluo-io links

2016-11-29 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/45#discussion_r90150435 --- Diff: _posts/release/2016-01-12-fluo-1.0.0-beta-2.md --- @@ -125,7 +125,7 @@ made. The following API changes happened between beta

[GitHub] incubator-fluo-website pull request #45: Update (most) old fluo-io links

2016-11-29 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/45#discussion_r90150535 --- Diff: docs/fluo/1.0.0-alpha-1/stress.md --- @@ -22,10 +22,10 @@ These tests can be run using `mvn verify`. Run trie stress test

[GitHub] incubator-fluo-website pull request #45: Update (most) old fluo-io links

2016-11-29 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/45#discussion_r90150711 --- Diff: resources/blog/stress_1/fluo.properties --- @@ -62,7 +62,7 @@ io.fluo.admin.accumulo.classpath=${io.fluo.admin.hdfs}/fluo/lib

[GitHub] incubator-fluo-website pull request #44: Update excercise 1 to add CFM and i...

2016-11-29 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/44 Update excercise 1 to add CFM and instructions on running in live ins… …tance. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-fluo issue #800: Investigate caching String to Bytes conversions a...

2016-11-17 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/800 In #804 I did some very narrow caching. This caching is only at a per transaction level and in some cases only per a transactions operation. --- If your project is set up

[GitHub] incubator-fluo issue #803: ColumnValue has no unit test

2016-11-17 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo/issues/803 ColumnValue has no unit test Need to add unit test for ColumnValue class. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-fluo pull request #802: Fluo 785

2016-11-14 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo/pull/802 Fluo 785 You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo fluo-785 Alternatively you can review and apply

[GitHub] incubator-fluo issue #801: Collisions only track last row/column that collid...

2016-11-14 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo/issues/801 Collisions only track last row/column that collided While working on #785 I discovered that collisions are only tracking the last row/col to collide for debugging purposes

[GitHub] incubator-fluo-website pull request #40: Added some basic twitter support

2016-11-08 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/40 Added some basic twitter support You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/incubator-fluo-website

[GitHub] incubator-fluo pull request #799: Use faster byte comparison

2016-11-08 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo/pull/799 Use faster byte comparison You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo fast_compare Alternatively you can

[GitHub] incubator-fluo-website pull request #39: Minor updates to Fluo Tour

2016-11-07 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/39 Minor updates to Fluo Tour You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/incubator-fluo-website

[GitHub] incubator-fluo pull request #797: fixes #425 added trace logging for scans

2016-11-04 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo/pull/797 fixes #425 added trace logging for scans You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo fluo-425 Alternatively

[GitHub] incubator-fluo-website issue #38: Mention logging config in tour

2016-11-04 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-website/issues/38 Mention logging config in tour Would be nice to link to docs from apache/incubator-fluo#796 in tour. Also mention that mini fluo uses log4j instead of logback

[GitHub] incubator-fluo issue #425: Log more informative trace for scans

2016-11-02 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/425 Could log scanned data at a more specific level that could be turned off. Something like `fluo.tx.scan` which could be suppressed. A single line summary about scanned data could

[GitHub] incubator-fluo issue #425: Log more informative trace for scans

2016-11-02 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/425 Need to make sure it logs something for a scan that returns nothing --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-fluo issue #796: Improved documentation for viewing Fluo logs

2016-11-01 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/pull/796 We also need to document that it would be best if observers used slf4j when doing their own logging. That could be a new issue/PR --- If your project is set up for it, you can reply

[GitHub] incubator-fluo issue #796: Improved documentation for viewing Fluo logs

2016-11-01 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/pull/796 May want to document the issues with logback vs log4j when running in Yarn, with some background on this issue. --- If your project is set up for it, you can reply to this email

[GitHub] incubator-fluo-recipes issue #124: Looking into adding recipes for Apache Ka...

2016-10-31 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-recipes/issues/124 Looking into adding recipes for Apache Kafka export and load. Not sure if there is anything to do here, just want to look into it. --- If your project is set up

[GitHub] incubator-fluo-recipes issue #123: Looking into adding recipes for Apache Ni...

2016-10-31 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-recipes/issues/123 Looking into adding recipes for Apache Nifi export and load. Not sure if there anything to do here. I just want to look into this. --- If your project is set up

[GitHub] incubator-fluo-recipes pull request #121: update description

2016-10-28 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-recipes/pull/121 update description You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo-recipes doc_summary_update

[GitHub] incubator-fluo-recipes pull request #120: fix link in docs

2016-10-28 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-recipes/pull/120 fix link in docs You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo-recipes doc-fix Alternatively you can

[GitHub] incubator-fluo-website pull request #35: added Kryo to powered by, it used b...

2016-10-28 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/35 added Kryo to powered by, it used by recipes You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/incubator-fluo

[GitHub] incubator-fluo-recipes issue #119: Look into using Chill

2016-10-28 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-recipes/issues/119 Look into using Chill https://github.com/twitter/chill --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-fluo-website issue #34: Add twitter and facebook sharing to each b...

2016-10-27 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-website/issues/34 Add twitter and facebook sharing to each blog post @ctubbsii suggested this in an offline conversation. --- If your project is set up for it, you can reply

[GitHub] incubator-fluo-recipes pull request #118: Added link to Balancing blog post

2016-10-27 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-recipes/pull/118 Added link to Balancing blog post You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo-recipes balancing-link

[GitHub] incubator-fluo-website pull request #32: Mention that tx are cross node on f...

2016-10-25 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/32 Mention that tx are cross node on front page You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/incubator-fluo

[GitHub] incubator-fluo-website issue #31: Add twitter link

2016-10-25 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-website/issues/31 Add twitter link I just created a twitter account for [ApacheFluo](https://twitter.com/apachefluo). Should link form website. --- If your project is set up

[GitHub] incubator-fluo pull request #794: fixes #762 improve how test waits for prop...

2016-10-25 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo/pull/794 fixes #762 improve how test waits for propogation of oldest timestamp You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner

[GitHub] incubator-fluo issue #716: Release fluo-parent-1-incubating (parent POM)

2016-10-25 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/716 @ctubbsii can this be closed? --- 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

[GitHub] incubator-fluo issue #793: Modify ITs to use Junit timeout rule

2016-10-25 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo/issues/793 Modify ITs to use Junit timeout rule The currents ITs should be modified to use Junit timeout rules like in #792.Thinking it would be nice to do the following for all ITs

[GitHub] incubator-fluo pull request #792: #791 give Oracle more time to stop in IT

2016-10-24 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo/pull/792 #791 give Oracle more time to stop in IT You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo fluo-791 Alternatively

[GitHub] incubator-fluo issue #791: OracleIT failed on travis.

2016-10-24 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo/issues/791 OracleIT failed on travis. Saw the following failure on TravisCI while building #789. I doubt the failure had anything to do with the changes made in #789. ``` Tests run: 6

[GitHub] incubator-fluo issue #790: Add verification code to RC email

2016-10-24 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo/issues/790 Check out the `Validating a Release` section of the following page. http://kylin.apache.org/development/howto_release.html --- If your project is set up for it, you can reply

[GitHub] incubator-fluo issue #790: Add verification code to RC email

2016-10-21 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo/issues/790 Add verification code to RC email The script that generates an RC and an email gives a suggested command to download the staging repo. If could also suggest a short script to verify sigs

[GitHub] incubator-fluo-recipes pull request #115: Correct naming in poms and NOTICE

2016-10-19 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-recipes/pull/115 Correct naming in poms and NOTICE You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo-recipes name-fix

[GitHub] incubator-fluo-recipes pull request #114: Document thrift dep in pom

2016-10-19 Thread keith-turner
Github user keith-turner closed the pull request at: https://github.com/apache/incubator-fluo-recipes/pull/114 --- 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

[GitHub] incubator-fluo-website pull request #30: Misc website updates

2016-10-19 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-website/pull/30 Misc website updates You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/incubator-fluo-website misc

[GitHub] incubator-fluo-recipes issue #114: Document thrift dep in pom

2016-10-18 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo-recipes/pull/114 > Don't need two spaces after period, and there's an extra space, which wrapped before that first comma on the third line. I manually wrapped. I can fix that

[GitHub] incubator-fluo-recipes pull request #114: Document thrift dep in pom

2016-10-18 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-recipes/pull/114 Document thrift dep in pom You can merge this pull request into a Git repository by running: $ git pull https://github.com/keith-turner/fluo-recipes document-thrift-dep

[GitHub] incubator-fluo-recipes issue #113: Misc release prep cleanup

2016-10-14 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo-recipes/pull/113 After I ran `mvn release:prepare` with these changes I see the following. ``` $ ls target/*.tar* target/fluo-recipes-1.0.0-incubating-source-release.tar.gz

[GitHub] incubator-fluo-recipes pull request #113: Misc release prep cleanup

2016-10-14 Thread keith-turner
GitHub user keith-turner opened a pull request: https://github.com/apache/incubator-fluo-recipes/pull/113 Misc release prep cleanup * removed prohibition on jar sealing, this necessitated some code changes * removed rat exclusions and added some licenses * fixed some

[GitHub] incubator-fluo-website pull request #27: Major refactoring of release posts

2016-10-14 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/27#discussion_r83466342 --- Diff: _posts/blog/2016-01-11-webindex-long-run.md --- @@ -1,6 +1,6 @@ --- title: "Running Webindex for 3 days o

[GitHub] incubator-fluo-website issue #26: Add linkchecker to readme

2016-10-13 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-website/issues/26 Add linkchecker to readme I used following command to check website... could add this to the readme ``` linkchecker --no-follow-url=".*apidocs.*" --check-ex

[GitHub] incubator-fluo-website issue #23: Fluo 1.0.0

2016-10-13 Thread keith-turner
Github user keith-turner commented on the issue: https://github.com/apache/incubator-fluo-website/pull/23 > Also, this raises the question... do we really need apidocs published on our site Personally, I like having the javadocs on the Fluo website. However I am not oppo

[GitHub] incubator-fluo-website pull request #23: Fluo 1.0.0

2016-10-13 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/23#discussion_r83218320 --- Diff: _layouts/default.html --- @@ -31,8 +31,10 @@ -Documentation

[GitHub] incubator-fluo-website pull request #23: Fluo 1.0.0

2016-10-13 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/23#discussion_r83222182 --- Diff: docs/fluo/1.0.0-incubating/metrics.md --- @@ -0,0 +1,117 @@ +--- +layout: fluo-doc +title: Fluo Metrics

[GitHub] incubator-fluo-website pull request #23: Fluo 1.0.0

2016-10-13 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/23#discussion_r83219860 --- Diff: _posts/release-notes/2016-09-27-fluo-1.0.0-incubating.md --- @@ -1,16 +1,19 @@ --- title: Fluo 1.0.0-incubating

[GitHub] incubator-fluo-website pull request #23: Fluo 1.0.0

2016-10-13 Thread keith-turner
Github user keith-turner commented on a diff in the pull request: https://github.com/apache/incubator-fluo-website/pull/23#discussion_r83219527 --- Diff: _posts/blog/2016-09-27-1.0.0-incubating-is-released.md --- @@ -0,0 +1,12 @@ +--- +title: "Fluo 1.0.0-incub

[GitHub] incubator-fluo-website issue #24: Add section on deplyoing app to Fluo Tour

2016-10-12 Thread keith-turner
GitHub user keith-turner opened an issue: https://github.com/apache/incubator-fluo-website/issues/24 Add section on deplyoing app to Fluo Tour It may be useful to add a page to the tour about using Uno to deploy a Fluo Application on a real system. --- If your project

  1   2   >