TianYi ZHU
Fantoccini

  • Commonwealth Bank
  • Sydney, Australia
  • Joined on Nov 4, 2011
Jul 7, 2016
Fantoccini commented on pull request CommBank/ebenezer#146
@Fantoccini

@toddmowen yes I know, I checked, as 1.7 involved a big refractor on entire parquet code base, cloudera haven't applied the fixes.

Jun 17, 2016
Fantoccini commented on pull request CommBank/ebenezer#146
@Fantoccini

@vineethvarghese sorry I'm working on a project recently. will update the comment in the code and write more explanation of this change.

May 23, 2016
Fantoccini commented on pull request CommBank/ebenezer#146
@Fantoccini

Thanks @rowandavies In Vineeth's sample, he's bypassing the cascading/scalding layer, it's a little bit different from this implementation. I've p…

May 23, 2016
@Fantoccini
  • @Fantoccini fd7af27
    Add comment, re-write ParquetScroogeSource
May 20, 2016
Fantoccini commented on pull request CommBank/ebenezer#146
@Fantoccini

Thanks @rowandavies , I've update the PR with my questions, please kindly review it again.

May 20, 2016
Fantoccini commented on pull request CommBank/ebenezer#146
@Fantoccini

Few article says "Never extend case class", I'm not sure if we follow this rule in Omnia. It's impossible to add the new parameter after the p : St…

May 20, 2016
Fantoccini commented on pull request CommBank/ebenezer#146
@Fantoccini

For other classes extend FileSplit in Java always create a new record reader here, it's a way they create immutable instances, so I write this and …

May 20, 2016
@Fantoccini
  • @Fantoccini 5a6a9f1
    Minor changes and add one test case
May 13, 2016
May 10, 2016
Fantoccini commented on pull request CommBank/ebenezer#146
@Fantoccini

Thanks @rowandavies , no, it's not the same exception, here's the stack trace and the steps to reproduce it: I have 4 directories with parquet fil…

May 6, 2016
Fantoccini opened pull request CommBank/beeswax#8
@Fantoccini
Please do NOT merge this PR
1 commit with 1 addition and 0 deletions
May 6, 2016