Skip to content

Commit

Permalink
Fix maven jenkins: Add explicit init for required tables in SQLQueryS…
Browse files Browse the repository at this point in the history
…uite

Sorry! I added this test at the last minute and failed to run it in maven as well.

Note that, this will probably not be sufficient to actually fix the maven jenkins build, as that does not use the dev/run-tests scripts.  We will need to configure it to also run dev/download-hive-tests.sh.  The other option would be to check in the tests as I suggested in the original PR. (I can do this if we agree its the right thing to do).

Long term it would probably be a good idea to also have maven run some sort of test env setup script so that we can decouple the test environment from the jenkins configuration.

Author: Michael Armbrust <michael@databricks.com>

Closes apache#191 from marmbrus/fixMaven and squashes the following commits:

3366e37 [Michael Armbrust] Fix maven jenkins: Add explicit init for required tables in SQLQuerySuite
  • Loading branch information
marmbrus authored and rxin committed Mar 21, 2014
1 parent 9aadcff commit e09139d
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,9 @@ import TestSQLContext._
import TestData._

class SQLQuerySuite extends QueryTest {
// Make sure the tables are loaded.
TestData

test("agg") {
checkAnswer(
sql("SELECT a, SUM(b) FROM testData2 GROUP BY a"),
Expand Down

0 comments on commit e09139d

Please sign in to comment.