thrift icon indicating copy to clipboard operation
thrift copied to clipboard

Published 20 hours ago • verified publisher icon apache

add a rust GitHub action and cross test

Open jimexist opened this issue 3 years ago • 1 comments

  • closes #2594

as it replaces both

  • [ ] Did you create an Apache Jira ticket? (not required for trivial changes)
  • [ ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
  • [ ] Did you squash your changes to a single commit? (not required, but preferred)
  • [ ] Did you do your best to avoid breaking changes? If one was needed, did you label the Jira ticket with "Breaking-Change"?
  • [ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

jimexist avatar Jul 04 '22 03:07 jimexist

Unfortunately it looks like all the rust cross-tests are failing in the "Build/cross-test" action - even the rust<->rust ones, which should always work.

allengeorge avatar Jul 07 '22 02:07 allengeorge

i'm merging this given the failure is due to missing uuid impl in rust:

testUuid("00112233-4455-6677-8899-aabbccddeeff")
*** FAILURE ***
org.apache.thrift.TApplicationException: unknown method testUuid
	at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:81)
	at thrift.test.ThriftTest$Client.recv_testUuid(ThriftTest.java:527)
	at thrift.test.ThriftTest$Client.testUuid(ThriftTest.java:514)
	at org.apache.thrift.test.TestClient.main(TestClient.java:257)
Min time: 0us
Max time: 0us
Avg time: 0us

jimexist avatar Oct 08 '22 05:10 jimexist