James Xu

Results 30 comments of James Xu

also, add some unit tests for this new feature.

tests are placed in `storm-core/test/clj/backtype/storm`, tests are all writen in clojure, and run the tests with `lein test`

Cool! I think we are almost there except that those pure whitespace-changes need to be removed(`project.clj`, `bootstrap.clj`, `Config.java`).

For Config.java, I think the easiest way is to revert the current changes and add your new STORM_MULTIANG_SERIALIZER back.

+1 for adding this PR into `0.9.1`

Oh, I think there is nothing more you need to do :) I am just voting to add your PR to the next release. Current release(`0.9.0`) only accept bugfix PR...

@yingsu00 The whole stack trace(from Presto UI) is: ``` std::exception: scalar type has no children ``` No log in the native worker's log. More clue: - `analyze customer` success. -...

@8dukongjian Very informative, thanks! I'd suggest submit issues for each problem you found with detailed description and reproducible case, and use this issue as an umbrella issue to track all...

> I can start adding something to TableScan to implement this then > > So the idea is to accumulate the output of operator instead of input (except the Exchange...

@qqibrow This is a very informative post, thanks! What's the status of Parquet support in Velox from your perspective?