Hey Keith, Just curious because of all the activity around this change in 1.10 (the subsequent partial revert and the merge commits to main, which seem to be -sours), what motivated the change to TabletIteratorTest in the older branch?
On Thu, Aug 5, 2021 at 6:40 PM <ktur...@apache.org> wrote: > > This is an automated email from the ASF dual-hosted git repository. > > kturner pushed a commit to branch 1.10 > in repository https://gitbox.apache.org/repos/asf/accumulo.git > > > The following commit(s) were added to refs/heads/1.10 by this push: > new 5d475b0 removes extraneous code from TabletIteratorTest > 5d475b0 is described below > > commit 5d475b00eabf9aa419dbc49d5a49465633a61815 > Author: Keith Turner <ktur...@apache.org> > AuthorDate: Thu Aug 5 18:37:12 2021 -0400 > > removes extraneous code from TabletIteratorTest > --- > server/base/.gitignore | 1 + > .../java/org/apache/accumulo/server/util/TabletIteratorTest.java | 6 > +----- > server/tserver/.gitignore | 1 + > test/.gitignore | 1 + > 4 files changed, 4 insertions(+), 5 deletions(-) > > diff --git a/server/base/.gitignore b/server/base/.gitignore > index e77a822..320dd27 100644 > --- a/server/base/.gitignore > +++ b/server/base/.gitignore > @@ -26,3 +26,4 @@ > /nbproject/ > /nbactions.xml > /nb-configuration.xml > +/bin/ > diff --git > a/server/base/src/test/java/org/apache/accumulo/server/util/TabletIteratorTest.java > > b/server/base/src/test/java/org/apache/accumulo/server/util/TabletIteratorTest.java > index 43888af..b60630d 100644 > --- > a/server/base/src/test/java/org/apache/accumulo/server/util/TabletIteratorTest.java > +++ > b/server/base/src/test/java/org/apache/accumulo/server/util/TabletIteratorTest.java > @@ -121,12 +121,8 @@ public class TabletIteratorTest { > createTabletData(data1, "3", "c", null); > createTabletData(data1, "3", "n", "c"); > > - TreeMap<Key,Value> data2 = new TreeMap<>(data1); > - > - createTabletData(data2, "3", null, "n"); > - > assertThrows(IllegalStateException.class, > - () -> runTest(Arrays.asList(data1, data2), Arrays.asList("3;c", > "3;n", "3<"))); > + () -> runTest(Arrays.asList(data1), Arrays.asList("3;c", "3;n"))); > } > > @Test > diff --git a/server/tserver/.gitignore b/server/tserver/.gitignore > index e77a822..320dd27 100644 > --- a/server/tserver/.gitignore > +++ b/server/tserver/.gitignore > @@ -26,3 +26,4 @@ > /nbproject/ > /nbactions.xml > /nb-configuration.xml > +/bin/ > diff --git a/test/.gitignore b/test/.gitignore > index 87da2f9..c92e5e7 100644 > --- a/test/.gitignore > +++ b/test/.gitignore > @@ -30,3 +30,4 @@ > # python ignores > *.pyc > > +/bin/