[GitHub] [incubator-hudi] bhasudha commented on issue #818: Fix typo in hoodie-presto-bundle

2019-08-02 Thread GitBox
bhasudha commented on issue #818: Fix typo in hoodie-presto-bundle URL: https://github.com/apache/incubator-hudi/pull/818#issuecomment-517885986 > I was getting some java import error involving Jackson which was resolved after I removed

[GitHub] [incubator-hudi] vinothchandar commented on issue #823: [HUDI-196] First draft of committership guidelines

2019-08-02 Thread GitBox
vinothchandar commented on issue #823: [HUDI-196] First draft of committership guidelines URL: https://github.com/apache/incubator-hudi/pull/823#issuecomment-517870843 All, please provide feedback This is an automated

[incubator-hudi] branch master updated: - Fix realtime queries by removing COLUMN_ID and COLUMN_NAME cache in inputformat (#814)

2019-08-02 Thread vinoth
This is an automated email from the ASF dual-hosted git repository. vinoth pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git The following commit(s) were added to refs/heads/master by this push: new 1a29d46 - Fix realtime queries by

[GitHub] [incubator-hudi] vinothchandar commented on issue #814: HUDI-198 : Fix for realtime queries

2019-08-02 Thread GitBox
vinothchandar commented on issue #814: HUDI-198 : Fix for realtime queries URL: https://github.com/apache/incubator-hudi/pull/814#issuecomment-517868026 Added a comment on HUDI-198, linking to PR This is an automated message

[GitHub] [incubator-hudi] bvaradar commented on issue #825: HUDI-149 - Remove platform dependencies and update NOTICE plugin

2019-08-02 Thread GitBox
bvaradar commented on issue #825: HUDI-149 - Remove platform dependencies and update NOTICE plugin URL: https://github.com/apache/incubator-hudi/pull/825#issuecomment-517862964 Does this fix the issue with NOTICE files self updating on build? the one we hit before and disabled the plugin?

[GitHub] [incubator-hudi] n3nash commented on issue #814: Fix for realtime queries

2019-08-02 Thread GitBox
n3nash commented on issue #814: Fix for realtime queries URL: https://github.com/apache/incubator-hudi/pull/814#issuecomment-517862260 I've captured it already in the commit message. I've updated this JIRA to reflect this : https://issues.apache.org/jira/browse/HUDI-151.

[GitHub] [incubator-hudi] vinothchandar commented on issue #814: Fix for realtime queries

2019-08-02 Thread GitBox
vinothchandar commented on issue #814: Fix for realtime queries URL: https://github.com/apache/incubator-hudi/pull/814#issuecomment-517860861 @n3nash JIRA please? This is an automated message from the Apache Git Service. To

[GitHub] [incubator-hudi] vinothchandar edited a comment on issue #814: Fix for realtime queries

2019-08-02 Thread GitBox
vinothchandar edited a comment on issue #814: Fix for realtime queries URL: https://github.com/apache/incubator-hudi/pull/814#issuecomment-517860861 @n3nash JIRA please? and can you capture that hive-on-spark is broken in the commit message for traceability?

[GitHub] [incubator-hudi] bvaradar commented on issue #825: HUDI-149 - Remove platform dependencies and update NOTICE plugin

2019-08-02 Thread GitBox
bvaradar commented on issue #825: HUDI-149 - Remove platform dependencies and update NOTICE plugin URL: https://github.com/apache/incubator-hudi/pull/825#issuecomment-517839366 @vinothchandar @n3nash : Ready for review This

[GitHub] [incubator-hudi] vinothchandar commented on issue #802: Cache RDD to avoid recomputing data ingestion and hbase puts actions

2019-08-02 Thread GitBox
vinothchandar commented on issue #802: Cache RDD to avoid recomputing data ingestion and hbase puts actions URL: https://github.com/apache/incubator-hudi/pull/802#issuecomment-517828653 @v3nkatesh can you please create a JIRA as well and link this PR there..

[GitHub] [incubator-hudi] n3nash merged pull request #802: Cache RDD to avoid recomputing data ingestion and hbase puts actions

2019-08-02 Thread GitBox
n3nash merged pull request #802: Cache RDD to avoid recomputing data ingestion and hbase puts actions URL: https://github.com/apache/incubator-hudi/pull/802 This is an automated message from the Apache Git Service. To

[incubator-hudi] branch master updated: Cache RDD to avoid recomputing data ingestion. Return result RDD after updating index so that this step is not skipped by chained actions on the same RDD

2019-08-02 Thread nagarwal
This is an automated email from the ASF dual-hosted git repository. nagarwal pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git The following commit(s) were added to refs/heads/master by this push: new 86b5fcd Cache RDD to avoid

[GitHub] [incubator-hudi] n3nash commented on issue #809: HUDI-180 : Adding support for hive registration using metastore along with JDBC

2019-08-02 Thread GitBox
n3nash commented on issue #809: HUDI-180 : Adding support for hive registration using metastore along with JDBC URL: https://github.com/apache/incubator-hudi/pull/809#issuecomment-517820333 @vinothchandar @bvaradar fixed checkstyle and added comments. Look at the jira for more detailed

[GitHub] [incubator-hudi] n3nash commented on issue #811: HUDI-182 : Adding HoodieCombineHiveInputFormat for COW tables

2019-08-02 Thread GitBox
n3nash commented on issue #811: HUDI-182 : Adding HoodieCombineHiveInputFormat for COW tables URL: https://github.com/apache/incubator-hudi/pull/811#issuecomment-517820193 @bvaradar @vinothchandar Fixed checkstyle and added comment about hive version

[GitHub] [incubator-hudi] n3nash commented on issue #814: Fix for realtime queries

2019-08-02 Thread GitBox
n3nash commented on issue #814: Fix for realtime queries URL: https://github.com/apache/incubator-hudi/pull/814#issuecomment-517819134 Yes, hive on spark is back to being broken (as it was before we made these changes) This

[GitHub] [incubator-hudi] n3nash commented on issue #802: Cache RDD to avoid recomputing data ingestion and hbase puts actions

2019-08-02 Thread GitBox
n3nash commented on issue #802: Cache RDD to avoid recomputing data ingestion and hbase puts actions URL: https://github.com/apache/incubator-hudi/pull/802#issuecomment-517818408 Merging this PR understanding that close() isn't the best method for an index but we couldn't find another

[GitHub] [incubator-hudi] vinothchandar commented on issue #814: Fix for realtime queries

2019-08-02 Thread GitBox
vinothchandar commented on issue #814: Fix for realtime queries URL: https://github.com/apache/incubator-hudi/pull/814#issuecomment-517800177 >how does this not hit the earlier issue you cited with multiple tasks running on the same executor? In other words, is hive-on-spark on RT

[GitHub] [incubator-hudi] n3nash commented on issue #814: Fix for realtime queries

2019-08-02 Thread GitBox
n3nash commented on issue #814: Fix for realtime queries URL: https://github.com/apache/incubator-hudi/pull/814#issuecomment-517796993 @vinothchandar Not sure about your confusion. Basically, I've reverted the change that I had made to enable realtime to work on Hive on Spark. So after

[GitHub] [incubator-hudi] bvaradar commented on issue #825: [WIP] HUDI-149 - Remove platform dependencies and update NOTICE plugin

2019-08-02 Thread GitBox
bvaradar commented on issue #825: [WIP] HUDI-149 - Remove platform dependencies and update NOTICE plugin URL: https://github.com/apache/incubator-hudi/pull/825#issuecomment-517793064 @vinothchandar @n3nash : looks like some unit test failure. Please hold on to this review

[GitHub] [incubator-hudi] bvaradar commented on issue #825: HUDI-149 - Remove platform dependencies and update NOTICE plugin

2019-08-02 Thread GitBox
bvaradar commented on issue #825: HUDI-149 - Remove platform dependencies and update NOTICE plugin URL: https://github.com/apache/incubator-hudi/pull/825#issuecomment-517783249 @vinothchandar @n3nash : Forgot to mention that ITTestHoodieDemo (demo automation ) passes with this change.

[GitHub] [incubator-hudi] bvaradar opened a new pull request #825: HUDI-149 - Remove platform dependencies and update NOTICE plugin

2019-08-02 Thread GitBox
bvaradar opened a new pull request #825: HUDI-149 - Remove platform dependencies and update NOTICE plugin URL: https://github.com/apache/incubator-hudi/pull/825 Details in https://issues.apache.org/jira/browse/LEGAL-461 HUDI-149 - Remove platform dependencies and update NOTICE

[incubator-hudi] branch asf-site updated: HUDI-185 Replace local hudi logo with the one added to apache (#824)

2019-08-02 Thread vinoth
This is an automated email from the ASF dual-hosted git repository. vinoth pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git The following commit(s) were added to refs/heads/asf-site by this push: new 5a6a1ad HUDI-185 Replace local

[GitHub] [incubator-hudi] vinothchandar merged pull request #824: HUDI-185 Replace local hudi logo with the one added to apache

2019-08-02 Thread GitBox
vinothchandar merged pull request #824: HUDI-185 Replace local hudi logo with the one added to apache URL: https://github.com/apache/incubator-hudi/pull/824 This is an automated message from the Apache Git Service. To

[incubator-hudi] branch master updated: HUDI-197 Hive Sync and othe CLIs using bundle picking sources jar instead of binary jar

2019-08-02 Thread vbalaji
This is an automated email from the ASF dual-hosted git repository. vbalaji pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git The following commit(s) were added to refs/heads/master by this push: new 8139ffd HUDI-197 Hive Sync and othe

[GitHub] [incubator-hudi] bvaradar merged pull request #822: HUDI-197 Hive Sync and othe CLIs using bundle picking sources jar instead of binary jar

2019-08-02 Thread GitBox
bvaradar merged pull request #822: HUDI-197 Hive Sync and othe CLIs using bundle picking sources jar instead of binary jar URL: https://github.com/apache/incubator-hudi/pull/822 This is an automated message from the Apache

[GitHub] [incubator-hudi] bvaradar commented on issue #816: [HUDI-121] Add lresende signing key to KEYS file

2019-08-02 Thread GitBox
bvaradar commented on issue #816: [HUDI-121] Add lresende signing key to KEYS file URL: https://github.com/apache/incubator-hudi/pull/816#issuecomment-517757553 No longer needed. This is an automated message from the Apache

[GitHub] [incubator-hudi] bvaradar closed pull request #816: [HUDI-121] Add lresende signing key to KEYS file

2019-08-02 Thread GitBox
bvaradar closed pull request #816: [HUDI-121] Add lresende signing key to KEYS file URL: https://github.com/apache/incubator-hudi/pull/816 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-hudi] bvaradar opened a new pull request #824: HUDI-185 Replace local hudi logo with the one added to apache

2019-08-02 Thread GitBox
bvaradar opened a new pull request #824: HUDI-185 Replace local hudi logo with the one added to apache URL: https://github.com/apache/incubator-hudi/pull/824 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-hudi] vinothchandar commented on issue #822: HUDI-197 Hive Sync and othe CLIs using bundle picking sources jar instead of binary jar

2019-08-02 Thread GitBox
vinothchandar commented on issue #822: HUDI-197 Hive Sync and othe CLIs using bundle picking sources jar instead of binary jar URL: https://github.com/apache/incubator-hudi/pull/822#issuecomment-517682090 LGTM. you can merge once CI passes

[GitHub] [incubator-hudi] vinothchandar opened a new pull request #823: [HUDI-196] First draft of committership guidelines

2019-08-02 Thread GitBox
vinothchandar opened a new pull request #823: [HUDI-196] First draft of committership guidelines URL: https://github.com/apache/incubator-hudi/pull/823 - Minor details on JIRA usage - Updated html files This is an

[incubator-hudi] branch master updated: HUDI-178 : Add keys for vinoth to KEYS file

2019-08-02 Thread vbalaji
This is an automated email from the ASF dual-hosted git repository. vbalaji pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git The following commit(s) were added to refs/heads/master by this push: new 8ddfa2e HUDI-178 : Add keys for

[GitHub] [incubator-hudi] bvaradar merged pull request #821: HUDI-178 : Add keys for vinoth to KEYS file

2019-08-02 Thread GitBox
bvaradar merged pull request #821: HUDI-178 : Add keys for vinoth to KEYS file URL: https://github.com/apache/incubator-hudi/pull/821 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-hudi] bvaradar opened a new pull request #822: HUDI-197 Hive Sync and othe CLIs using bundle picking sources jar instead of binary jar

2019-08-02 Thread GitBox
bvaradar opened a new pull request #822: HUDI-197 Hive Sync and othe CLIs using bundle picking sources jar instead of binary jar URL: https://github.com/apache/incubator-hudi/pull/822 HUDI-197 Hive Sync and othe CLIs using bundle picking sources jar instead of binary jar

[GitHub] [incubator-hudi] vinothchandar opened a new pull request #821: HUDI-178 : Add keys for vinoth to KEYS file

2019-08-02 Thread GitBox
vinothchandar opened a new pull request #821: HUDI-178 : Add keys for vinoth to KEYS file URL: https://github.com/apache/incubator-hudi/pull/821 This is an automated message from the Apache Git Service. To respond to the