Github user tdas commented on a diff in the pull request:

    https://github.com/apache/spark/pull/3803#discussion_r23966691
  
    --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
    @@ -657,6 +657,10 @@ class SparkContext(config: SparkConf) extends Logging 
with ExecutorAllocationCli
        *
        * Load data from a flat binary file, assuming the length of each record 
is constant.
        *
    +   * '''Note:''' Normally getBytes returns an array padded with extra 
values,
    --- End diff --
    
    Does the end user ever need to call getBytes() given that this RDD is of 
type Array[Byte]? Does the user even need to know that this feature uses 
FixedLengthBinaryInputFormat underneath. And if they dont, then this note can 
be simplified and mentioned in terms of the RDD, something like "Each record in 
this DStream/RDD is of fixed length with padded bytes / right length / etc."


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to