From: <tho...@us...> - 2011-06-20 22:02:51
|
Revision: 4751 http://bigdata.svn.sourceforge.net/bigdata/?rev=4751&view=rev Author: thompsonbry Date: 2011-06-20 22:02:45 +0000 (Mon, 20 Jun 2011) Log Message: ----------- Per [1], I have taken the HA quorum ZK integration test out of CI since it is not yet supported. Also per [1], I have made testReallyLongLiteralRoundTrip() silently succeed in the QUADS branch. This will not be fixed in the QUADS branch, but it has already been fixed in the TERMS branch as part of the "large literals refactor". [1] https://sourceforge.net/apps/trac/bigdata/ticket/297 Modified Paths: -------------- branches/QUADS_QUERY_BRANCH/bigdata-jini/src/test/com/bigdata/quorum/zk/TestAll.java branches/QUADS_QUERY_BRANCH/bigdata-sails/src/test/com/bigdata/rdf/sail/tck/BigdataStoreTest.java branches/TERMS_REFACTOR_BRANCH/bigdata-sails/src/test/com/bigdata/rdf/sail/tck/BigdataStoreTest.java Modified: branches/QUADS_QUERY_BRANCH/bigdata-jini/src/test/com/bigdata/quorum/zk/TestAll.java =================================================================== --- branches/QUADS_QUERY_BRANCH/bigdata-jini/src/test/com/bigdata/quorum/zk/TestAll.java 2011-06-20 21:36:37 UTC (rev 4750) +++ branches/QUADS_QUERY_BRANCH/bigdata-jini/src/test/com/bigdata/quorum/zk/TestAll.java 2011-06-20 22:02:45 UTC (rev 4751) @@ -76,8 +76,8 @@ // unit tests for a singleton quorum. suite.addTestSuite(TestZkSingletonQuorumSemantics.class); - // unit tests for a highly available quorum. - suite.addTestSuite(TestZkHA3QuorumSemantics.class); + // FIXME Enable HA test suite again: unit tests for a highly available quorum. +// suite.addTestSuite(TestZkHA3QuorumSemantics.class); return suite; Modified: branches/QUADS_QUERY_BRANCH/bigdata-sails/src/test/com/bigdata/rdf/sail/tck/BigdataStoreTest.java =================================================================== --- branches/QUADS_QUERY_BRANCH/bigdata-sails/src/test/com/bigdata/rdf/sail/tck/BigdataStoreTest.java 2011-06-20 21:36:37 UTC (rev 4750) +++ branches/QUADS_QUERY_BRANCH/bigdata-sails/src/test/com/bigdata/rdf/sail/tck/BigdataStoreTest.java 2011-06-20 22:02:45 UTC (rev 4751) @@ -33,25 +33,18 @@ */ package com.bigdata.rdf.sail.tck; -import info.aduna.iteration.CloseableIteration; import info.aduna.iteration.Iteration; import info.aduna.iteration.Iterations; import java.io.File; import java.io.IOException; -import java.util.Iterator; import java.util.Properties; import org.apache.log4j.Logger; import org.openrdf.model.vocabulary.RDF; import org.openrdf.model.vocabulary.RDFS; -import org.openrdf.query.Binding; -import org.openrdf.query.BindingSet; -import org.openrdf.query.QueryEvaluationException; import org.openrdf.query.QueryLanguage; -import org.openrdf.query.algebra.TupleExpr; import org.openrdf.query.impl.EmptyBindingSet; -import org.openrdf.query.impl.MapBindingSet; import org.openrdf.query.parser.ParsedTupleQuery; import org.openrdf.query.parser.QueryParserUtil; import org.openrdf.sail.RDFStoreTest; @@ -206,39 +199,40 @@ } +// /** +// * FIXME This one is failing because of this code: +// * <code> +// * bindings.addBinding("Y", painter); +// * iter = con.evaluate(tupleExpr, null, bindings, false); +// * resultCount = verifyQueryResult(iter, 1); +// * </code> +// * Adding a binding for the "Y" variable causes that binding to appear in +// * the result set, even though "Y" is not one of the selected variables. This +// * is a bigdata bug and should be fixed. +// * +// * @see https://sourceforge.net/apps/trac/bigdata/ticket/254 +// */ +// @Override +// public void testQueryBindings() +// throws Exception +// { +// log.warn("FIXME"); +// super.testQueryBindings(); +// } + /** - * FIXME This one is failing because of this code: - * <code> - * bindings.addBinding("Y", painter); - * iter = con.evaluate(tupleExpr, null, bindings, false); - * resultCount = verifyQueryResult(iter, 1); - * </code> - * Adding a binding for the "Y" variable causes that binding to appear in - * the result set, even though "Y" is not one of the selected variables. This - * is a bigdata bug and should be fixed. + * FIXME This one is failing because we cannot handle literals longer than + * 65535 characters. This is a known issue. It has been resolved in the + * TERMS branch. * - * @see https://sourceforge.net/apps/trac/bigdata/ticket/254 + * @see https://sourceforge.net/apps/trac/bigdata/ticket/109 */ @Override - public void testQueryBindings() - throws Exception - { - log.warn("FIXME"); - super.testQueryBindings(); - } - - /** - * FIXME This one is failing because we cannot handle literals longer than - * 65535 characters. This is a known issue. - * - * @see https://sourceforge.net/apps/trac/bigdata/ticket/109 - */ - @Override public void testReallyLongLiteralRoundTrip() throws Exception { - log.warn("FIXME"); - super.testReallyLongLiteralRoundTrip(); + log.warn("FIXME: This is fixed in the TERMS branch."); +// super.testReallyLongLiteralRoundTrip(); } /** Modified: branches/TERMS_REFACTOR_BRANCH/bigdata-sails/src/test/com/bigdata/rdf/sail/tck/BigdataStoreTest.java =================================================================== --- branches/TERMS_REFACTOR_BRANCH/bigdata-sails/src/test/com/bigdata/rdf/sail/tck/BigdataStoreTest.java 2011-06-20 21:36:37 UTC (rev 4750) +++ branches/TERMS_REFACTOR_BRANCH/bigdata-sails/src/test/com/bigdata/rdf/sail/tck/BigdataStoreTest.java 2011-06-20 22:02:45 UTC (rev 4751) @@ -220,19 +220,19 @@ super.testQueryBindings(); } - /** - * FIXME This one is failing because we cannot handle literals longer than - * 65535 characters. This is a known issue. - * - * @see https://sourceforge.net/apps/trac/bigdata/ticket/109 - */ - @Override - public void testReallyLongLiteralRoundTrip() - throws Exception - { - log.warn("FIXME"); - super.testReallyLongLiteralRoundTrip(); - } +// /** +// * FIXME This one is failing because we cannot handle literals longer than +// * 65535 characters. This is a known issue. +// * +// * @see https://sourceforge.net/apps/trac/bigdata/ticket/109 +// */ +// @Override +// public void testReallyLongLiteralRoundTrip() +// throws Exception +// { +// log.warn("FIXME"); +// super.testReallyLongLiteralRoundTrip(); +// } /** * Bigdata uses snapshot isolation for transactions while openrdf assumes This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |