You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xtable.apache.org by "rahil-c (via GitHub)" <gi...@apache.org> on 2025/05/12 19:57:22 UTC
[PR] Add intial IT test for XTable REST Service [incubator-xtable]
rahil-c opened a new pull request, #716:
URL: https://github.com/apache/incubator-xtable/pull/716
## *Important Read*
- *Please ensure the GitHub issue is mentioned at the beginning of the PR*
## What is the purpose of the pull request
*(For example: This pull request implements the sync for delta format.)*
## Brief change log
*(for example:)*
- *Fixed JSON parsing error when persisting state*
- *Added unit tests for schema evolution*
## Verify this pull request
*(Please pick either of the following options)*
This pull request is a trivial rework / code cleanup without any test coverage.
*(or)*
This pull request is already covered by existing tests, such as *(please describe tests)*.
(or)
This change added tests and can be verified as follows:
*(example:)*
- *Added integration tests for end-to-end.*
- *Added TestConversionController to verify the change.*
- *Manually verified the change by running a job locally.*
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add intial IT for XTable REST Service [incubator-xtable]
Posted by "rahil-c (via GitHub)" <gi...@apache.org>.
rahil-c commented on PR #716:
URL: https://github.com/apache/incubator-xtable/pull/716#issuecomment-3002136920
<img width="1008" alt="Screenshot 2025-06-24 at 4 20 00 PM" src="https://github.com/user-attachments/assets/a6c2bd3a-f086-4d35-9955-b04dd8780940" />
Put a fix again, ran the test locally passing, hoping CI passes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add intial IT for XTable REST Service [incubator-xtable]
Posted by "rahil-c (via GitHub)" <gi...@apache.org>.
rahil-c commented on PR #716:
URL: https://github.com/apache/incubator-xtable/pull/716#issuecomment-3002248755
CI did not fail for an error but it seemed like it was canceled?
```
Also ran from my local terminal instead of intellij `mvn clean test -Dtest=ITConversionService` and seems test pass fine, so wondering if we can trigger rerun for this?
INFO] T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.xtable.service.ITConversionService
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.apache.spark.unsafe.Platform (file:/Users/rahil/.m2/repository/org/apache/spark/spark-unsafe_2.12/3.4.2/spark-unsafe_2.12-3.4.2.jar) to constructor java.nio.DirectByteBuffer(long,int)
WARNING: Please consider reporting this to the maintainers of org.apache.spark.unsafe.Platform
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
[INFO] Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 102.1 s -- in org.apache.xtable.service.ITConversionService
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 6, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO]
[INFO] --- surefire:3.3.0:test (default) @ xtable-service ---
[INFO] Skipping execution of surefire because it has already been run for this configuration
[INFO] Saved Build to local file: /Users/rahil/.m2/build-cache/v1/org.apache.xtable/xtable-service/d9e9c1b952afb9bc/local/buildinfo.xml
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:51 min
[INFO] Finished at: 2025-06-24T17:37:15-07:00
[INFO] ------------------------------------------------------------------------
rahil@mac ~/workplace/incubator-xtable/xtable-service integ-tests
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add intial IT test for XTable REST Service [incubator-xtable]
Posted by "rahil-c (via GitHub)" <gi...@apache.org>.
rahil-c commented on PR #716:
URL: https://github.com/apache/incubator-xtable/pull/716#issuecomment-2918664841
Ensure why the IT itself is flaky on the CI as it passes fine locally.
<img width="2865" alt="Screenshot 2025-05-29 at 1 14 20 AM" src="https://github.com/user-attachments/assets/8d4f8fb6-a8ca-4e14-9161-419a5b23dea2" />
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add intial IT for XTable REST Service [incubator-xtable]
Posted by "vinishjail97 (via GitHub)" <gi...@apache.org>.
vinishjail97 commented on PR #716:
URL: https://github.com/apache/incubator-xtable/pull/716#issuecomment-2923153771
`Error: ITConversionService.testVariousOperations(String, boolean)[1] » Spark Job aborted due to stage failure: Task 0 in stage 36.0 failed 1 times, most recent failure: Lost task 0.0 in stage 36.0 (TID 104) (fv-az1983-640.xojnruca2udebm2tjfdy52k3le.bx.internal.cloudapp.net executor driver): scala.MatchError: TimestampNTZType (of class org.apache.spark.sql.types.TimestampNTZType$)`
Is this passing locally?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add intial IT for XTable REST Service [incubator-xtable]
Posted by "vinishjail97 (via GitHub)" <gi...@apache.org>.
vinishjail97 merged PR #716:
URL: https://github.com/apache/incubator-xtable/pull/716
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add intial IT for XTable REST Service [incubator-xtable]
Posted by "rahil-c (via GitHub)" <gi...@apache.org>.
rahil-c commented on PR #716:
URL: https://github.com/apache/incubator-xtable/pull/716#issuecomment-3002813911
@vinishjail97 can you merge this when you get a chance ci passes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@xtable.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org