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 https://github.com/apache/drill/blob/master/exec/java-exec/src/test/java/org/apache/drill/exec/store/pcap/TestPcapDecoder.java <https://github.com/apache/drill/blob/master/exec/java-exec/src/test/java/org/apache/drill/exec/store/pcap/TestPcapDecoder.java#L57> - I added at line 57 the code: @Test 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 org.apache.maven.plugins:maven-surefire-plugin:3.0.0-M2:test 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 < [email protected]> 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: [email protected] With regards, Apache Git Services
