jonathan-albrecht-ibm commented on PR #49811: URL: https://github.com/apache/spark/pull/49811#issuecomment-2671599293
Thanks for reviewing @HeartSaVioR > Could you please update the PR title and description, as well as JIRA? Especially for PR - we are using the title and description to fill out commit title/message. 🤦 I didn't think of this, thx. done > > Also, looks like linter is failing. Could you please fix it via running `dev/scalastyle` and fixing them? CI also gives you the error. 🤦 Thanks for pointing me to dev/scalastyle. done > > It'd be awesome if you could post the test run for these test suites to verify they are excluded, because I (and CI) can't verify the fix via running tests on that system. Here are the lines from the test run on s390x showing that the v1 tests are excluded and the v2 tests are included: ```txt - flatMapGroupsWithState - streaming with processing time timeout - state format version 1 !!! CANCELED !!! FlatMapGroupsWithStateSuite.this.isStateFormatSupported(FlatMapGroupsWithStateSuite.this.sqlConf.getConf[Int](org.apache.spark.sql.internal.SQLConf.FLATMAPGROUPSWITHSTATE_STATE_FORMAT_VERSION)) was false (FlatMapGroupsWithStateSuite.scala:471) -- - flatMapGroupsWithState - streaming with processing time timeout - state format version 2 -- - flatMapGroupsWithState - streaming with processing time timeout - state format version 1 (RocksDBStateStore) !!! CANCELED !!! FlatMapGroupsWithStateSuite.this.isStateFormatSupported(FlatMapGroupsWithStateSuite.this.sqlConf.getConf[Int](org.apache.spark.sql.internal.SQLConf.FLATMAPGROUPSWITHSTATE_STATE_FORMAT_VERSION)) was false (FlatMapGroupsWithStateSuite.scala:471) -- - flatMapGroupsWithState - streaming with processing time timeout - state format version 1 (RocksDBStateStore with changelog checkpointing) !!! CANCELED !!! FlatMapGroupsWithStateSuite.this.isStateFormatSupported(FlatMapGroupsWithStateSuite.this.sqlConf.getConf[Int](org.apache.spark.sql.internal.SQLConf.FLATMAPGROUPSWITHSTATE_STATE_FORMAT_VERSION)) was false (FlatMapGroupsWithStateSuite.scala:471) -- - flatMapGroupsWithState - streaming with processing time timeout - state format version 2 (RocksDBStateStore) -- - flatMapGroupsWithState - streaming with processing time timeout - state format version 2 (RocksDBStateStore with changelog checkpointing) -- - flatMapGroupsWithState - streaming w/ event time timeout + watermark - state format version 1 !!! CANCELED !!! FlatMapGroupsWithStateSuite.this.isStateFormatSupported(FlatMapGroupsWithStateSuite.this.sqlConf.getConf[Int](org.apache.spark.sql.internal.SQLConf.FLATMAPGROUPSWITHSTATE_STATE_FORMAT_VERSION)) was false (FlatMapGroupsWithStateSuite.scala:539) -- - flatMapGroupsWithState - streaming w/ event time timeout + watermark - state format version 2 -- - flatMapGroupsWithState - streaming w/ event time timeout + watermark - state format version 1 (RocksDBStateStore) !!! CANCELED !!! FlatMapGroupsWithStateSuite.this.isStateFormatSupported(FlatMapGroupsWithStateSuite.this.sqlConf.getConf[Int](org.apache.spark.sql.internal.SQLConf.FLATMAPGROUPSWITHSTATE_STATE_FORMAT_VERSION)) was false (FlatMapGroupsWithStateSuite.scala:539) -- - flatMapGroupsWithState - streaming w/ event time timeout + watermark - state format version 1 (RocksDBStateStore with changelog checkpointing) !!! CANCELED !!! FlatMapGroupsWithStateSuite.this.isStateFormatSupported(FlatMapGroupsWithStateSuite.this.sqlConf.getConf[Int](org.apache.spark.sql.internal.SQLConf.FLATMAPGROUPSWITHSTATE_STATE_FORMAT_VERSION)) was false (FlatMapGroupsWithStateSuite.scala:539) -- - flatMapGroupsWithState - streaming w/ event time timeout + watermark - state format version 2 (RocksDBStateStore) -- - flatMapGroupsWithState - streaming w/ event time timeout + watermark - state format version 2 (RocksDBStateStore with changelog checkpointing) -- - flatMapGroupsWithState, state ver 1 !!! CANCELED !!! java.nio.ByteOrder.nativeOrder().equals(java.nio.ByteOrder.LITTLE_ENDIAN) was false (StateDataSourceReadSuite.scala:802) -- - flatMapGroupsWithState, state ver 2 -- - flatMapGroupsWithState, state ver 1 !!! CANCELED !!! java.nio.ByteOrder.nativeOrder().equals(java.nio.ByteOrder.LITTLE_ENDIAN) was false (StateDataSourceReadSuite.scala:802) -- - flatMapGroupsWithState, state ver 2 -- - flatMapGroupsWithState, state ver 1 !!! CANCELED !!! java.nio.ByteOrder.nativeOrder().equals(java.nio.ByteOrder.LITTLE_ENDIAN) was false (StateDataSourceReadSuite.scala:802) -- - flatMapGroupsWithState, state ver 2 ``` -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org