@dsnp/parquetjs
Advanced tools
Comparing version 0.0.0-ce1f98 to 0.0.0-eb029b
@@ -29,3 +29,7 @@ "use strict"; | ||
const parquet_thrift = __importStar(require("../gen-nodejs/parquet_types")); | ||
// May not be needed anymore, Issue at https://github.com/LibertyDSNP/parquetjs/issues/41 | ||
/** | ||
* We need to patch Thrift's TFramedTransport class bc the TS type definitions | ||
* do not define a `readPos` field, even though the class implementation has | ||
* one. | ||
*/ | ||
class fixedTFramedTransport extends thrift_1.default.TFramedTransport { | ||
@@ -39,16 +43,3 @@ inBuf; | ||
} | ||
readString(len = 0) { | ||
this.ensureAvailable(len); | ||
var buffer = this.inBuf.slice(this.readPos, this.readPos + len); | ||
var str = this.inBuf.toString('utf8', this.readPos, this.readPos + len); | ||
this.readPos += len; | ||
//@ts-ignore | ||
return (Buffer.from(str).equals(buffer)) ? str : buffer; | ||
} | ||
} | ||
/** Patch PageLocation to be three element array that has getters/setters | ||
* for each of the properties (offset, compressed_page_size, first_row_index) | ||
* This saves space considerably as we do not need to store the full variable | ||
* names for every PageLocation | ||
*/ | ||
// Issue at https://github.com/LibertyDSNP/parquetjs/issues/42 | ||
@@ -55,0 +46,0 @@ const previousPageLocation = new parquet_thrift.PageLocation(); |
@@ -5,3 +5,3 @@ { | ||
"main": "dist/parquet.js", | ||
"version": "0.0.0-ce1f98", | ||
"version": "0.0.0-eb029b", | ||
"homepage": "https://github.com/LibertyDSNP/parquetjs", | ||
@@ -8,0 +8,0 @@ "license": "MIT", |
Sorry, the diff of this file is too big to display
Sorry, the diff of this file is too big to display
Sorry, the diff of this file is too big to display
Sorry, the diff of this file is not supported yet
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
2996753
17900