-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GraalVM support with testing #749
Closed
Closed
Changes from 9 commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
3496298
Added compatibiltiy
maschnetwork 4500fb5
Added compatibiltiy
maschnetwork 7a41775
Maven setup for native image testing
maschnetwork ed7aac5
Working Junit Tests
maschnetwork 76c9ed4
Working version with reduced dependencies and profile
maschnetwork d51a3bf
Fully working example
maschnetwork 4d4bbef
GraalVM Support incl. testing
maschnetwork bd5ed86
Removed JUnit reflection configuration, move vintage out of profile
maschnetwork a310d4d
feat: added new configuration files
maschnetwork 29b6a79
try to run a CI for graalVM
TingDaoK d091ef5
only enable graalvm
TingDaoK 95adfa0
retry
TingDaoK 51f9e0a
maybe try not import jdk8?
TingDaoK 2a972c4
farewell builder
TingDaoK c334564
17?
TingDaoK 8943f0b
double matrix
TingDaoK dd59763
no fail-fast
TingDaoK 87b4be7
seems like 8 and 11 are not available
TingDaoK 3930c68
eh
TingDaoK ff1f2e8
remove unneeded change
TingDaoK bc196ff
add one more
TingDaoK 3b843dd
fix resume token class cache
TingDaoK 34db048
Merge branch 'fix-pause-resume' into graalvm-support
TingDaoK f433f5f
windows naming is a bit different
TingDaoK ae14cd3
try to use builder for the mqtt tests
TingDaoK 6876e7d
it is called variant,,,
TingDaoK e48e28e
it's dict not list
TingDaoK e7be1dd
try this?
TingDaoK 55338a7
hmmmmm
TingDaoK afaefc4
setup the env var from builder as well
TingDaoK 54aece7
10secs is too fast..
TingDaoK 57a9336
no need to run those other tests again for GraalVM
TingDaoK 2f81a01
Updated GraalVM plugin versions and adapted to default for strict sta…
maschnetwork 094af5e
Example for registration via feature
maschnetwork 8e2c185
Revert "Example for registration via feature"
maschnetwork 28fa5d4
Removed build execution
maschnetwork File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just want to get your opinion, do we need the build phase? Should it be the application using our lib to build the native executable?
From my understanding, we will not package our lib as a native lib for other package to use, right??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct, for the current setup we don't need it. Let's remove it together with the tracing agent generation if we agree on the json vs. class format for the hints.