Todd Lipcon has posted comments on this change. Change subject: Ported delta encoding from Impala to KUDU. ......................................................................
Patch Set 7: Working on going through this patch for another round of reviews. We should also get some perf/size numbers for a time series use case before committing this -- since encodings "live forever", we shoudl make sure that it's a substantial improvement over bitshuffle for at least one real workload. Otherwise, we could consider committing it behind some kind of flag like "--enable_experimental_encodings" and not commit to maintaining compatibility until it's been made non-experimental? -- To view, visit http://gerrit.cloudera.org:8080/1210 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1446a78f22773c28a7cc877fbe861697e39b2af8 Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: [email protected] Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: No
