You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wayang.apache.org by "damik3 (via GitHub)" <gi...@apache.org> on 2023/05/03 15:41:26 UTC
[GitHub] [incubator-wayang] damik3 opened a new pull request, #316: Added the following flink operator tests: MaterializedGroupByOperator, SortOperator, UnionAllOperator
damik3 opened a new pull request, #316:
URL: https://github.com/apache/incubator-wayang/pull/316
Note that the FlinkMaterializedGroupByOperatorTest, although it succeeds, it produces the following messages:
[ERROR] WARNING: An illegal reflective access operation has occurred
[ERROR] WARNING: Illegal reflective access by com.esotericsoftware.kryo.util.UnsafeUtil (file:/C:/Users/Mike/.m2/repository/com/esotericsoftware/kryo/kryo/2.24.0/kryo-2.24.0.jar) to constructor java.nio.DirectByteBuffer(long,int,java.lang.Object)
[ERROR] WARNING: Please consider reporting this to the maintainers of com.esotericsoftware.kryo.util.UnsafeUtil
[ERROR] WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
[ERROR] WARNING: All illegal access operations will be denied in a future release
These are also produced by its respective Spark test, SparkMaterializedGroupByOperatorTest.
--
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@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] damik3 commented on pull request #316: Added the following flink operator tests: MaterializedGroupByOperator, SortOperator, UnionAllOperator
Posted by "damik3 (via GitHub)" <gi...@apache.org>.
damik3 commented on PR #316:
URL: https://github.com/apache/incubator-wayang/pull/316#issuecomment-1534514954
> Do you think the error/warning comes from the AtomicInteger class that the tests are using?
No, I tried it without the AtomicInteger, but it's still there. I also found a discussion [here](https://github.com/EsotericSoftware/kryo/issues/626) and [here](https://github.com/EsotericSoftware/kryo/issues/747), but I don't think it's been solved. The `[ERROR] WARNING` prefix is a bit confusing, but I'd say it's more of a warning.
--
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@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] zkaoudi merged pull request #316: Added the following flink operator tests: MaterializedGroupByOperator, SortOperator, UnionAllOperator
Posted by "zkaoudi (via GitHub)" <gi...@apache.org>.
zkaoudi merged PR #316:
URL: https://github.com/apache/incubator-wayang/pull/316
--
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@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] zkaoudi commented on pull request #316: Added the following flink operator tests: MaterializedGroupByOperator, SortOperator, UnionAllOperator
Posted by "zkaoudi (via GitHub)" <gi...@apache.org>.
zkaoudi commented on PR #316:
URL: https://github.com/apache/incubator-wayang/pull/316#issuecomment-1534455958
Do you think the error/warning comes from the AtomicInteger class that the tests are using?
--
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@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-wayang] damik3 commented on pull request #316: Added the following flink operator tests: MaterializedGroupByOperator, SortOperator, UnionAllOperator
Posted by "damik3 (via GitHub)" <gi...@apache.org>.
damik3 commented on PR #316:
URL: https://github.com/apache/incubator-wayang/pull/316#issuecomment-1534466734
> Thank you Michalis for this PR! Can you please add the Apache licence headers in all the new files?
Oops, just added them!
--
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@wayang.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org