[GitHub] phoenix pull request: PHOENIX-2517 Bulk load tools should support ...

2015-12-11 Thread ndimiduk
GitHub user ndimiduk opened a pull request: https://github.com/apache/phoenix/pull/145 PHOENIX-2517 Bulk load tools should support multiple input files You can merge this pull request into a Git repository by running: $ git pull https://github.com/ndimiduk/phoenix 2517

[GitHub] phoenix pull request: PHOENIX-2495 Fix test failures in CsvBulkLoa...

2015-12-10 Thread ndimiduk
Github user ndimiduk closed the pull request at: https://github.com/apache/phoenix/pull/143 --- 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

[GitHub] phoenix pull request: PHOENIX-2495 Fix test failures in CsvBulkLoa...

2015-12-09 Thread ndimiduk
GitHub user ndimiduk opened a pull request: https://github.com/apache/phoenix/pull/143 PHOENIX-2495 Fix test failures in CsvBulkLoadToolIT FYI @JamesRTaylor @twdsilva You can merge this pull request into a Git repository by running: $ git pull https://github.com/ndimiduk

[GitHub] phoenix pull request: PHOENIX-2492 Expose PhoenixRecordWritable ou...

2015-12-07 Thread ndimiduk
Github user ndimiduk commented on the pull request: https://github.com/apache/phoenix/pull/142#issuecomment-162668915 Please help yourself @jmahonin. I've been able to move forward with my own project via the class in this PR. I should be getting into some spark stuff later this

[GitHub] phoenix pull request: PHOENIX-2481 JSON bulkload tool

2015-12-07 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/141#discussion_r46853590 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/JsonBulkLoadTool.java --- @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request: PHOENIX-2481 JSON bulkload tool

2015-12-07 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/141#discussion_r46853357 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/JsonToKeyValueMapper.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request: PHOENIX-2492 Expose PhoenixRecordWritable ou...

2015-12-05 Thread ndimiduk
Github user ndimiduk commented on the pull request: https://github.com/apache/phoenix/pull/142#issuecomment-162270732 Sounds like we have a game-plan to proceed. I'm happy to pursue the work of consolidation, though I'm concerned about touching (and breaking) so many

[GitHub] phoenix pull request: PHOENIX-2492 Expose PhoenixRecordWritable ou...

2015-12-04 Thread ndimiduk
GitHub user ndimiduk opened a pull request: https://github.com/apache/phoenix/pull/142 PHOENIX-2492 Expose PhoenixRecordWritable outside of spark This is a simple translation of phoenix-spark's `PhoenixRecordWritable` to java, and has been helpful for my needs. Probably the

[GitHub] phoenix pull request: PHOENIX-2481 JSON bulkload tool

2015-12-04 Thread ndimiduk
Github user ndimiduk commented on the pull request: https://github.com/apache/phoenix/pull/141#issuecomment-162108877 Okay. Went through all javadocs. Also rebased onto master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] phoenix pull request: PHOENIX-2481 JSON bulkload tool

2015-12-04 Thread ndimiduk
Github user ndimiduk commented on the pull request: https://github.com/apache/phoenix/pull/141#issuecomment-162033375 > For JSON, is only single level supported? If not, how is nested JSON handled? No special handling is provided. A nested json object will be read as Map

[GitHub] phoenix pull request: PHOENIX-2481 JSON bulkload tool

2015-12-02 Thread ndimiduk
Github user ndimiduk commented on the pull request: https://github.com/apache/phoenix/pull/141#issuecomment-161410084 Add a tool for loading json documents (one record per line) and refactor code in common between JSON and CSV loading. --- If your project is set up for it, you can

[GitHub] phoenix pull request: PHOENIX-2481 JSON bulkload tool

2015-12-02 Thread ndimiduk
GitHub user ndimiduk opened a pull request: https://github.com/apache/phoenix/pull/141 PHOENIX-2481 JSON bulkload tool You can merge this pull request into a Git repository by running: $ git pull https://github.com/ndimiduk/phoenix 2481-json-bulkload-tool Alternatively you

[GitHub] phoenix pull request: PHOENIX-2187: Adding front-end of Tracing We...

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/112#discussion_r37574069 --- Diff: phoenix-tracing-webapp/src/build/build.xml --- @@ -0,0 +1,40

