drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] jculio commented on issue #1665: DRILL-7054: timestamp in milliseconds
Date Fri, 01 Mar 2019 19:13:04 GMT
jculio commented on issue #1665: DRILL-7054: timestamp in milliseconds
URL: https://github.com/apache/drill/pull/1665#issuecomment-468777946
   I tied to follow your instruction by doing:
   - I created the same file
   - I added at line 57 the code:
   public void testMicrotQuery() throws Exception {
     runSQLVerifyCount("select `timestamp_micro` from
   dfs.`store/pcap/cap-test.pcap`", 1);
   where cap-test.pcap is the file I added in the pcap folder.
   -Then I run the command
   mvn -Dtest=TestPcapDecoder#testMicrotQuery test and I got the error:
   Failed to execute goal
   Failed to execute goal
   org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M2:test (default-test)
   on project drill-protocol: No tests were executed!
   First time running/writing unit test, so I will need help.
   Thank you very much
   Il giorno ven 1 mar 2019 alle ore 15:27 Charles S. Givre <
   notifications@github.com> ha scritto:
   > HI @jculio <https://github.com/jculio>
   > Take a look at:
   > https://github.com/apache/drill/blob/master/exec/java-exec/src/test/java/org/apache/drill/exec/store/pcap/TestPcapDecoder.java#L57
   > .
   > Specifically here:
   > https://github.com/apache/drill/blob/fee4b1cdc1a1bf4ac3dfdbbf435a5696540ce59a/exec/java-exec/src/test/java/org/apache/drill/exec/store/pcap/TestPcapDecoder.java#L57
   > You can just add one line to call your new function and then verify the
   > result. Once you've done that, run the test locally to make sure that it
   > works:
   > mvn =Dtest=TestPcapDecoder#testBasics test and assuming it does, push it
   > to your remote repo.
   > As @vdiravka <https://github.com/vdiravka> said, you'll need to squash
   > your commits into one. Here's how: After verifying that everything is
   > working, count how many commits you have made.
   > Then:
   > git reset --soft HEAD~3
   > git commit -m "DRILL-7054: timestamp in milliseconds"
   > git push origin +master
   > The first line there uncommits the last n commits you made. The second
   > line then recommits everything into one commit. Note that Drill requires
   > commits to have a message identifying the PR. The final line pushes
   > everything back to github and overwrites the previous commits on github
   > with one commit.
   > Thank you very much for this contribution!!
   > —
   > You are receiving this because you were mentioned.
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/drill/pull/1665#issuecomment-468681971>, or mute
   > the thread
   > <https://github.com/notifications/unsubscribe-auth/At31l2Qek0NUqNA4H8HUZnPXxf7-orlEks5vSTjEgaJpZM4bXgAB>
   > .

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

With regards,
Apache Git Services

View raw message