Istvan Toth
Istvan Toth
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |:----:|----------:|--------:|:--------| | +0 :ok: | reexec | 0m 32s | Docker mode activated. | ||| _ Prechecks _...
That failure sounds like PHOENIX-6552, @gjacoby126 , which is a simple infra overload issue (though the test is also overly optimistic wrt GC pauses, etc.)
It has been removed from the addColumnMutation() setup, and the addColumnMutation() method.
As we have no IT for it, the real test is to manully take a dump of a skeleton Phonix 4.7 install, load it, and test if the upgrade is...
I like that you put the logic inside the addColumnMutationInternal() method. LGTM, but let's add the test that we've discussed offline.
I've seen Yetus have trouble with multiple commits in a PR before. You may want to try to squash your changes. (Maybe in another PR, if you don't want to...
Please do not use private JIRA IDs in the Apache project. Use the PHOENIX-300 ticket ID.
There are a few issues to be addressed: - child tables (global secondary index tables and and global secondary indexes on views) also need to be handled. - This doesn't...
You are preserving the splits during truncate, which is good, but you may also want to consider exposing an option that lets users remove the existing split points. (not for...
That sounds good, @ragarkar .