[GitHub] phoenix pull request: PHOENIX-2187: Adding front-end of Tracing We...

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/112#discussion_r37573877 --- Diff: phoenix-tracing-webapp/pom.xml --- @@ -0,0 +1,171 @@ + + + + http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37573572 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/TraceServlet.java --- @@ -0,0 +1,160 @@ +/* + * Licensed to

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37573414 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/TraceServlet.java --- @@ -0,0 +1,160 @@ +/* + * Licensed to

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37573194 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/Main.java --- @@ -0,0 +1,78 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37573048 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/Main.java --- @@ -0,0 +1,78 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37573002 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/Main.java --- @@ -0,0 +1,78 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37572962 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/Main.java --- @@ -0,0 +1,78 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37572381 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/EntityFactory.java --- @@ -0,0 +1,98 @@ +/* + * Licensed to

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37572345 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/EntityFactory.java --- @@ -0,0 +1,98 @@ +/* + * Licensed to

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37572197 --- Diff: phoenix-tracing-webapp/src/main/java/org/apache/phoenix/tracingwebapp/http/EntityFactory.java --- @@ -0,0 +1,98 @@ +/* + * Licensed to

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37572065 --- Diff: phoenix-tracing-webapp/pom.xml --- @@ -0,0 +1,173 @@ + + + + http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37571986 --- Diff: phoenix-tracing-webapp/src/build/build.xml --- @@ -0,0 +1,40 @@ + --- End diff -- Do we still need this ant file? We'r

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37571771 --- Diff: phoenix-tracing-webapp/pom.xml --- @@ -0,0 +1,173 @@ + + + + http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://

[GitHub] phoenix pull request: PHOENIX-2186 : Back end services

2015-08-20 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/111#discussion_r37571839 --- Diff: phoenix-tracing-webapp/pom.xml --- @@ -0,0 +1,173 @@ + + + + http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://

[GitHub] phoenix pull request: PHOENIX-2023 Build tar.gz only on release pr...

2015-08-19 Thread ndimiduk
Github user ndimiduk commented on the pull request: https://github.com/apache/phoenix/pull/110#issuecomment-132789228 I haven't done a phoenix release, but this looks like what I had in mind. Thanks for fixing the missing headers while you're in there -- these should be cau

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-05 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33898894 --- Diff: phoenix-tracing-webapp/pom.xml --- @@ -0,0 +1,73 @@ + + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi=

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-05 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33898864 --- Diff: phoenix-tracing-webapp/src/main/webapp/index.html --- @@ -0,0 +1,114

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-05 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33898867 --- Diff: phoenix-tracing-webapp/pom.xml --- @@ -0,0 +1,73 @@ + + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi=

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-03 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33877074 --- Diff: phoenix-tracing-webapp/src/main/webapp/index.html --- @@ -0,0 +1,114

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-03 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33877027 --- Diff: phoenix-tracing-webapp/src/main/webapp/css/nv.d3.css --- @@ -0,0 +1,769 @@ + --- End diff -- What library is this? What is its

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-03 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33876991 --- Diff: phoenix-tracing-webapp/src/main/java/org/example/Tracing.java --- @@ -0,0 +1,56 @@ +/* + * Licensed to the Apache Software Foundation (ASF

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-03 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33876914 --- Diff: phoenix-tracing-webapp/src/main/java/org/example/Tracing.java --- @@ -0,0 +1,56 @@ +/* + * Licensed to the Apache Software Foundation (ASF

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-03 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33876874 --- Diff: phoenix-tracing-webapp/src/main/java/org/example/TraceServlet.java --- @@ -0,0 +1,42 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-03 Thread ndimiduk
Github user ndimiduk commented on a diff in the pull request: https://github.com/apache/phoenix/pull/96#discussion_r33876704 --- Diff: phoenix-tracing-webapp/pom.xml --- @@ -0,0 +1,73 @@ + + + +http://maven.apache.org/POM/4.0.0"; + xmlns:xsi=

[GitHub] phoenix pull request: Adding tracingweb app

2015-07-03 Thread ndimiduk
Github user ndimiduk commented on the pull request: https://github.com/apache/phoenix/pull/96#issuecomment-118404084 Yeah, this looks much better. --- 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

[GitHub] phoenix pull request: PHOENIX-1514 Break up PDataType Enum

2014-12-09 Thread ndimiduk
GitHub user ndimiduk opened a pull request: https://github.com/apache/phoenix/pull/29 PHOENIX-1514 Break up PDataType Enum You can merge this pull request into a Git repository by running: $ git pull https://github.com/ndimiduk/phoenix 1514-breakup-pdatatype-enum