Sure! I just submitted #1285 <https://github.com/apache/iceberg/pull/1285> to exclude the refactor. Once #1285 is merged I'll rebase the existing PR to do the refactor. Thanks for the input!
On Sun, Aug 2, 2020 at 4:41 AM Ryan Blue <rb...@netflix.com.invalid> wrote: > Thanks, Jungtaek! I agree it would be great to fix that problem. I took a > quick look at the PR and it is a little big to go into a patch release > since it refactors quite a few places to consolidate the list copy. What do > you think about making a PR that just fixes the problem with > BaseCombinedScanTask and Kryo, then doing the remainder of the refactor in > master? > > On Fri, Jul 31, 2020 at 5:29 PM Jungtaek Lim <kabhwan.opensou...@gmail.com> > wrote: > >> If we still have some more days I think #1280 >> <https://github.com/apache/iceberg/pull/1280>: "fix serialization issue >> in BaseCombinedScanTask with Kyro" is a good candidate to be included. The >> bug affects both Spark and Flink (according to #1279 >> <https://github.com/apache/iceberg/pull/1279>). >> >> On Sat, Aug 1, 2020 at 8:04 AM Ryan Blue <b...@apache.org> wrote: >> >>> Hi everyone, >>> >>> We’ve accumulated a few bug fixes in the last couple of weeks and I >>> think it might make sense to get some of them out in an 0.9.1 release since >>> they make it harder to work with Iceberg. Here are the ones I know about: >>> >>> - #1282 <https://github.com/apache/iceberg/pull/1282>: rewriteNot >>> fails for binary and unary predicates >>> - #1278 <https://github.com/apache/iceberg/pull/1278>: Bad import >>> from commons-compress causes query failures >>> - #1251 <https://github.com/apache/iceberg/pull/1251>: Fixes more >>> imports from non-Iceberg Guava >>> - #1283 <https://github.com/apache/iceberg/pull/1283>: Query >>> descriptions fail when IN predicates are pushed >>> - #1228 <https://github.com/apache/iceberg/pull/1228>: Data imports >>> fail when paths include whitespace >>> - #1194 <https://github.com/apache/iceberg/pull/1194>: USING should >>> set format when used in a CTAS command >>> - #1203 <https://github.com/apache/iceberg/pull/1203>: Table cache >>> should not expire >>> >>> If there are no objections, I’ll get started and create a release >>> branch. And please reply if there are other issues you’ve seen that should >>> also be included in a patch release. >>> >>> rb >>> -- >>> Ryan Blue >>> >> > > -- > Ryan Blue > Software Engineer > Netflix >
