Bug#25152165 DATA NODES RESTART AFTER ALTER ONLINE TABLE REORGANIZE PARTITION

Never send keydata in KEYINFO20 to SUMA.

Make Suma ignore sections for KEYINFO20.

Author: zmur

storage/ndb/src/kernel/blocks/dblqh/DblqhMain.cpp

@@ -13206,8 +13206,9 @@ Uint32 Dblqh::sendKeyinfo20(Signal* signal,
    *  messing with if's below...
    */
   Uint32 keyLen ;
-  if (refToMain(ref) == SUMA && nodeId == getOwnNodeId())
+  if (refToMain(ref) == SUMA)
   {
+    ndbassert(refToNode(ref) == getOwnNodeId());
     keyLen = 0;
   }
   else

storage/ndb/src/kernel/blocks/suma/Suma.cpp

@@ -1,5 +1,5 @@
 /*
-   Copyright (c) 2003, 2015, Oracle and/or its affiliates. All rights reserved.
+   Copyright (c) 2003, 2017, Oracle and/or its affiliates. All rights reserved.
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -4457,6 +4457,17 @@ Suma::execKEYINFO20(Signal* signal)
   ndbrequire(syncPtr.p->m_headersSection != RNIL);
   ndbrequire(syncPtr.p->m_dataSection != RNIL);
 
+  /* SUMA requests a special 'scanInfo only' KeyInfo */
+  ndbassert(data->keyLen == 0);
+  ndbassert(signal->getNoOfSections() == 0);
+
+  /* If there are sections, ignore them.  This should never happen. */
+  if (signal->getNoOfSections() > 0)
+  {
+    SectionHandle handle(this, signal);
+    releaseSections(handle);
+  }
+
   sendScanSubTableData(signal, syncPtr, takeOver);
 }