/
Testing
---
added code test and added QL test. Tests passed in CI, but other, unrelated
tests failed.
Thanks,
david seraf
3612
-------
On Sept. 12, 2014, 9:51 p.m., david seraf wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25178/
> -
> > Also, both methods are new and private. I'm not sure if I undertand the
> > complexity of the change.
>
> david seraf wrote:
> I misunderstood. I'm fine with changing those 2 methods.
I looked at this in more detail and the problem is that drop
y generated e-mail. To reply, visit:
https://reviews.apache.org/r/25178/#review53259
---
On Sept. 12, 2014, 9:51 p.m., david seraf wrote:
>
> ---
> Thi
/drop_table_purge.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/25178/diff/
Testing
---
added code test and added QL test. Tests passed in CI, but other, unrelated
tests failed.
Thanks,
david seraf
for the solution to be complete. We'll
> > need a new `dropPartitions()` overload that takes an ifPurge flag. This
> > method should forward to the new method.
>
> david seraf wrote:
> I'm looking at it, but I think it may be a large enough change to justify
e-mail. To reply, visit:
https://reviews.apache.org/r/25178/#review52911
-------
On Sept. 12, 2014, 8:28 p.m., david seraf wrote:
>
> ---
> This is an automatically generated e
. Tests passed in CI, but other, unrelated
tests failed.
Thanks,
david seraf
/drop_table_purge.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/25178/diff/
Testing
---
added code test and added QL test. Tests passed in CI, but other, unrelated
tests failed.
Thanks,
david seraf
PRE-CREATION
Diff: https://reviews.apache.org/r/25178/diff/
Testing
---
added code test and added QL test. Tests passed in CI, but other, unrelated
tests failed.
Thanks,
david seraf
7;ll
> > need a new `dropPartitions()` overload that takes an ifPurge flag. This
> > method should forward to the new method.
I'm looking at it, but I think it may be a large enough change to justify a
separate ticket.
- david
--------
--
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25178/#review52279
-------
On Sept. 2, 2014, 11:41 p.m., david seraf wrote:
>
>
://reviews.apache.org/r/25178/diff/
Testing
---
added code test and added QL test. Tests passed in CI, but other, unrelated
tests failed.
Thanks,
david seraf
/25178/diff/
Testing
---
added code test and added QL test. Tests passed in CI, but other, unrelated
tests failed.
Thanks,
david seraf
.
Thanks,
david seraf
15 matches
Mail list logo
